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

Try to use same NPM/Node for sub-processes #24

Merged
merged 1 commit into from Jan 27, 2016

Conversation

Projects
None yet
2 participants
@justmoon
Copy link
Contributor

justmoon commented Jan 27, 2016

If the user runs the demo with a different node or NPM than what's in their path, they'll expect the sub-processes to run with that version as well.

We can do a slightly better job at making sure of that using the npm_execpath and npm_node_execpath environment variables.

@sentientwaffle

This comment has been minimized.

Copy link
Contributor

sentientwaffle commented Jan 27, 2016

LGTM

justmoon added a commit that referenced this pull request Jan 27, 2016

Merge pull request #24 from interledger/fix/pin_npm
Try to use same NPM/Node for sub-processes

@justmoon justmoon merged commit 7633bf6 into master Jan 27, 2016

@justmoon justmoon deleted the fix/pin_npm branch Jan 27, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.