WINDOWS REGRESSION: command with double-quotes cannot be executed

[warappa]

WINDOWS REGRESSION: command with double-quotes cannot be executed

[gwicksted]

Hi @warappa on my Windows 10 machine, I tested the following with both cmd.exe and powershell:

exec: {
    command: '"C:\\windows\\notepad.exe" "C:\\test.json"'
}

Which caused notepad to ask if I wanted to create the file C:\test.json (it does not exist on my disk). This is the expected behavior.

Can you perhaps provide some additional information? eg. Windows version, shell (and version), grunt version, node version? The full path would also be helpful. Perhaps yours has spaces? (my test did not)

To do this, run npm list, npm -v, node -v,

If you are running in powershell, please execute the following:

$PSVersionTable.PSVersion
[System.Environment]::OSVersion.Version
[System.Environment]::Is64BitProcess
[Environment]::Is64BitOperatingSystem
(Get-WmiObject win32_processor | Where-Object{$_.deviceID -eq "CPU0"}).AddressWidth
(Get-WmiObject -Class Win32_ComputerSystem).SystemType
npm -v
node -v
npm list
$pwd
Get-Content gruntfile.js | Where { $_.Contains('command: ') } | Out-String

[warappa]

Windows PowerShell
Copyright (C) 2016 Microsoft Corporation. Alle Rechte vorbehalten.

PS C:\full-path> $PSVersionTable.PSVersion

Major  Minor  Build  Revision
-----  -----  -----  --------
5      1      14393  1066


PS C:\full-path> [System.Environment]::OSVersion.Version

Major  Minor  Build  Revision
-----  -----  -----  --------
10     0      14393  0


PS C:\full-path> [System.Environment]::Is64BitProcess
False
PS C:\full-path> [Environment]::Is64BitOperatingSystem
True
PS C:\full-path> (Get-WmiObject win32_processor | Where-Object{$_.deviceID -eq "CPU0"}).AddressWidth
64
PS C:\full-path> (Get-WmiObject -Class Win32_ComputerSystem).SystemType
x64-based PC
PS C:\full-path> npm -v
3.10.3
PS C:\full-path> node -v
v6.3.0
PS C:\full-path> npm list
[email protected] C:\full-path
+-- [email protected]
| +-- [email protected]
| +-- [email protected]
| +-- [email protected]
| +-- [email protected]
| +-- [email protected]
| +-- [email protected]
| +-- [email protected]
| | +-- [email protected]
| | | `-- [email protected]
| | `-- [email protected]
| +-- [email protected]
| +-- [email protected]
| | +-- [email protected]
| | `-- [email protected]
| +-- [email protected]
| | +-- [email protected]
| | | +-- [email protected]
| | | `-- [email protected]
| | +-- [email protected]
| | `-- [email protected]
| +-- [email protected]
| +-- [email protected]
| +-- [email protected]
| +-- [email protected]
| | +-- [email protected]
| | | +-- [email protected]
| | | `-- [email protected]
| | `-- [email protected]
| +-- [email protected]
| +-- [email protected]
| | +-- [email protected]
| | `-- [email protected]
| +-- [email protected]
| | `-- [email protected]
| +-- [email protected]
| +-- [email protected]
| `-- [email protected]
+-- [email protected]
| `-- [email protected]
|   +-- [email protected]
|   | +-- [email protected]
|   | +-- [email protected]
|   | +-- [email protected]
|   | +-- [email protected]
|   | +-- [email protected]
|   | +-- [email protected]
|   | +-- [email protected]
|   | +-- [email protected]
|   | +-- [email protected]
|   | +-- [email protected]
|   | +-- [email protected]
|   | +-- [email protected]
|   | +-- [email protected]
|   | +-- [email protected]
|   | +-- [email protected]
|   | +-- [email protected]
|   | +-- [email protected]
|   | `-- [email protected]
|   +-- [email protected]
|   | +-- [email protected]
|   | `-- [email protected]
|   +-- [email protected]
|   | `-- [email protected]
|   +-- [email protected]
|   +-- [email protected]
|   | `-- [email protected]
|   `-- [email protected]
+-- [email protected]
+-- [email protected]
| +-- [email protected]
| | +-- [email protected]
| | +-- [email protected]
| | +-- [email protected]
| | | `-- [email protected]
| | +-- [email protected]
| | `-- [email protected]
| `-- [email protected]
|   `-- [email protected]
+-- [email protected]
| +-- [email protected]
| | +-- [email protected]
| | +-- [email protected]
| | | `-- [email protected]
| | +-- [email protected]
| | `-- [email protected]
| +-- [email protected]
| +-- [email protected]
| | +-- [email protected]
| | | +-- [email protected]
| | | +-- [email protected]
| | | | `-- [email protected]
| | | +-- [email protected]
| | | `-- [email protected]
| | +-- [email protected]
| | +-- [email protected]
| | | +-- [email protected]
| | | | `-- [email protected]
| | | `-- [email protected]
| | |   `-- [email protected]
| | `-- [email protected]
| +-- [email protected]
| | +-- [email protected]
| | +-- [email protected]
| | +-- [email protected]
| | `-- [email protected]
| |   +-- [email protected]
| |   +-- [email protected]
| |   | +-- [email protected]
| |   | | +-- [email protected]
| |   | | | +-- [email protected]
| |   | | | `-- [email protected]
| |   | | `-- [email protected]
| |   | +-- [email protected]
| |   | `-- [email protected]
| |   +-- [email protected]
| |   `-- [email protected]
| `-- [email protected]
+-- [email protected]
| +-- [email protected]
| +-- [email protected]
| | `-- [email protected]
| |   `-- [email protected]
| +-- [email protected]
| `-- [email protected]
|   +-- [email protected]
|   +-- [email protected]
|   +-- [email protected]
|   | `-- [email protected]
|   `-- [email protected]
+-- [email protected]
| `-- [email protected]
|   +-- [email protected]
|   | `-- [email protected]
|   |   +-- [email protected]
|   |   +-- [email protected]
|   |   +-- [email protected]
|   |   `-- [email protected]
|   +-- [email protected]
|   +-- [email protected]
|   | +-- [email protected]
|   | +-- [email protected]
|   | +-- [email protected]
|   | `-- [email protected]
|   |   +-- [email protected]
|   |   +-- [email protected]
|   |   `-- [email protected]
|   `-- [email protected]
+-- [email protected]
+-- [email protected]
| +-- [email protected]
| +-- [email protected]
| +-- [email protected]
| `-- [email protected]
+-- [email protected]
| +-- [email protected]
| | +-- [email protected]
| | `-- [email protected]
| +-- [email protected]
| | +-- [email protected]
| | +-- [email protected]
| | | +-- [email protected]
| | | +-- [email protected]
| | | | `-- [email protected]
| | | +-- [email protected]
| | | `-- [email protected]
| | +-- [email protected]
| | | +-- [email protected]
| | | | +-- [email protected]
| | | | `-- [email protected]
| | | `-- [email protected]
| | |   `-- [email protected]
| | +-- [email protected]
| | | `-- [email protected]
| | |   `-- [email protected]
| | +-- [email protected]
| | +-- [email protected]
| | | +-- [email protected]
| | | +-- [email protected]
| | | | `-- [email protected]
| | | +-- [email protected]
| | | `-- [email protected]
| | +-- [email protected]
| | +-- [email protected]
| | +-- [email protected]
| | +-- [email protected]
| | +-- [email protected]
| | | +-- [email protected]
| | | | `-- [email protected]
| | | +-- [email protected]
| | | | +-- [email protected]
| | | | | `-- [email protected]
| | | | `-- [email protected]
| | | +-- [email protected]
| | | +-- [email protected]
| | | +-- [email protected]
| | | | +-- [email protected]
| | | | +-- [email protected]
| | | | | `-- [email protected]
| | | | `-- [email protected]
| | | |   +-- [email protected]
| | | |   | `-- [email protected]
| | | |   `-- [email protected]
| | | +-- [email protected]
| | | | +-- [email protected]
| | | | | +-- [email protected]
| | | | | `-- [email protected]
| | | | |   `-- [email protected]
| | | | `-- [email protected]
| | | |   +-- [email protected]
| | | |   | +-- [email protected]
| | | |   | +-- [email protected]
| | | |   | | `-- [email protected]
| | | |   | |   `-- [email protected]
| | | |   | +-- [email protected]
| | | |   | `-- [email protected]
| | | |   |   `-- [email protected]
| | | |   `-- [email protected]
| | | |     `-- [email protected]
| | | +-- [email protected]
| | | | +-- [email protected]
| | | | | `-- [email protected]
| | | | |   `-- [email protected]
| | | | `-- [email protected]
| | | `-- [email protected]
| | +-- [email protected]
| | | `-- [email protected]
| | +-- [email protected]
| | +-- [email protected]
| | | +-- [email protected]
| | | | `-- [email protected]
| | | +-- [email protected]
| | | +-- [email protected]
| | | +-- [email protected]
| | | | `-- [email protected]
| | | |   +-- [email protected]
| | | |   `-- [email protected]
| | | +-- [email protected]
| | | +-- [email protected]
| | | | +-- [email protected]
| | | | `-- [email protected]
| | | +-- [email protected]
| | | `-- [email protected]
| | |   `-- [email protected]
| | +-- [email protected]
| | | +-- [email protected]
| | | | `-- [email protected]
| | | +-- [email protected]
| | | +-- [email protected]
| | | | +-- [email protected]
| | | | +-- [email protected]
| | | | +-- [email protected]
| | | | | +-- [email protected]
| | | | | +-- [email protected]
| | | | | | `-- [email protected]
| | | | | `-- [email protected]
| | | | |   `-- [email protected]
| | | | +-- [email protected]
| | | | | `-- [email protected]
| | | | `-- [email protected]
| | | `-- [email protected]
| | +-- [email protected]
| | | +-- [email protected]
| | | +-- [email protected]
| | | +-- [email protected]
| | | +-- [email protected]
| | | | `-- [email protected]
| | | +-- [email protected]
| | | +-- [email protected]
| | | +-- [email protected]
| | | | `-- [email protected]
| | | +-- [email protected]
| | | | +-- [email protected]
| | | | | +-- [email protected]
| | | | | `-- [email protected]
| | | | |   `-- [email protected]
| | | | `-- [email protected]
| | | +-- [email protected]
| | | | +-- [email protected]
| | | | +-- [email protected]
| | | | +-- [email protected]
| | | | `-- [email protected]
| | | +-- [email protected]
| | | | +-- [email protected]
| | | | +-- [email protected]
| | | | | +-- [email protected]
| | | | | +-- [email protected]
| | | | | +-- [email protected]
| | | | | `-- [email protected]
| | | | `-- [email protected]
| | | |   +-- [email protected]
| | | |   +-- [email protected]
| | | |   +-- [email protected]
| | | |   +-- [email protected]
| | | |   | `-- [email protected]
| | | |   +-- [email protected]
| | | |   +-- [email protected]
| | | |   | `-- [email protected]
| | | |   +-- [email protected]
| | | |   +-- [email protected]
| | | |   `-- [email protected]
| | | +-- [email protected]
| | | +-- [email protected]
| | | +-- [email protected]
| | | +-- [email protected]
| | | | `-- [email protected]
| | | +-- [email protected]
| | | +-- [email protected]
| | | +-- [email protected]
| | | +-- [email protected]
| | | +-- [email protected]
| | | +-- [email protected]
| | | | `-- [email protected]
| | | +-- [email protected]
| | | `-- [email protected]
| | +-- [email protected]
| | | +-- [email protected]
| | | | `-- [email protected]
| | | +-- [email protected]
| | | +-- [email protected]
| | | | `-- [email protected]
| | | `-- [email protected]
| | |   +-- [email protected]
| | |   +-- [email protected]
| | |   | +-- [email protected]
| | |   | | `-- [email protected]
| | |   | `-- [email protected]
| | |   |   `-- [email protected]
| | |   |     `-- [email protected]
| | |   +-- [email protected]
| | |   +-- [email protected]
| | |   | `-- [email protected]
| | |   |   `-- [email protected]
| | |   +-- [email protected]
| | |   +-- [email protected]
| | |   +-- [email protected]
| | |   +-- [email protected]
| | |   `-- [email protected]
| | |     `-- [email protected]
| | `-- [email protected]
| |   `-- [email protected]
| |     +-- [email protected]
| |     +-- [email protected]
| |     +-- [email protected]
| |     +-- [email protected]
| |     `-- [email protected]
| `-- [email protected]
+-- [email protected]
+-- [email protected]
| +-- [email protected]
| +-- [email protected]
| | +-- [email protected]
| | | +-- [email protected]
| | | `-- [email protected]
| | |   +-- [email protected]
| | |   | `-- [email protected]
| | |   +-- [email protected]
| | |   +-- [email protected]
| | |   | +-- [email protected]
| | |   | | `-- [email protected]
| | |   | |   +-- [email protected]
| | |   | |   +-- [email protected]
| | |   | |   `-- [email protected]
| | |   | +-- [email protected]
| | |   | `-- [email protected]
| | |   +-- [email protected]
| | |   | `-- [email protected]
| | |   +-- [email protected]
| | |   +-- [email protected]
| | |   +-- [email protected]
| | |   | `-- [email protected]
| | |   +-- [email protected]
| | |   | `-- [email protected]
| | |   +-- [email protected]
| | |   | +-- [email protected]
| | |   | | `-- [email protected]
| | |   | `-- [email protected]
| | |   +-- [email protected]
| | |   | +-- [email protected]
| | |   | `-- [email protected]
| | |   `-- [email protected]
| | |     +-- [email protected]
| | |     `-- [email protected]
| | +-- [email protected]
| | +-- UNMET OPTIONAL DEPENDENCY fsevents@^1.0.0
| | +-- [email protected]
| | +-- [email protected]
| | +-- [email protected]
| | | `-- [email protected]
| | +-- [email protected]
| | | `-- [email protected]
| | +-- [email protected]
| | `-- [email protected]
| |   +-- [email protected]
| |   `-- [email protected]
| `-- [email protected]
`-- UNMET PEER DEPENDENCY tslint@>=2.5.0-beta

