RFR: 1061: The skara update command is broken for platforms without downloaded JDK
Erik Joelsson
erikj at openjdk.java.net
Wed May 26 23:46:36 UTC 2021
This patch changes the CLI launcher scripts to no longer export JAVA_HOME. This is to prevent this exported variable from interfering with the "git skara update" command. The update command needs to be able to run Gradle. Gradle cannot run with the JAVA_HOME exported from the Skara CLI scripts. Whatever JAVA_HOME the user has needs to survive through to the Gradle invocation.
I'm not sure why JAVA_HOME was set here. It seems to have been around for a long time. I'm assuming it just happened to be written that way, but if anyone can think of a reason, that would be good. I'm running with this patch myself right now to try to catch any potential problems.
-------------
Commit messages:
- Stop setting JAVA_HOME in cli launchers
Changes: https://git.openjdk.java.net/skara/pull/1174/files
Webrev: https://webrevs.openjdk.java.net/?repo=skara&pr=1174&range=00
Issue: https://bugs.openjdk.java.net/browse/SKARA-1061
Stats: 4 lines in 1 file changed: 0 ins; 0 del; 4 mod
Patch: https://git.openjdk.java.net/skara/pull/1174.diff
Fetch: git fetch https://git.openjdk.java.net/skara pull/1174/head:pull/1174
PR: https://git.openjdk.java.net/skara/pull/1174
More information about the skara-dev
mailing list