kwin commented on code in PR #295:
URL: 
https://github.com/apache/jackrabbit-filevault/pull/295#discussion_r1195001323


##########
parent/pom.xml:
##########
@@ -373,7 +374,7 @@ Bundle-Category: jackrabbit
             <dependency>
                 <groupId>org.apache.jackrabbit</groupId>
                 <artifactId>oak-jackrabbit-api</artifactId>
-                <version>${oak.version}</version>
+                <version>${oak.version.compile}</version>

Review Comment:
   I know that you disagree here, but again, that is the job of the 
distribution to ship current runtime dependencies. Applications only depend on 
the minimum API level version which is required (as long as the newest version 
is backwards compatible with it, i.e. fulfills the package-import version 
range). No need to for FileVault to no longer support being compatible with 
unmaintained API versions.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to