Skip to content

Commit

Permalink
use extended set syntax to fix Win64 bugs
Browse files Browse the repository at this point in the history
On 64-bit Windows, JES gets installed to "Program Files (x86)".
The batch interpreter can mess up when you use a variable containing
parentheses in a parenthesized block.

I was testing on a 32-bit machine, so it didn't have the parens,
and I didn't notice this before. Now that I have a 64-bit machine to
test on, I have fixed this.
  • Loading branch information
leafstorm committed Aug 5, 2014
1 parent 62a7201 commit 94f7c5b
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions jes.bat
Original file line number Diff line number Diff line change
Expand Up @@ -25,18 +25,18 @@ rem What Java should we use?

set java_exe=java.exe

set java_bundled=%jes_base%\dependencies\jre-win32
set "java_bundled=%jes_base%\dependencies\jre-win32"

if DEFINED JES_JAVA_HOME (
set java=%JES_JAVA_HOME%\bin\%java_exe%
set "java=%JES_JAVA_HOME%\bin\%java_exe%"
) else (
if EXIST "%java_bundled%" (
set java=%java_bundled%\bin\%java_exe%
set "java=%java_bundled%\bin\%java_exe%"
) else (
if DEFINED JAVA_HOME (
set java=%JAVA_HOME%\bin\%java_exe%
set "java=%JAVA_HOME%\bin\%java_exe%"
) else (
set java=%java_exe%
set "java=%java_exe%"
)
)
)
Expand Down

0 comments on commit 94f7c5b

Please sign in to comment.