-
Notifications
You must be signed in to change notification settings - Fork 57
Update System Requirements to Java 17 as per Java section. #176
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…ection of documentation.
|
I think the difference here is that java 11 should work for most targets, it's just android that requires a higher version. Perhaps rather than increasing the minimum requirements we should add a note about Android target? |
marchbold
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a distinction between minimum requirements for AIR vs the android target.
|
Thank you for clarifying that @marchbold. With respect to AIR vs Android targets, it was something I wasn't sure about myself. Perhaps something to consider, albeit not related to compatibility but rather availability, is the JDK 17 link (added in #177) seems to be unique in that it does not require an Oracle account in order to download it. JDK 11 on the other hand, does require an Oracle account to download. My target is AIR not Android, but I grabbed JDK 17 on that basis alone. Perhaps though, I've missed another legitimate way of downloading JDK 11 that does not require an Oracle account? |
|
I guess the question I didn't quite ask there is, are there any issues with using the more available JDK 17 for AIR targets? If there aren't any issues, perhaps a note might offer that JDK 11 is recommended for non-Android targets, but JDK 17 will also suffice. |
|
No issues using JDK 17 for all, I'm actually using 21 mostly now. |
|
It's not that 11 is recommended just that it's the minimum. Personally I agree with you and would say a higher version is recommended (either 17 or 21). Maybe it should read "
|
|
Hi @marchbold, I've updated it to specify JDK 11 as the minimum version required, and recommended JDK 17 and JDK 21, including links to download. |
In the Getting Started section for each OS, the System Requirements specify Java 11 JDK:
However the Java section at the bottom of the same page recommends Java 17. This pull request seeks to make this less confusing by bringing the System Requirements in-line with the recommended Java JDK version.