-
-
Notifications
You must be signed in to change notification settings - Fork 246
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
installing nodejs 4 leads to npm being installed as well, which fails the nodejs install #165
Comments
Hmm. Can you provide a OS details? Also, does this occur on a fresh box, or just on one with native npm already installed? |
Happened on Ubuntu Trusty Tahr, and it happened while upgrading from node 0.10.x, which was also installed via this module. If I recall correctly, I traced the problem to some puppet code that installed npm regardless of the node version. In the 0.10.x package from NodeSource, both of them were seperate packages, but in the node 4 repo they are one package that provides and also conficts with npm. Thus an I manage the repo and installation myself now, so I can't provide any more information or testing. |
I'm also running into this error |
@Jmeyering what OS/versions? |
Ubuntu 14.04 node repo version 4.x or 5.x |
Having the same issue, can we get some update this is seriously broken and entirely useless? I'm getting this on Ubuntu 14.04/ Trusty Tahr. A potential work around I have tried is just using repo_url_suffix and making sure it doesn't try and install NPM additionally. |
@Jmeyering @realmreith Is this still occurring with the latest release? |
So setting https://github.com/voxpupuli/puppet-nodejs/blob/master/manifests/install.pp#L55-L60 When its node version >=4 npm shouldn't be installed separately. |
Still suffering from this over 12 months later? Is there a known fix or proper workaround? |
Bumped into the same issue, on Ubuntu 16.04, node 6.x and module version 2.1.0. As mentioned in the previous comments and @chrislea's comment here, the nodejs package from NodeSource includes
|
Closing, as I believe most of the issues discussed here have already been fixed. |
Closing, as I believe most of the issues discussed here have already been fixed. |
leads to:
This is on ubuntu trusty tahr.
The text was updated successfully, but these errors were encountered: