Update to 2024-06 release

eclipse-tycho · Jun 16, 2024 · 93c4072 · 93c4072
1 parent 44d2a90
commit 93c4072
Show file tree

Hide file tree

Showing 17 changed files with 60 additions and 52 deletions.
diff --git a/p2-maven-plugin/pom.xml b/p2-maven-plugin/pom.xml
@@ -37,7 +37,7 @@
 		<dependency>
 			<groupId>org.eclipse.platform</groupId>
 			<artifactId>org.eclipse.equinox.p2.publisher</artifactId>
-			<version>1.9.100</version>
+			<version>1.9.200</version>
 		</dependency>
 		<dependency>
 			<groupId>org.eclipse.platform</groupId>
@@ -47,7 +47,7 @@
 		<dependency>
 			<groupId>org.eclipse.platform</groupId>
 			<artifactId>org.eclipse.equinox.p2.engine</artifactId>
-			<version>2.10.0</version>
+			<version>2.10.200</version>
 		</dependency>
 		<dependency>
 			<groupId>org.eclipse.platform</groupId>
@@ -70,12 +70,12 @@
 		<dependency>
 			<groupId>org.eclipse.platform</groupId>
 			<artifactId>org.eclipse.equinox.frameworkadmin</artifactId>
-			<version>2.3.100</version>
+			<version>2.3.200</version>
 		</dependency>
 		<dependency>
 			<groupId>org.eclipse.platform</groupId>
 			<artifactId>org.eclipse.equinox.preferences</artifactId>
-			<version>3.11.0</version>
+			<version>3.11.100</version>
 		</dependency>
 		<dependency>
 			<groupId>org.apache.maven</groupId>
@@ -96,7 +96,7 @@
 		<dependency>
 			<groupId>org.eclipse.platform</groupId>
 			<artifactId>org.eclipse.equinox.p2.metadata.repository</artifactId>
-			<version>1.5.300</version>
+			<version>1.5.400</version>
 		</dependency>
 		<dependency>
 			<groupId>org.eclipse.platform</groupId>
@@ -123,27 +123,27 @@
 		<dependency>
 			<groupId>org.eclipse.platform</groupId>
 			<artifactId>org.eclipse.equinox.p2.jarprocessor</artifactId>
-			<version>1.3.300</version>
+			<version>1.3.400</version>
 		</dependency>
 		<dependency>
 			<groupId>org.eclipse.platform</groupId>
 			<artifactId>org.eclipse.equinox.p2.touchpoint.natives</artifactId>
-			<version>1.5.200</version>
+			<version>1.5.300</version>
 		</dependency>
 		<dependency>
 			<groupId>org.eclipse.platform</groupId>
 			<artifactId>org.eclipse.equinox.p2.touchpoint.eclipse</artifactId>
-			<version>2.4.200</version>
+			<version>2.4.300</version>
 		</dependency>
 		<dependency>
 			<groupId>org.eclipse.platform</groupId>
 			<artifactId>org.eclipse.equinox.p2.garbagecollector</artifactId>
-			<version>1.3.200</version>
+			<version>1.3.300</version>
 		</dependency>
 		<dependency>
 			<groupId>org.eclipse.platform</groupId>
 			<artifactId>org.eclipse.equinox.p2.director.app</artifactId>
-			<version>1.3.300</version>
+			<version>1.3.400</version>
 		</dependency>
 		<dependency>
 			<groupId>org.eclipse.platform</groupId>
@@ -154,12 +154,12 @@
 		<dependency>
 			<groupId>org.eclipse.platform</groupId>
 			<artifactId>org.eclipse.equinox.concurrent</artifactId>
-			<version>1.3.0</version>
+			<version>1.3.100</version>
 		</dependency>
 		<dependency>
 			<groupId>org.eclipse.platform</groupId>
 			<artifactId>org.eclipse.equinox.simpleconfigurator</artifactId>
-			<version>1.5.200</version>
+			<version>1.5.300</version>
 		</dependency>
 		<dependency>
 			<groupId>org.eclipse.platform</groupId>
@@ -169,7 +169,7 @@
 		<dependency>
 			<groupId>org.eclipse.platform</groupId>
 			<artifactId>org.eclipse.equinox.frameworkadmin.equinox</artifactId>
