Restore quotes around module detection JAVA_HOME#6638
Restore quotes around module detection JAVA_HOME#6638headius merged 1 commit intojruby:jruby-9.2from
Conversation
|
According to @walro in #6615 (comment) the affected path with spaces is where Oracle JDK 8 installed on MacOS (/Library/Internet Something). This differs from where my system installs, in /LIbrary/Java/JavaVirtualMachines and may indicate a new layout for JDKs in newer MacOS releases. I have asked for more information. |
|
Thanks! I noticed this since I, for reasons I do not remember now, had my
|
I think it has been like this "forever", I might just be odd favoring a JRE installation (I generally don't do any Java development) ;-) |
|
@walro Aha that explains it! We should run fine with a JRE so that is no problem, but I did not realize that the Oracle JRE gets installed in a path with spaces (and referencing the long-deprecated Applet feature, no less!) I will look into adding a test for this situation, at least when on MacOS and such a path is available. |
See #6615 (review)
The quoting around
$JAVA_HOMEwas inadvertently removed while fixing module detection in #6615. The review linked above points out that this causes problems when the JDK is installed in a path with spaces. This is not common on unixy environments, and the bash script is not used on Windows where it is more common, but the regression is real.This PR restores the quoting around
$JAVA_HOMEto fix the issue.