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

README: troubleshooting scenario and homebrew unsupported note #951

Closed
wants to merge 1 commit into from

Conversation

myconode
Copy link
Contributor

Add learnings from a recent issue

- [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 OSX, homebrew installation is not supported.
Copy link
Member

Choose a reason for hiding this comment

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

no need to specify OS X, just "homebrew installation is not supported" would be sufficient.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍 updated

@myconode myconode force-pushed the UpdateREADME branch 2 times, most recently from b8b99b2 to bf85896 Compare December 27, 2015 20:43
@@ -249,6 +251,8 @@ After the v0.8.6 release of node, nvm tries to install from binary packages. But

nvm install -s 0.8.6

If setting the `default` alias does not establish the node version in new shells (i.e. `nvm current` yields `system`), ensure that the system's node PATH is set before the `nvm.sh` source line in your shell profile (see [#658](/../../issues/658))
Copy link
Member

Choose a reason for hiding this comment

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

will /../../issues/658 link properly? it might be better to put the full URL, if github won't auto-link #658

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No problem, will update. btw took the relative-path pattern from another link in readme (606 in compatibility section), can update that one if you'd like as well.

Copy link
Member

Choose a reason for hiding this comment

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

sure, why not :-)

@ljharb
Copy link
Member

ljharb commented Dec 27, 2015

Merged in 1a6f85d

@ljharb ljharb closed this Dec 27, 2015
@myconode myconode deleted the UpdateREADME branch December 27, 2015 23:41
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 this pull request may close these issues.

None yet

2 participants