Skip to content
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

Support JDK_HOME and JAVA_HOME #3248

Closed
bluelovers opened this issue Dec 5, 2017 · 1 comment
Closed

Support JDK_HOME and JAVA_HOME #3248

bluelovers opened this issue Dec 5, 2017 · 1 comment
Assignees
Milestone

Comments

@bluelovers
Copy link

Please, provide the details below:

some ppl use JRE and JDK at same time

and some JRE didn't have javac

i think maybe should use process.env.JDK_HOME > process.env.JAVA_HOME

https://github.com/telerik/mobile-cli-lib/pull/1031/files

@rosen-vladimirov
Copy link
Contributor

Hey @bluelovers ,
Thanks for your efforts. I've marked this change as breaking, as it may cause unexpected behavior on some machines where JDK_HOME is already set. Current CLI version may work as expected, but the JDK_HOME may point to a JAVA version that is not supported, so after implementing this feature, Android builds may fail.

@rosen-vladimirov rosen-vladimirov added this to the 4.0.0 milestone Jan 11, 2018
@rosen-vladimirov rosen-vladimirov changed the title About JDK_HOME & JAVA_HOME Support JDK_HOME and JAVA_HOME Mar 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants