Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

build: add 'no npm' install target #2200

Closed
bnoordhuis opened this issue Nov 26, 2011 · 6 comments
Closed

build: add 'no npm' install target #2200

bnoordhuis opened this issue Nov 26, 2011 · 6 comments
Milestone

Comments

@bnoordhuis
Copy link
Member

http://groups.google.com/group/nodejs/msg/a87848b675ee07ed

@rmustacc
Copy link

Shouldn't installing the bundled npm only be the default on Windows similar to OpenSSL?

@rmg
Copy link

rmg commented Nov 27, 2011

It saves a step when setting up a new development environment on any type of system, which I found immediately useful.

Might make sense to make the default install target "smart" to only install npm if it isn't already installed and then have a mechanism for forcing the decision in either direction.

Another solution is to have a separate target that wraps 'curl http://npmjs.org/install.sh | sh' called install-npm and mention it in the build instructions and possibly at the end of 'make install'.

@isaacs
Copy link

isaacs commented Nov 28, 2011

I think that this should be a ./configure --without-npm option.

@rmustacc
Copy link

Since the previous default was not there, I guess I'd rather see it be --with-npm. Similar to how we don't default to installing the bundled OpenSSL or zlib libraries.

@isaacs
Copy link

isaacs commented Nov 28, 2011

@rmustacc Well, sure, but we want to have npm installed by default when you do ./configure && make && sudo make install.

@bnoordhuis
Copy link
Member Author

Fixed in 3ca0517.

alexkwolfe pushed a commit to alexkwolfe/node that referenced this issue Dec 23, 2011
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants