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

Update README.markdown #1101

Merged
merged 1 commit into from
May 25, 2016
Merged

Update README.markdown #1101

merged 1 commit into from
May 25, 2016

Conversation

kelexel
Copy link
Contributor

@kelexel kelexel commented May 25, 2016

Added notes for OS X users regarding globally installed modules.

@@ -23,6 +23,11 @@ Note: We still have some problems with FreeBSD, because there is no pre-built bi
Note: On OS X, if you do not have Xcode installed and you do not wish to download the ~4.3GB file, you can install the `Command Line Tools`. You can check out this blog post on how to just that:
- [How to Install Command Line Tools in OS X Mavericks & Yosemite (Without Xcode)](http://osxdaily.com/2014/02/12/install-command-line-tools-mac-os-x/)

Note: On OS X, if you have/had a "regular" nodejs installed and want to install modules globally, keep in mind that:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's change this phrase to "system" node - nodejs is ubuntu's legacy way to refer to the apt package, and "regular" has connotations i don't like :-) (same below)

@ljharb
Copy link
Member

ljharb commented May 25, 2016

Thanks!

@ljharb ljharb added informational needs followup We need some info or action from whoever filed this issue/PR. labels May 25, 2016
Added notes for OS X users regarding globally installed modules.
@ljharb ljharb removed the needs followup We need some info or action from whoever filed this issue/PR. label May 25, 2016
@ljharb ljharb merged commit b227f4d into nvm-sh:master May 25, 2016
@ljharb ljharb modified the milestone: v0.31.2 Jun 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants