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