npm ERR! peer dep missing: tslint@>=2.5.0-beta, required by [email protected]
PS C:\full-path> $pwd

Path
----
C:\full-path


PS C:\full-path> Get-Content gruntfile.js | Where { $_.Contains('command: ')
} | Out-String
                    //command: '"' + sourceDirectory + '\\packages\\tool\\tools\\chutzpah.console.exe" /path ":C\\full-path\\Tests\\chutzpah.json"'
                    command: '"C:\\windows\\notepad.exe" "C:\\test.json"'

[warappa]

BTW: I'm running the grunt-file with Task Runner Explorer in Visual Studio 2015 Update 3 (14.0.25431.01)

[gwicksted]

Does the notepad example work for you?

I see a potential typo in the commented out command here (colon before C):
":C\\full-path\\Tests\\chutzpah.json"

Other than that, I'm running PS 64 bit (vs 32) with npm 4.5.0, node 7.3.0, grunt-cli 1.2.0 and grunt 1.0.1

The biggest difference is probably that older grunt version. I don't think anything changed in node 6 vs 7 related to spawn. I'll have to do some testing with different versions.

P.S. You may want to update grunt beyond 1.0.0 (if possible). I know it can be a big hassle upgrading packages with npm -- lately I've been using yarn which has been great (on par or better than npm-check-updates for upgrading and way better for build vs dev server consistency). Just run:

npm install yarn -g
yarn
yarn clean
yarn upgrade

The yarn clean step is optional but it should save space/time and yarn upgrade will bump versions of your packages where possible.

From then on instead of npm install (which will still work) on your build server, just run yarn. Unlike npm, yarn is deterministic and always deploys the same node_modules structure/version where npm deploys different structure/versions depending on the order packages were installed.

[gwicksted]

Confirmed notepad test works with [email protected] and [email protected] (still using node 7)

[warappa]

The typo was only introduced while blacking the data ;)

No spaces in the path.

---

I just found the difference: Task Runner Explorer uses node in $(VSINSTALLDIR)\Web\External which is v5.4.1.
With node v6.3.0, which is on my PATH, it works.

@gwicksted, thank you for your help!