Skip to content

Commit

Permalink
Merge pull request #6000 from timothyjward/resolver-verify
Browse files Browse the repository at this point in the history
Fixes for the bnd-resolver-maven-plugin verification
  • Loading branch information
timothyjward committed Jan 29, 2024
2 parents 23a72d6 + 2a56d93 commit 9b1025c
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 4 deletions.
11 changes: 11 additions & 0 deletions maven-plugins/bnd-resolver-maven-plugin/src/it/verify/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,14 @@
<it.resolve.skip>true</it.resolve.skip>
</properties>

<dependencies>
<dependency>
<groupId>com.fasterxml.jackson.core</groupId>
<artifactId>jackson-databind</artifactId>
<version>2.11.1</version>
</dependency>
</dependencies>

<build>
<plugins>
<plugin>
Expand All @@ -29,6 +37,9 @@
<goals>
<goal>verify</goal>
</goals>
<configuration>
<skip>false</skip>
</configuration>
</execution>
</executions>
</plugin>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
-standalone: ${projectsDirectory}/index/index.xml.gz
-runfw: org.apache.felix.framework
-runrequires: osgi.identity;filter:='(osgi.identity=org.apache.felix.eventadmin)'
-runrequires: osgi.identity;filter:='(osgi.identity=com.fasterxml.jackson.core.jackson-databind)'

-runbundles: org.apache.felix.eventadmin;version="[1.4.6,1.4.7)"
-runbundles: com.fasterxml.jackson.core.jackson-databind;version='[2.11.1,2.11.2)',\
com.fasterxml.jackson.core.jackson-core;version='[2.11.1,2.11.2)',\
com.fasterxml.jackson.core.jackson-annotations;version='[2.11.1,2.11.2)'

Original file line number Diff line number Diff line change
Expand Up @@ -243,7 +243,7 @@ public void processCandidates(Requirement requirement, Set<Capability> wired, Li
Capability id = ResourceUtils.getIdentityCapability(it.next()
.getResource());
if (bundleRequirements.stream()
.noneMatch(r -> ResourceUtils.matches(requirement, id))) {
.noneMatch(r -> ResourceUtils.matches(r, id))) {
it.remove();
}
}
Expand Down

0 comments on commit 9b1025c

Please sign in to comment.