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

invalid PATH detection #25

Open
max-mapper opened this issue Nov 23, 2013 · 3 comments
Open

invalid PATH detection #25

max-mapper opened this issue Nov 23, 2013 · 3 comments

Comments

@max-mapper
Copy link
Contributor

at nodeschool IRL events lately the #1 issue is definitely computers where global modules arent in the PATH.

i'm not a shell expert by any means but maybe we could do some sort of PATH analysis as a post-install hook or something that said very clearly:

You just installed learnyounode, use this command to run it:
learnyounode

or if their PATH is messed up:

You just installed learnyounode, use this command to run it:
/usr/local/lib/node_modules/learnyounode
@max-mapper
Copy link
Contributor Author

also twice i've seen this crazy bug where running learnyounode on windows results in a "JScript" error, which I think is a result of usr/bin/env not being interpreted and .js files on windows try to get evaluated with JScript and not node?

@rvagg
Copy link
Contributor

rvagg commented Nov 23, 2013

The "JScript" error was only happening on Windows right??

I guess we could also run some post-install stuff to check that an executable runs right; it's odd though because npm itself uses #!/usr/bin/env node and presumably they are using npm to install.

@max-mapper
Copy link
Contributor Author

yea JScript only exists on windows

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants