Removes Require-Capability for osgi.service AnyService

juergen-albert · pkriens · commit 1b4ca76fb8b5 · 2023-09-12T12:07:29.000+02:00
fixes #5750 Signed-off-by: Juergen Albert <j.albert@data-in-motion.biz> --- Signed-off-by: Peter Kriens <Peter.Kriens@aQute.biz>
diff --git a/biz.aQute.bndlib.tests/test/test/component/DSAnnotationTest.java b/biz.aQute.bndlib.tests/test/test/component/DSAnnotationTest.java
@@ -4395,7 +4395,6 @@ public void anyservice() throws Exception {
 			assertOk(b);
 			Attributes a = getAttr(jar);
 			checkProvides(a);
-			checkRequires(a, ComponentConstants.COMPONENT_SPECIFICATION_VERSION, AnyService.class.getName());
 
 			//
 			// Test all the defaults
@@ -4444,6 +4443,25 @@ public void anyservice() throws Exception {
 		}
 	}
 
+	@Test
+	public void anyserviceNoServiceRequirement() throws Exception {
+		try (Builder b = new Builder()) {
+			b.setProperty(Constants.DSANNOTATIONS, "test.component.DSAnnotationTest$AnyServiceUse");
+			b.setProperty("Private-Package", "test.component");
+			b.addClasspath(new File("bin_test"));
+
+			Jar jar = b.build();
+			assertOk(b);
+			Attributes a = getAttr(jar);
+			checkProvides(a);
+
+			jar.getManifest()
+				.write(System.out);
+
+			assertThat(a.getValue(Constants.REQUIRE_CAPABILITY)).doesNotContain(AnyService.class.getName());
+		}
+	}
+
 	@Component
 	public static class AnyServiceUseNoObject {
 		@Activate
diff --git a/biz.aQute.bndlib/src/aQute/bnd/component/DSAnnotations.java b/biz.aQute.bndlib/src/aQute/bnd/component/DSAnnotations.java
@@ -318,6 +318,9 @@ private void addServiceCapability(ComponentDef definition, Set<String> provides,
 
 	private void addServiceRequirement(ReferenceDef ref, MergedRequirement requires) {
 		String objectClass = ref.service;
+		if ("org.osgi.service.component.AnyService".equals(objectClass)) {
+			return;
+		}
 		ReferenceCardinality cardinality = ref.cardinality;
 		boolean optional = cardinality == ReferenceCardinality.OPTIONAL || cardinality == ReferenceCardinality.MULTIPLE;
 		boolean multiple = cardinality == ReferenceCardinality.MULTIPLE
