Skip to content

Commit

Permalink
.aliases: Remove use-iojs and use-node in favor of nvm
Browse files Browse the repository at this point in the history 
https://github.com/creationix/nvm
  • Loading branch information
@mathiasbynens
mathiasbynens committed Mar 12, 2015
1 parent ee673f4 commit 98d67fc
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 11 deletions.
4 changes: 0 additions & 4 deletions .aliases
View file
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,3 @@ alias afk="/System/Library/CoreServices/Menu\ Extras/User.menu/Contents/Resource

# Reload the shell (i.e. invoke as a login shell)
alias reload="exec $SHELL -l"

# Easily switch between `io.js` and `Node.js`
alias use-iojs="brew unlink node && brew link --force iojs"
alias use-node="brew unlink iojs && brew link --force node"
7 changes: 0 additions & 7 deletions brew.sh
View file
Original file line number Diff line number Diff line change
Expand Up @@ -95,12 +95,5 @@ brew install tree
brew install webkit2png
brew install zopfli

# Install Node.js. Note: this installs `npm` too, using the recommended
# installation method.
brew install node

# Install io.js
brew install iojs

# Remove outdated versions from the cellar.
brew cleanup

9 comments on commit 98d67fc

@mathiasbynens
Copy link
Owner Author

Choose a reason for hiding this comment

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

FYI @alrra

@s10wen
Copy link
Contributor

@s10wen s10wen commented on 98d67fc Aug 10, 2015

Choose a reason for hiding this comment

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

@mathiasbynens just double checking you meant to get rid of the original brew install node here as well?

@mathiasbynens
Copy link
Owner Author

Choose a reason for hiding this comment

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

Yes, use nvm install $NODE_OR_IOJS_VERSION instead.

@s10wen
Copy link
Contributor

@s10wen s10wen commented on 98d67fc Aug 10, 2015

Choose a reason for hiding this comment

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

Cool, I can't find nvm in your dotfiles, wondering how you handle this please?

@mathiasbynens
Copy link
Owner Author

Choose a reason for hiding this comment

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

https://github.com/creationix/nvm#install-script

@s10wen
Copy link
Contributor

@s10wen s10wen commented on 98d67fc Aug 18, 2015

Choose a reason for hiding this comment

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

Tried installing this but getting -bash: nvm: command not found wondering if you've added:
[[ -s $HOME/.nvm/nvm.sh ]] && . $HOME/.nvm/nvm.sh or something similar? or maybe my install just tripped up?

@AlecRust
Copy link

Choose a reason for hiding this comment

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

@s10wen have you restarted your terminal after successfully installing nvm using that install command?

This commit removes node/iojs from the dotfiles since @mathiasbynens and many others choose to handle this with nvm instead. Any issues installing or using nvm can go in the nvm issue tracker.

@s10wen
Copy link
Contributor

@s10wen s10wen commented on 98d67fc Aug 18, 2015

Choose a reason for hiding this comment

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

@AlecRust yes I restarted, adding [[ -s $HOME/.nvm/nvm.sh ]] && . $HOME/.nvm/nvm.sh to my .bash_profile seemed to fix it.

@AlecRust
Copy link

Choose a reason for hiding this comment

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

@s10wen ah yes that's part of the install. Just . ~/.nvm/nvm.sh should do it but glad you got it working 👍

Please sign in to comment.