-
Notifications
You must be signed in to change notification settings - Fork 554
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
Document: CORDOVA_JAVA_HOME environment variable #1292
Comments
PR #1293 is OK,
This is other issue however, as for sure I bet specific OS should be omitted. |
For sure... I didn't change what was already, there I just appended. However this is definitely true and I think this is a good opportunity to address that. Thank you for pointing that out. |
It is live at https://cordova.apache.org/docs/en/11.x/guide/platforms/android/index.html#java-development-kit-jdk And it feels like it should be moved to section https://cordova.apache.org/docs/en/11.x/guide/platforms/android/index.html#setting-environment-variables |
I would... also agree with that. A PR is welcome to move it. |
As pointed out by apache/cordova-android#1229 (comment)
The
CORDOVA_JAVA_HOME
is not documented as an option.CORDOVA_JAVA_HOME
will take precedence overJAVA_HOME
if set.The feature was added in v10 of cordova-android.
The text was updated successfully, but these errors were encountered: