Skip to content

Commit 54b8cc8

Browse files
committed
[I/Y-Build] Convert Equinox SDK to p2-repository
And prevent the Eclipse OSGi bundle from being listed second time in the 'Other Required Bundles' section at the Equinox build site.
1 parent 53971a1 commit 54b8cc8

File tree

9 files changed

+24
-226
lines changed

9 files changed

+24
-226
lines changed

JenkinsJobs/Builds/build.jenkinsfile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -431,8 +431,8 @@ pipeline {
431431
popd
432432

433433
# Gather Equinox SDK
434-
pushd ${PRODUCTS_DIR}/equinox-sdk/target
435-
cp equinox-sdk-*-SNAPSHOT.zip ${EQUINOX_DROP_DIR}/${BUILD_ID}/equinox-SDK-$BUILD_ID.zip
434+
pushd ${SITES_DIR}/equinox-sdk-repository/target
435+
cp equinox-sdk-repository.eclipse-repository-*.zip ${EQUINOX_DROP_DIR}/${BUILD_ID}/equinox-SDK-$BUILD_ID.zip
436436
popd
437437
pushd ${EQUINOX_DROP_DIR}/${BUILD_ID}
438438
unzip -o -j equinox-SDK-$BUILD_ID.zip plugins/*.jar -x plugins/*.source_*

products/equinox-sdk/equinox-sdk.product

Lines changed: 0 additions & 165 deletions
This file was deleted.

products/equinox-sdk/pom.xml

Lines changed: 0 additions & 49 deletions
This file was deleted.

products/pom.xml

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -33,11 +33,8 @@
3333
<module>equinox-launcher</module>
3434
<module>equinox-starterkit</module>
3535
<module>eclipse-junit-tests</module>
36-
<module>equinox-sdk</module>
3736
</modules>
3837

39-
<!-- TODO: Move site like products (e.g. equinox-sdk and junit-tests) into sites folder, which also avoids inheriting (for them) unwanted configuration?-->
40-
4138
<properties>
4239
<!-- If product.installSources is set to true,
4340
repository.includeAllSources must always be true too.
@@ -97,7 +94,7 @@
9794
</plugins>
9895
</pluginManagement>
9996
</build>
100-
97+
10198
<profiles>
10299
<profile>
103100
<id>eclipse-sign</id>

scripts/releng/BuildDropDataGenerator.java

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,8 @@
1313
* Hannes Wellmann - initial API and implementation
1414
*******************************************************************************/
1515

16+
import static java.util.function.Predicate.not;
17+
1618
import java.util.Map.Entry;
1719

1820
import utilities.JSON;
@@ -139,15 +141,16 @@ void mainEquinoxPageData() throws IOException {
139141
buildProperties.add("equinoxRepository",
140142
collectFileEntries(files, filename -> filename.startsWith("equinox-SDK-")));
141143

142-
buildProperties.add("equinoxFramework",
143-
collectFileEntries(files, filename -> filename.startsWith("org.eclipse.osgi_")));
144+
Predicate<String> isOSGiBundle = filename -> filename.startsWith("org.eclipse.osgi_");
145+
buildProperties.add("equinoxFramework", collectFileEntries(files, isOSGiBundle));
144146

145147
Predicate<String> isAddonBundle = filename -> (filename.startsWith("org.eclipse.equinox.")
146148
|| filename.startsWith("org.eclipse.osgi.")) && !filename.startsWith("org.eclipse.equinox.p2.");
147149
Predicate<String> isJar = filename -> filename.endsWith(".jar");
148150

149151
buildProperties.add("addonBundles", collectFileEntries(files, isAddonBundle.and(isJar)));
150-
buildProperties.add("otherBundles", collectFileEntries(files, isAddonBundle.negate().and(isJar)));
152+
buildProperties.add("otherBundles",
153+
collectFileEntries(files, not(isAddonBundle).and(not(isOSGiBundle)).and(isJar)));
151154

152155
buildProperties.add("launchers", collectFileEntries(files, filename -> filename.startsWith("launchers-")));
153156

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
11
# Maven/Tycho pom model adjustments, see https://github.com/eclipse/tycho/wiki/Tycho-Pomless
22
pom.model.property.repository.name = Eclipse ${releaseVersion}
3+
pom.model.property.repository.includeAllDependencies = true
34
pom.model.property.eclipse.signing.skip = false
Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
# Maven/Tycho pom model adjustments, see https://github.com/eclipse-tycho/tycho/wiki/Tycho-Pomless
2+
pom.model.property.repository.name = Equinox and Eclipse p2 ${releaseVersion}
3+
pom.model.property.repository.includeAllDependencies = false
4+
pom.model.property.eclipse.signing.skip = false
Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
<?xml version="1.0" encoding="UTF-8"?>
2+
<site>
3+
<feature id="org.eclipse.equinox.sdk"/>
4+
<feature id="org.eclipse.equinox.p2.sdk"/>
5+
<feature id="org.eclipse.core.runtime.feature"/>
6+
</site>

sites/pom.xml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
1717
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd">
1818
<modelVersion>4.0.0</modelVersion>
19-
19+
2020
<parent>
2121
<groupId>org.eclipse</groupId>
2222
<artifactId>eclipse-platform-parent</artifactId>
@@ -29,6 +29,7 @@
2929

3030
<modules>
3131
<module>eclipse-platform-repository</module>
32+
<module>equinox-sdk-repository</module>
3233
</modules>
3334

3435
<build>
@@ -38,15 +39,15 @@
3839
<groupId>org.eclipse.tycho</groupId>
3940
<artifactId>tycho-p2-repository-plugin</artifactId>
4041
<configuration>
41-
<includeAllDependencies>true</includeAllDependencies>
42+
<includeAllDependencies>${repository.includeAllDependencies}</includeAllDependencies>
4243
<includeAllSources>true</includeAllSources>
4344
<repositoryName>${repository.name}</repositoryName>
4445
</configuration>
4546
</plugin>
4647
</plugins>
4748
</pluginManagement>
4849
</build>
49-
50+
5051
<profiles>
5152
<profile>
5253
<id>eclipse-sign</id>

0 commit comments

Comments
 (0)