-			<version>1.3.100</version>
+			<version>1.3.200</version>
 		</dependency>
 		<dependency>
 			<groupId>org.eclipse.platform</groupId>

diff --git a/pom.xml b/pom.xml
@@ -297,17 +297,17 @@
 			<dependency>
 				<groupId>org.eclipse.platform</groupId>
 				<artifactId>org.eclipse.equinox.p2.repository</artifactId>
-				<version>2.8.100</version>
+				<version>2.9.100</version>
 			</dependency>
 			<dependency>
 				<groupId>org.eclipse.platform</groupId>
 				<artifactId>org.eclipse.equinox.p2.publisher.eclipse</artifactId>
-				<version>1.6.0</version>
+				<version>1.6.100</version>
 			</dependency>
 			<dependency>
 				<groupId>org.eclipse.platform</groupId>
 				<artifactId>org.eclipse.equinox.simpleconfigurator.manipulator</artifactId>
-				<version>2.3.100</version>
+				<version>2.3.200</version>
 			</dependency>
 			<dependency>
 				<groupId>org.bouncycastle</groupId>

diff --git a/tycho-api/pom.xml b/tycho-api/pom.xml
@@ -33,7 +33,7 @@
 		<dependency>
 			<groupId>org.eclipse.platform</groupId>
 			<artifactId>org.eclipse.equinox.p2.publisher</artifactId>
-			<version>1.9.100</version>
+			<version>1.9.200</version>
 		</dependency>
 	</dependencies>
 	<build>

diff --git a/tycho-apitools-plugin/pom.xml b/tycho-apitools-plugin/pom.xml
@@ -50,7 +50,7 @@
 		<dependency>
 		    <groupId>org.eclipse.pde</groupId>
 		    <artifactId>org.eclipse.pde.core</artifactId>
-		    <version>3.18.0</version>
+		    <version>3.18.100</version>
 		    <type>jar</type>
 		    <exclusions>
                 <exclusion>

diff --git a/tycho-build/pom.xml b/tycho-build/pom.xml
@@ -59,7 +59,7 @@
 		<dependency>
 			<groupId>org.eclipse.platform</groupId>
 			<artifactId>org.eclipse.equinox.p2.publisher</artifactId>
-			<version>1.9.100</version>
+			<version>1.9.200</version>
 		</dependency>
 
 		<dependency>
@@ -71,7 +71,7 @@
 		<dependency>
 			<groupId>org.eclipse.platform</groupId>
 			<artifactId>org.eclipse.equinox.p2.engine</artifactId>
-			<version>2.10.0</version>
+			<version>2.10.200</version>
 		</dependency>
 
 
@@ -123,7 +123,7 @@
 		<dependency>
 			<groupId>org.eclipse.platform</groupId>
 			<artifactId>org.eclipse.equinox.p2.metadata.repository</artifactId>
-			<version>1.5.300</version>
+			<version>1.5.400</version>
 		</dependency>
 		<dependency>
 			<groupId>org.eclipse.tycho</groupId>

diff --git a/tycho-core/pom.xml b/tycho-core/pom.xml
@@ -319,7 +319,7 @@
 		<dependency>
 		    <groupId>org.eclipse.pde</groupId>
 		    <artifactId>org.eclipse.pde.core</artifactId>
-		    <version>3.18.0</version>
+		    <version>3.18.100</version>
 		    <scope>test</scope>
 		    <exclusions>
 				<exclusion>

diff --git a/tycho-core/src/main/java/org/eclipse/tycho/core/resolver/MavenTargetDefinitionContent.java b/tycho-core/src/main/java/org/eclipse/tycho/core/resolver/MavenTargetDefinitionContent.java
@@ -350,7 +350,7 @@ public MavenTargetDefinitionContent(MavenGAVLocation location, MavenDependencies
                     throw new TargetDefinitionResolutionException("feature generation failed!", e);
                 }
             }
-            FeaturePublisher.publishFeatures(features, repositoryContent::put, logger);
+            FeaturePublisher.publishFeatures(features, repositoryContent::put, artifactRepository, logger);
         }
     }
 
