You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by en...@apache.org on 2013/04/29 23:27:47 UTC

svn commit: r1477353 - in /hbase/branches/0.95/bin: hbase-config.cmd hbase.cmd

Author: enis
Date: Mon Apr 29 21:27:47 2013
New Revision: 1477353

URL: http://svn.apache.org/r1477353
Log:
HBASE-8456 HBase Windows scripts fail when there's a blank space in JAVA_HOME (Mostafa Elhemali)

Modified:
    hbase/branches/0.95/bin/hbase-config.cmd
    hbase/branches/0.95/bin/hbase.cmd

Modified: hbase/branches/0.95/bin/hbase-config.cmd
URL: http://svn.apache.org/viewvc/hbase/branches/0.95/bin/hbase-config.cmd?rev=1477353&r1=1477352&r2=1477353&view=diff
==============================================================================
--- hbase/branches/0.95/bin/hbase-config.cmd (original)
+++ hbase/branches/0.95/bin/hbase-config.cmd Mon Apr 29 21:27:47 2013
@@ -29,12 +29,12 @@ if not defined JAVA_HOME (
   set JAVA_HOME=c:\apps\java
 )
 
-if not exist %JAVA_HOME%\bin\java.exe (
+if not exist "%JAVA_HOME%\bin\java.exe" (
   echo Error: JAVA_HOME is incorrectly set or could not find java at the location %JAVA_HOME%\bin\
   exit /B 2
 )
 
-set JAVA=%JAVA_HOME%\bin\java
+set JAVA="%JAVA_HOME%\bin\java"
 
 for %%i in (%0) do (
   if not defined HBASE_BIN_PATH (

Modified: hbase/branches/0.95/bin/hbase.cmd
URL: http://svn.apache.org/viewvc/hbase/branches/0.95/bin/hbase.cmd?rev=1477353&r1=1477352&r2=1477353&view=diff
==============================================================================
--- hbase/branches/0.95/bin/hbase.cmd (original)
+++ hbase/branches/0.95/bin/hbase.cmd Mon Apr 29 21:27:47 2013
@@ -82,7 +82,7 @@ if not "%HBASE_HEAPSIZE%" == "" (
   set JAVA_HEAP_MAX=-Xmx%HBASE_HEAPSIZE%m
 )
 
-set CLASSPATH=%HBASE_CONF_DIR%;%JAVA_HOME%\lib\tools.jar
+set CLASSPATH=%HBASE_CONF_DIR%;"%JAVA_HOME%\lib\tools.jar"
 
 rem Add maven target directory
 set cached_classpath_filename=%HBASE_HOME%\target\cached_classpath.txt