node-gyp: fix custom node-gyp env var quoting on windows

orangemocha · zkat · commit 24b13fbc57d3 · 2016-01-20T18:02:50.000-08:00
npm sets node_config_node_gyp without quotes, so its usage should be quoted. Indeed, it is a better practice to define environment variables that contain paths without quotes and quote their usage. PR-URL: #11158 Credit: @orangemocha
diff --git a/bin/node-gyp-bin/node-gyp.cmd b/bin/node-gyp-bin/node-gyp.cmd
@@ -1,5 +1,5 @@
 if not defined npm_config_node_gyp (
   node "%~dp0\..\..\node_modules\node-gyp\bin\node-gyp.js" %*
 ) else (
-  node %npm_config_node_gyp% %*
+  node "%npm_config_node_gyp%" %*
 )

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`if not defined npm_config_node_gyp (`
`2`	`2`	`node "%~dp0\..\..\node_modules\node-gyp\bin\node-gyp.js" %*`
`3`	`3`	`) else (`
`4`		`- node %npm_config_node_gyp% %*`
	`4`	`+ node "%npm_config_node_gyp%" %*`
`5`	`5`	`)`