Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

README could mention a more compatible run command for Windows: 'npx love.js.cmd' #48

Closed
IoriBranford opened this issue Oct 5, 2021 · 0 comments
Labels
documentation Improvements or additions to documentation

Comments

@IoriBranford
Copy link

On Windows cmd and Powershell, I can run love.js only with npx love.js.cmd, not npx love.js which tries to run the Unix shell script instead of the cmd or Powershell ones:

basedir=$(dirname "$(echo "$0" | sed -e 's,\\,/,g')")
          ^^^^^^^

SyntaxError: missing ) after argument list
    at wrapSafe (internal/modules/cjs/loader.js:1001:16)
    at Module._compile (internal/modules/cjs/loader.js:1049:27)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1114:10)
    at Module.load (internal/modules/cjs/loader.js:950:32)
    at Function.Module._load (internal/modules/cjs/loader.js:790:12)
    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:76:12)
    at internal/main/run_main_module.js:17:47

Maybe this has something to do with my Node.js installation (installed LTS with winget, and associated js files with node.exe) but even so, it's worth mentioning in the readme.

@Davidobot Davidobot added the documentation Improvements or additions to documentation label Nov 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants