cziegeler commented on code in PR #23:
URL: 
https://github.com/apache/sling-org-apache-sling-commons-log/pull/23#discussion_r2042494181


##########
pom.xml:
##########
@@ -285,6 +285,97 @@
 
     <build>
         <plugins>
+            <plugin>
+                <groupId>biz.aQute.bnd</groupId>
+                <artifactId>bnd-maven-plugin</artifactId>
+                <executions>
+                    <execution>
+                        <id>bnd-process</id>
+                        <configuration>
+                            <bnd><![CDATA[
+Bundle-Name: ${project.name}
+Bundle-SymbolicName: ${project.artifactId}
+Bundle-DocURL: http://sling.apache.org/site/logging.html
+Import-Package: org.osgi.service.cm;version="[1.2,2)";resolution:=dynamic, \
+    org.osgi.service.event;version="[1.2,2)";resolution:=dynamic, \
+    javax.xml.transform.*;resolution:=dynamic, \
+    org.slf4j.bridge;resolution:=optional, \
+    *
+
+Require-Capability: 
osgi.extender;filter:="(&(osgi.extender=osgi.serviceloader.registrar)(version>=1.0.0)(!(version>=2.0.0)))"
+Provide-Capability: 
osgi.serviceloader;osgi.serviceloader="ch.qos.logback.classic.spi.Configurator"
+]]></bnd>
+                            
<manifestPath>${project.build.directory}/nodeps-classes/META-INF/MANIFEST.MF</manifestPath>
+                            
<outputDir>${project.build.directory}/nodeps-classes</outputDir>
+                        </configuration>
+                    </execution>
+                    <execution>
+                        <id>bnd-process-deps</id>
+                        <goals>
+                            <goal>bnd-process</goal>
+                        </goals>
+                        <configuration>
+                            <bnd><![CDATA[
+Bundle-Name: ${project.name} (Including dependencies)
+Bundle-SymbolicName: ${project.artifactId}.all
+Bundle-DocURL: http://sling.apache.org/site/logging.html
+-includeresource:@logback-core-[0-9.]*(-SNAPSHOT)?.jar, \
+    @logback-classic-[0-9.]*(-SNAPSHOT)?.jar, \
+    @slf4j-api-[0-9.]*(-SNAPSHOT)?.jar
+Import-Package: org.osgi.service.cm;version="[1.2,2)";resolution:=dynamic, \
+    org.osgi.service.event;version="[1.2,2)";resolution:=dynamic, \
+    javax.xml.transform.*;resolution:=dynamic, \
+    org.slf4j.bridge;resolution:=optional, \
+    jakarta.mail;resolution:=optional, \
+    jakarta.mail.internet;resolution:=optional, \
+    jakarta.servlet;resolution:=optional, \
+    jakarta.servlet.http;resolution:=optional, \
+    org.codehaus.commons.compiler;resolution:=optional;version="[3.1,4)", \
+    org.codehaus.janino;resolution:=optional;version="[3.1,4)", \
+    org.fusesource.jansi;resolution:=optional;version="[2.4,3)", \
+    org.apache.commons.lang3;resolution:=optional;version="[3.0,4)", \
+    *
+Export-Package: ch.qos.logback.classic.*, \
+    ch.qos.logback.core.*, \
+    org.slf4j.*, \
+    org.slf4j;version="1.7.36", \
+    org.slf4j.helpers;version="1.7.36", \

Review Comment:
   Those are static and will never change. bnd correctly generates the right 
export versions for 2.x - but these two packages need also be exported with the 
latest 1.x release for compatibility.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@sling.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to