Build Test failed on Windows 7 #16023

Activity

1. mscdex

   added

   testIssues and PRs related to Node.js core tests and test infrastructure.Issues and PRs related to Node.js core tests and test infrastructure.

   windowsIssues and PRs related to the Windows platform.Issues and PRs related to the Windows platform.

   on Oct 6, 2017
2. vsemozhetbyt commented on Oct 6, 2017

   

   vsemozhetbyt

   on Oct 6, 2017

   Last edited by vsemozhetbyt

   Contributor

   More actions

   Reduced case:

   const trickyString = 'c:\\xfolder\\file.js';
   
   require("child_process").execSync(`"${process.execPath}" -e "'${trickyString}'"`);

   [eval]:1
   'c:\xfolder\file.js'
      ^^^^
   
   SyntaxError: Invalid hexadecimal escape sequence
       at createScript (vm.js:80:10)
       at Object.runInThisContext (vm.js:139:10)
       at Object.<anonymous> ([eval]-wrapper:6:22)
       at Module._compile (module.js:624:30)
       at evalScript (bootstrap_node.js:461:27)
       at startup (bootstrap_node.js:163:9)
       at bootstrap_node.js:607:3
   child_process.js:635
       throw err;
       ^
   
   Error: Command failed: "C:\Program Files\nodejs\node.exe" -e "'c:\xfolder\file.js'"
   [eval]:1
   'c:\xfolder\file.js'
      ^^^^
   
   SyntaxError: Invalid hexadecimal escape sequence
       at createScript (vm.js:80:10)
       at Object.runInThisContext (vm.js:139:10)
       at Object.<anonymous> ([eval]-wrapper:6:22)
       at Module._compile (module.js:624:30)
       at evalScript (bootstrap_node.js:461:27)
       at startup (bootstrap_node.js:163:9)
       at bootstrap_node.js:607:3
   
       at checkExecSyncError (child_process.js:592:13)
       at Object.execSync (child_process.js:632:13)
       at Object.<anonymous> (...\test.js:5:26)
       at Module._compile (module.js:624:30)
       at Object.Module._extensions..js (module.js:635:10)
       at Module.load (module.js:545:32)
       at tryModuleLoad (module.js:508:12)
       at Function.Module._load (module.js:500:3)
       at Function.Module.runMain (module.js:665:10)
       at startup (bootstrap_node.js:187:16)
3. bzoz

   added a commit that references this issue on Oct 10, 2017

   win, test: escape script filename on Windows

   ...

   8f87a8a
4. jasnell

   added a commit that references this issue on Oct 13, 2017

   test: escape script filename on Windows

   ...

   07ec10d
5. bzoz commented on Nov 9, 2017

   

   bzoz

   on Nov 9, 2017

   Contributor

   More actions

   This was fixed in 07ec10d
6. bzoz

   closed this as completedon Nov 9, 2017
7. MylesBorins

   added 2 commits that reference this issue on Nov 21, 2017

   test: escape script filename on Windows

   ...

   548b74c

   test: escape script filename on Windows

   ...

   04f3f6d