alien11689 commented on code in PR #299:
URL: https://github.com/apache/aries/pull/299#discussion_r1903328157


##########
blueprint/blueprint-spring/pom.xml:
##########
@@ -119,13 +119,13 @@
         <dependency>
             <groupId>org.springframework</groupId>
             <artifactId>spring-context</artifactId>
-            <version>4.2.2.RELEASE</version>
+            <version>6.1.15</version>

Review Comment:
   There are a lot of possible users who stay with Java 8 so it's better to 
keep all the artifacts compatible with Java 8 if possible. Especially that 
Spring is not often used in OSGi projects.
   
   But personally I see the value in using the newest version of Spring. 
   Maybe the solution here may be creating the new artifact 
`blueprint-spring6`. Existing `blueprint-spring`  could use newest spring 5.x 
version supporting Java 8. Perfectly the code should be reused between modules 
so the new module should only override necessary parts (if possible).
   WDYT?



-- 
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...@aries.apache.org

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

Reply via email to