Michael Blow has uploaded a new change for review.

  https://asterix-gerrit.ics.uci.edu/1189

Change subject: Handle Unquoted %JAVA_HOME% Containing Spaces
......................................................................

Handle Unquoted %JAVA_HOME% Containing Spaces

Fixes sample cluster start / stop scripts on windows when the
%JAVA_HOME% variable contains spaces but is not surrounded in
%double-quotes

Change-Id: I10fafe29d76116698f97f0dd126cc5a14928586b
---
M asterixdb/asterix-server/src/main/samples/local/bin/start-sample-cluster.bat
M asterixdb/asterix-server/src/main/samples/local/bin/stop-sample-cluster.bat
2 files changed, 4 insertions(+), 2 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/89/1189/1

diff --git 
a/asterixdb/asterix-server/src/main/samples/local/bin/start-sample-cluster.bat 
b/asterixdb/asterix-server/src/main/samples/local/bin/start-sample-cluster.bat
index 4d10261..dea842a 100644
--- 
a/asterixdb/asterix-server/src/main/samples/local/bin/start-sample-cluster.bat
+++ 
b/asterixdb/asterix-server/src/main/samples/local/bin/start-sample-cluster.bat
@@ -24,7 +24,8 @@
   goto :ERROR
 )
 REM ensure JAVA_HOME has no spaces nor quotes, since appassembler can't handle 
them
-for %%I in (%JAVA_HOME%) do (
+set JAVA_HOME=%JAVA_HOME:"=%
+for %%I in (""%JAVA_HOME%") do (
   set JAVA_HOME=%%~sI
 )
 
diff --git 
a/asterixdb/asterix-server/src/main/samples/local/bin/stop-sample-cluster.bat 
b/asterixdb/asterix-server/src/main/samples/local/bin/stop-sample-cluster.bat
index dfd1ada..a0bcd2b 100644
--- 
a/asterixdb/asterix-server/src/main/samples/local/bin/stop-sample-cluster.bat
+++ 
b/asterixdb/asterix-server/src/main/samples/local/bin/stop-sample-cluster.bat
@@ -24,7 +24,8 @@
 )
 
 REM ensure JAVA_HOME has no spaces nor quotes, since appassembler can't handle 
them
-for %%I in (%JAVA_HOME%) do (
+set JAVA_HOME=%JAVA_HOME:"=%
+for %%I in (""%JAVA_HOME%") do (
   set JAVA_HOME=%%~sI
 )
 

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1189
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I10fafe29d76116698f97f0dd126cc5a14928586b
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow <mb...@apache.org>

Reply via email to