@@ -396,7 +396,7 @@ private IInstallableUnit publish(BundleDescription bundleDescription, File bundl
     private IInstallableUnit publish(BundleDescription bundleDescription, File bundleLocation, IPropertyAdvice advice) {
         IArtifactKey key = BundlesAction.createBundleArtifactKey(bundleDescription.getSymbolicName(),
                 bundleDescription.getVersion().toString());
-        IArtifactDescriptor descriptor = FileArtifactRepository.forFile(bundleLocation, key);
+        IArtifactDescriptor descriptor = FileArtifactRepository.forFile(bundleLocation, key, artifactRepository);
         PublisherInfo publisherInfo = new PublisherInfo();
         publisherInfo.addAdvice(advice);
         publisherInfo.addAdvice(new MavenChecksumAdvice(bundleLocation));

diff --git a/tycho-core/src/main/java/org/eclipse/tycho/core/resolver/target/FileArtifactRepository.java b/tycho-core/src/main/java/org/eclipse/tycho/core/resolver/target/FileArtifactRepository.java
@@ -39,6 +39,7 @@
 import org.eclipse.equinox.p2.query.IQueryResult;
 import org.eclipse.equinox.p2.query.IQueryable;
 import org.eclipse.equinox.p2.repository.artifact.IArtifactDescriptor;
+import org.eclipse.equinox.p2.repository.artifact.IArtifactRepository;
 import org.eclipse.equinox.p2.repository.artifact.IArtifactRequest;
 import org.eclipse.equinox.p2.repository.artifact.IFileArtifactRepository;
 import org.eclipse.equinox.p2.repository.artifact.spi.AbstractArtifactRepository;
@@ -222,17 +223,18 @@ private static boolean matches(IArtifactKey key, IArtifactDescriptor descriptor)
                 && key.getVersion().equals(descriptorKey.getVersion()));
     }
 
