vorburger commented on issue #609: require Java 11
URL: https://github.com/apache/fineract/pull/609#issuecomment-509779718
 
 
   > > This has a number of pre-requisites
   > 
   > including #608, and possibly 
[radcortez/openjpa-gradle-plugin#8](https://github.com/radcortez/openjpa-gradle-plugin/issues/8)
   
   most importantly, this will require #525 to get done first, which we should 
focus on... @lingalamadhukar what I did when I started this PR was locally 
merge #525 into it (I just didn't want to push that onto this PR), then the 
problems re. OpenJPA you describe above are gone (for me). Sorry if this wasn't 
made clear enough here.
   
   So what I'd like to do is finish up #525 without Java 11, and only then get 
back to this. As for the other purely Java 11 related point you raised over in 
#525, some feedback:
   
   > > how weird that I didn't hit this...
   > 
   > P.S. @vorburger , [_The apache#609 CI failure_](http://bit.ly/Trvs609J11) 
is most likely due to the last change to 
[_fineract-provider/gradle/wrapper/gradle-wrapper.properties_](http://bit.ly/GrdlWrpr210)
 being four+ years ago. Version 2.10 of the wrapper needs to be updated to at 
least 5.2.x I believe.
   
   Nah, that's solved by the 2nd commit currently on this PR which switches 
Travis CI to build using Java 11 instead of 8.

----------------------------------------------------------------
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]


With regards,
Apache Git Services

Reply via email to