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

npm install -g llnode fails #77

Closed
rnchamberlain opened this issue Apr 12, 2017 · 1 comment
Closed

npm install -g llnode fails #77

rnchamberlain opened this issue Apr 12, 2017 · 1 comment

Comments

@rnchamberlain
Copy link
Contributor

It's common practice to install tools using the-goption, but this seems to fail with llnode:

rchamberlain@ThinkCentre-M57p:~/test$ npm install -g llnode

> llnode@1.4.1 preinstall /home/rchamberlain/node-v7.7.3-linux-x64/lib/node_modules/.staging/llnode-4a7a78c7
> node scripts/configure.js

Build dir is: /home/rchamberlain/node-v7.7.3-linux-x64/lib/node_modules/.staging/llnode-4a7a78c7
Checking for headers, version is 3.9
Installing llnode for /usr/lib/llvm-3.9/bin/lldb, lldb version 3.9
Linking lldb to include directory /usr/lib/llvm-3.9
npm ERR! Linux 4.4.0-72-generic
npm ERR! argv "/home/rchamberlain/node-v7.7.3-linux-x64/bin/node" "/home/rchamberlain/node-v7.7.3-linux-x64/bin/npm" "explore" "node-gyp" "pwd"
npm ERR! node v7.7.3
npm ERR! npm  v4.1.2

npm ERR! It doesn't look like node-gyp is installed.
npm ERR! 
npm ERR! If you need help, you may report this error at:
npm ERR!     <https://github.com/npm/npm/issues>

npm ERR! Please include the following file with any support request:
npm ERR!     /home/rchamberlain/node-v7.7.3-linux-x64/lib/node_modules/npm/npm-debug.log
npm ERR! weird error 1
child_process.js:512
    throw err;
    ^
Error: Command failed: npm -g explore npm npm explore node-gyp pwd

Problem seems to be this line in the config.js script: https://github.com/nodejs/llnode/blob/master/scripts/configure.js#L89

Using npm install llnode without the -g option works fine.

@hhellyer
Copy link
Contributor

PR #94 has been landed and published to https://www.npmjs.com/package/llnode - Closing this now.

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

Successfully merging a pull request may close this issue.

2 participants