Skip to content

Commit

Permalink
remove win JAVA_HOME lookup because it has problems with spaces in th…
Browse files Browse the repository at this point in the history
…e PATH

It'd be nice to replace this with something that works better. I asked someone to try

    for /f "delims=;" %%P IN (%PATH%) do if exist %%P\java.exe set JAVA_HOME=%%P..\

but it didn't work. Now, it'll print an error message about setting JAVA_HOME, which will work. For reference, the conversation was http://donttreadonme.co.uk/mirah/2012/07/16.html#message_16
  • Loading branch information
baroquebobcat committed Jul 22, 2012
1 parent cb10541 commit 827434c
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions distbin/_mirahvars.bat
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,6 @@ rem ----- Verify and Set Required Environment Variables -----------------------

if not "%JAVA_HOME%" == "" goto gotJava

for %%P in (%PATH%) do if exist %%P\java.exe set JAVA_HOME=%%P..\
if not "%JAVA_HOME%" == "" goto gotJava

echo You must set JAVA_HOME to point at your JRE/JDK installation
set MIRAH_BAT_ERROR=1
exit /b 1
Expand Down

0 comments on commit 827434c

Please sign in to comment.