[TIMOB-25429] CLI: Support detection of Java 9
GitHub Issue | n/a |
---|---|
Type | Improvement |
Priority | Critical |
Status | Closed |
Resolution | Fixed |
Resolution Date | 2018-04-02T23:40:29.000+0000 |
Affected Version/s | n/a |
Fix Version/s | Release 7.1.1 |
Components | CLI |
Labels | android, cli, defect |
Reporter | jzevin@nbme.org |
Assignee | Ewan Harris |
Created | 2017-10-21T17:36:10.000+0000 |
Updated | 2018-04-04T18:36:24.000+0000 |
Hello, Please run "appc ti setup check" or "appc setup check" in the terminal and send the log in here. Thanks.
[~jzevin@nbme.org] This isn't actually due to the java.home setting not working but due to some changes made in Java 9 around install locations. I'll move this ticket over to TIMOB and make it an improvement ticket. For now try using Java 1.8
PR for node-appc https://github.com/appcelerator/node-appc/pull/127
Not sure what version this will go out in, so I created PRs for master (7.2.0) and 7_1_X (7.1.1?). Master: https://github.com/appcelerator/titanium_mobile/pull/9924 7_1_X: https://github.com/appcelerator/titanium_mobile/pull/9925
[~cwilliams] Just to confuse things a little jdk detection goes through the titanium cli package. Made a PR to update the node-appc version there https://github.com/appcelerator/titanium/pull/272
Mentioning new PR's here: Master: https://github.com/appcelerator/titanium_mobile/pull/9964 7_1_X: https://github.com/appcelerator/titanium_mobile/pull/9965
Found issues while building android app with java 9.0.4. Commented in the master PR.
FR Passed. PR's merged.
Verified the fix in SDK 7.1.1.v20180403163228 & 7.2.0.v20180403153400, Closing. Studio Ver: 5.1.0 OS Ver: 10.13.2 Xcode Ver: Xcode 9.3 Appc NPM: 4.2.13-2 Appc CLI: 7.0.3-master.27 Daemon Ver: 1.0.1 Ti CLI Ver: 5.1.0 Alloy Ver: 1.11.0 Node Ver: 8.9.1 NPM Ver: 5.5.1 Java Ver: 9.0.4 Devices: ⇨ google Nexus 6P --- Android 8.0.0 ⇨ google Nexus 5 --- Android 6.0.1