jcesarmobile opened a new pull request #1241: URL: https://github.com/apache/cordova-android/pull/1241
This is another approach to the java versions handling/requirement proposed here https://github.com/apache/cordova-android/pull/1230 At the moment we require java 8, the linked PR will require java 11, but I think we don't really need to require any specific version (or well, maybe >= java 8, but I don't think people is using java 7 or older at this point). So this PR just removes the java version check (it still checks that java is installed, but just "ignores" which version, as long as java is installed any version should be fine) At the moment I have java 11 installed because I use the one that Android Studio includes and Android Studio 4.2 started using java 11 instead of java 8, but I can still run cordova apps from the CLI that supposedly require java 8, and I think that java versions are backward compatible and if an user had java 16 it should also work (but didn't try). Even if we want to require java 11 and go with the linked PR, I think we should change the version check to >= 11 and not a strict 11, because some people work with java for server/desktop apps and use newer java versions and are forced to downgrade to use Cordova, when it looks like it's not really necessary. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
