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

Provide option to install node using Brew #236

Closed
amosshapira opened this issue Jul 1, 2016 · 3 comments
Closed

Provide option to install node using Brew #236

amosshapira opened this issue Jul 1, 2016 · 3 comments
Labels
enhancement New feature or request

Comments

@amosshapira
Copy link

When using the module to install Node on OSX, I get the following errors:

Error: /Package[nodejs]: Provider macports is not functional on this host
Error: /Package[nodejs-devel]: Provider macports is not functional on this host
Error: /Package[npm]: Provider macports is not functional on this host

I use Homebrew to install node on the system. I don't use Macports anywhere on our network.
I need a way to tell the module to use Brew instead of Macports.

@amosshapira amosshapira changed the title Darwin required macports, but I install node using Brew Darwin requires macports, but I install node using Brew Jul 1, 2016
@juniorsysadmin juniorsysadmin added the enhancement New feature or request label Jul 2, 2016
@juniorsysadmin juniorsysadmin changed the title Darwin requires macports, but I install node using Brew Provide option to install node using Brew Jul 2, 2016
@kizzie
Copy link
Contributor

kizzie commented Sep 20, 2017

So this seems to be annoyingly easy and really hard at the same time. To get it to use homebrew you can edit the params.pp file and change line 171 to read

Package { provider => 'homebrew' }

Problem is I can't think of an elegent way to let people set that value (so as to not change current functionality) as the params class doesn't take inputs and moving the entire case statement out of the params.pp seems a little extreme for a simple fix pull request.

@kizzie
Copy link
Contributor

kizzie commented Sep 20, 2017

Have a vague idea at a PR - #321

Cause either I need to do one to update the macports package name (its now nodejs6|7|8 or allow for homebrew...)

@juniorsysadmin
Copy link
Member

Fixed by #321

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

No branches or pull requests

3 participants