-    public static IArtifactDescriptor forFile(File file, IArtifactKey key) {
-        return new FileArtifactDescriptor(file, key);
+    public static IArtifactDescriptor forFile(File file, IArtifactKey key, IArtifactRepository repository) {
+        return new FileArtifactDescriptor(file, key, repository);
     }
 
     private static final class FileArtifactDescriptor extends ArtifactDescriptor {
 
         private File file;
 
-        private FileArtifactDescriptor(File file, IArtifactKey key) {
+        private FileArtifactDescriptor(File file, IArtifactKey key, IArtifactRepository repository) {
             super(key);
             this.file = file;
+            setRepository(repository);
         }
 
     }

diff --git a/tycho-core/src/main/java/org/eclipse/tycho/p2/resolver/BundlePublisher.java b/tycho-core/src/main/java/org/eclipse/tycho/p2/resolver/BundlePublisher.java
@@ -33,12 +33,12 @@
 import org.eclipse.equinox.p2.publisher.Publisher;
 import org.eclipse.equinox.p2.publisher.PublisherInfo;
 import org.eclipse.equinox.p2.publisher.actions.IPropertyAdvice;
-import org.eclipse.tycho.p2maven.tmp.BundlesAction;
 import org.eclipse.equinox.p2.repository.artifact.IArtifactDescriptor;
 import org.eclipse.equinox.p2.repository.artifact.IArtifactRepository;
 import org.eclipse.equinox.p2.repository.metadata.IMetadataRepository;
 import org.eclipse.osgi.service.resolver.BundleDescription;
 import org.eclipse.tycho.core.resolver.target.FileArtifactRepository;
+import org.eclipse.tycho.p2maven.tmp.BundlesAction;
 import org.osgi.framework.BundleException;
 
 public class BundlePublisher extends BundlesAction {
@@ -47,8 +47,8 @@ private BundlePublisher(BundleDescription bundleDescription) {
         super(new BundleDescription[] { bundleDescription });
     }
 
-    public static Optional<IInstallableUnit> getBundleIU(File bundleLocation, IPublisherAdvice... advices)
-            throws IOException, BundleException {
+    public static Optional<IInstallableUnit> getBundleIU(File bundleLocation, IArtifactRepository repository,
+            IPublisherAdvice... advices) throws IOException, BundleException {
         BundleDescription bundleDescription = BundlesAction.createBundleDescription(bundleLocation);
         if (bundleDescription == null) {
             //seems it is not a bundle
@@ -65,7 +65,7 @@ public static Optional<IInstallableUnit> getBundleIU(File bundleLocation, IPubli
         }
         IArtifactKey key = BundlesAction.createBundleArtifactKey(symbolicName,
                 bundleDescription.getVersion().toString());
-        IArtifactDescriptor descriptor = FileArtifactRepository.forFile(bundleLocation, key);
+        IArtifactDescriptor descriptor = FileArtifactRepository.forFile(bundleLocation, key, repository);
         return Optional.ofNullable(publishBundle(bundleDescription, descriptor, publisherInfo));
     }
 

diff --git a/tycho-core/src/main/java/org/eclipse/tycho/p2/resolver/FeaturePublisher.java b/tycho-core/src/main/java/org/eclipse/tycho/p2/resolver/FeaturePublisher.java
@@ -42,7 +42,8 @@
 public class FeaturePublisher {
 
     public static void publishFeatures(List<Feature> features,
-            BiConsumer<IArtifactDescriptor, IInstallableUnit> consumer, MavenLogger logger) {
+            BiConsumer<IArtifactDescriptor, IInstallableUnit> consumer, IArtifactRepository artifactRepository,
+            MavenLogger logger) {
         if (features.isEmpty()) {
             return;
         }
@@ -85,7 +86,7 @@ protected IInstallableUnit generateFeatureJarIU(Feature feature, IPublisherInfo
                     //The actual published feature...
                     for (IArtifactKey key : artifacts) {
                         IArtifactDescriptor fileDescriptor = FileArtifactRepository
-                                .forFile(new File(publishedFeature.getLocation()), key);
+                                .forFile(new File(publishedFeature.getLocation()), key, artifactRepository);
                         consumer.accept(fileDescriptor, unit);
                     }
                 }

diff --git a/tycho-core/src/main/java/org/eclipse/tycho/p2/resolver/FileTargetDefinitionContent.java b/tycho-core/src/main/java/org/eclipse/tycho/p2/resolver/FileTargetDefinitionContent.java
@@ -27,7 +27,6 @@
 import org.eclipse.equinox.p2.metadata.IInstallableUnit;
 import org.eclipse.equinox.p2.publisher.IPublisherInfo;
 import org.eclipse.equinox.p2.publisher.PublisherInfo;
-import org.eclipse.tycho.p2maven.tmp.BundlesAction;
 import org.eclipse.equinox.p2.publisher.eclipse.Feature;
 import org.eclipse.equinox.p2.query.IQuery;
 import org.eclipse.equinox.p2.query.IQueryResult;
@@ -37,6 +36,7 @@
 import org.eclipse.osgi.service.resolver.BundleDescription;
 import org.eclipse.tycho.core.resolver.target.FileArtifactRepository;
 import org.eclipse.tycho.core.resolver.target.SupplierMetadataRepository;
+import org.eclipse.tycho.p2maven.tmp.BundlesAction;
 import org.eclipse.tycho.targetplatform.TargetDefinitionContent;
 import org.eclipse.tycho.targetplatform.TargetDefinitionResolutionException;
 import org.osgi.framework.BundleException;
@@ -85,14 +85,16 @@ public synchronized void preload(IProgressMonitor monitor) {
             boolean hasPlugins = pluginsPath.isDirectory();
             boolean hasFeatures = featurePath.isDirectory();
             if (hasPlugins) {
-                readBundles(pluginsPath, repositoryContent::put, hasFeatures ? subMonitor.split(100) : subMonitor);
+                readBundles(pluginsPath, repositoryContent::put, hasFeatures ? subMonitor.split(100) : subMonitor,
+                        artifactRepository);
             }
             if (hasFeatures) {
-                readFeatures(featurePath, repositoryContent::put, hasPlugins ? subMonitor.split(100) : subMonitor);
+                readFeatures(featurePath, repositoryContent::put, artifactRepository,
+                        hasPlugins ? subMonitor.split(100) : subMonitor);
             }
             if (!hasFeatures && !hasPlugins) {
-                readBundles(location, repositoryContent::put, subMonitor.split(100));
-                readFeatures(location, repositoryContent::put, subMonitor.split(100));
+                readBundles(location, repositoryContent::put, subMonitor.split(100), artifactRepository);
+                readFeatures(location, repositoryContent::put, artifactRepository, subMonitor.split(100));
             }
         } catch (ResolverException e) {
             throw new TargetDefinitionResolutionException("resolving location " + location + " failed", e);
@@ -112,7 +114,7 @@ private Map<IArtifactDescriptor, IInstallableUnit> getRepositoryContent() {
     }
 
     private static void readFeatures(File path, BiConsumer<IArtifactDescriptor, IInstallableUnit> consumer,
-            IProgressMonitor monitor) {
+            IArtifactRepository artifactRepository, IProgressMonitor monitor) {
         PublisherInfo publisherInfo = new PublisherInfo();
         publisherInfo.setArtifactOptions(IPublisherInfo.A_INDEX);
         if (path.isDirectory()) {
@@ -132,7 +134,7 @@ private static void readFeatures(File path, BiConsumer<IArtifactDescriptor, IIns
                     Feature feature = new FeatureParser().parse(featureLocation);
                     if (feature != null) {
                         feature.setLocation(featureLocation.getAbsolutePath());
-                        FeaturePublisher.publishFeatures(List.of(feature), consumer, null);
+                        FeaturePublisher.publishFeatures(List.of(feature), consumer, artifactRepository, null);
                     }
                     subMonitor.worked(1);
                 }
@@ -141,7 +143,7 @@ private static void readFeatures(File path, BiConsumer<IArtifactDescriptor, IIns
     }
 
     private static void readBundles(File path, BiConsumer<IArtifactDescriptor, IInstallableUnit> consumer,
-            IProgressMonitor monitor) throws ResolverException {
+            IProgressMonitor monitor, IArtifactRepository artifactRepository) throws ResolverException {
         PublisherInfo publisherInfo = new PublisherInfo();
         publisherInfo.setArtifactOptions(IPublisherInfo.A_INDEX);
         if (path.isDirectory()) {
@@ -163,7 +165,7 @@ private static void readBundles(File path, BiConsumer<IArtifactDescriptor, IInst
                         if (bundleDescription != null) {
                             IArtifactKey key = BundlesAction.createBundleArtifactKey(
                                     bundleDescription.getSymbolicName(), bundleDescription.getVersion().toString());
-                            consumer.accept(FileArtifactRepository.forFile(bundleLocation, key),
+                            consumer.accept(FileArtifactRepository.forFile(bundleLocation, key, artifactRepository),
                                     BundlesAction.createBundleIU(bundleDescription, key, publisherInfo));
                         }
                     } catch (BundleException | IOException | RuntimeException e) {

diff --git a/tycho-core/src/main/java/org/eclipse/tycho/p2resolver/PomDependencyCollectorImpl.java b/tycho-core/src/main/java/org/eclipse/tycho/p2resolver/PomDependencyCollectorImpl.java
@@ -61,12 +61,14 @@ public class PomDependencyCollectorImpl implements PomDependencyCollector {
     private final List<IArtifactDescriptor> fileDescriptors = new ArrayList<>();
     private FileRepositoryArtifactProvider fileRepositoryArtifactProvider;
 
+    private FileArtifactRepository artifactRepository;
+
     public PomDependencyCollectorImpl(Logger logger, ReactorProject project, IProvisioningAgent agent) {
         this.logger = logger;
         this.project = project;
+        artifactRepository = new FileArtifactRepository(agent, fileDescriptors::iterator);
         fileRepositoryArtifactProvider = new FileRepositoryArtifactProvider(
-                Collections.singletonList(new FileArtifactRepository(agent, fileDescriptors::iterator)),
-                ArtifactTransferPolicies.forLocalArtifacts());
+                Collections.singletonList(artifactRepository), ArtifactTransferPolicies.forLocalArtifacts());
     }
 
     public File getProjectLocation() {
@@ -85,7 +87,8 @@ public Entry<ArtifactKey, IArtifactDescriptor> addMavenArtifact(IArtifactFacade
             for (IArtifactKey key : unit.getArtifacts()) {
                 ArtifactKey artifactKey = ArtifactTypeHelper.toTychoArtifactKey(unit, key);
                 if (artifactKey != null) {
-                    IArtifactDescriptor descriptor = FileArtifactRepository.forFile(artifact.getLocation(), key);
+                    IArtifactDescriptor descriptor = FileArtifactRepository.forFile(artifact.getLocation(), key,
+                            artifactRepository);
                     fileDescriptors.add(descriptor);
                     descriptorMap.put(artifact, descriptor);
                     resultArtifactKey = new SimpleEntry<>(artifactKey, descriptor);