Skip to content

Commit

Permalink
tools: fix setting path containing an ampersand
Browse files Browse the repository at this point in the history
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: nodejs#4802
PR-URL: nodejs#4804
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
mscdex authored and Michael Scovetta committed Apr 2, 2016
1 parent f9c09dc commit 40646cf
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tools/msvs/nodevars.bat
Original file line number Diff line number Diff line change
@@ -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"
Expand Down

0 comments on commit 40646cf

Please sign in to comment.