bosschaert commented on a change in pull request #2:
URL:
https://github.com/apache/sling-org-apache-sling-javax-activation/pull/2#discussion_r460173604
##########
File path: pom.xml
##########
@@ -77,6 +100,7 @@
<dependency>
<groupId>org.osgi</groupId>
<artifactId>org.osgi.core</artifactId>
+ <version>4.2.0</version>
Review comment:
OSGi R 4.2 is the minimum OSGi API version that this code needs. It does
not require anything newer than that right now so why increase the APIs that it
uses higher than what it actually needs? It will work on any OSGi version after
that.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]