tools: fix setting path containing an ampersand

This commit fixes an issue with the Node.js command prompt
on Windows where the PATH environment variable would not be
set correctly if the existing PATH contained an '&'.

Fixes: https://github.com/nodejs/node/issues/4802
PR-URL: https://github.com/nodejs/node/pull/4804
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

diff --git a/tools/msvs/nodevars.bat b/tools/msvs/nodevars.bat
index 22d2ec4..c94c446 100644 (file)
--- a/tools/msvs/nodevars.bat
+++ b/tools/msvs/nodevars.bat
@@ -1,7 +1,7 @@
 @echo off
 
 rem Ensure this Node.js and npm are first in the PATH
-set PATH=%APPDATA%\npm;%~dp0;%PATH%
+set "PATH=%APPDATA%\npm;%~dp0;%PATH%"
 
 setlocal enabledelayedexpansion
 pushd "%~dp0"