Error when convert: Command substitution
See original GitHub issuenode lib/cli.js uasp.sh
(script can be found here: https://github.com/kennethrisa/uasp)
return: D:\bash-shell-to-bat-converter\node_modules\bash-parser\src\index.js:52 throw new Error(err.stack || err.message); ^
Error: Parse error on line 34: Unexpected ‘OPEN_PAREN’ at Object.parse [as default] (D:\bash-shell-to-bat-converter\node_modul es\bash-parser\src\index.js:52:9) at Object.convertBashToWin (D:\bash-shell-to-bat-converter\lib\convert- bash.js:69:36) at Object.<anonymous> (D:\bash-shell-to-bat-converter\lib\cli.js:9:26) at Module._compile (internal/modules/cjs/loader.js:689:30) at Object.Module._extensions…js (internal/modules/cjs/loader.js:700:10) at Module.load (internal/modules/cjs/loader.js:599:32) at tryModuleLoad (internal/modules/cjs/loader.js:538:12) at Function.Module._load (internal/modules/cjs/loader.js:530:3) at Function.Module.runMain (internal/modules/cjs/loader.js:742:12) at startup (internal/bootstrap/node.js:283:19)
Windows 10 node 10.15.1
Issue Analytics
- State:
- Created 3 years ago
- Comments:9 (3 by maintainers)
Top GitHub Comments
trying your patch: node lib/cli.js uasp.sh F:\rust\bash-shell-to-bat-converter\lib\convert-bash.js:7 window = { process: { env: { NODE_NEV: ‘mock’ } } }; ^
ReferenceError: window is not defined at Object.<anonymous> (F:\rust\bash-shell-to-bat-converter\lib\convert-bash.js:7:12) at Module._compile (internal/modules/cjs/loader.js:689:30) at Object.Module._extensions…js (internal/modules/cjs/loader.js:700:10) at Module.load (internal/modules/cjs/loader.js:599:32) at tryModuleLoad (internal/modules/cjs/loader.js:538:12) at Function.Module._load (internal/modules/cjs/loader.js:530:3) at Module.require (internal/modules/cjs/loader.js:637:17) at require (internal/modules/cjs/helpers.js:22:18) at Object.<anonymous> (F:\rust\bash-shell-to-bat-converter\lib\cli.js:5:22) at Module._compile (internal/modules/cjs/loader.js:689:30)
closing this as a duplicate of #51 (I know, this predates the other issue, but here we have comments regarding another issue as well, so the other is cleaner…)