search

fgrehm / ventriloquist

Development environments made easy
MIT License
359 stars 20 forks source link

Added NodeJS support with NVM #22

Closed ndrluis closed 11 years ago

ndrluis commented 11 years ago

Issue #21

fgrehm commented 11 years ago

This looks awesome! I just need to do some testing and I'll get back to you :) Thanks a lot!

fgrehm commented 11 years ago

I'm tried this out by hand on a clean machine before doing it from the plugin and it seems we'll need to have git installed in order to use nvm:

vagrant@raring-base:~$ curl https://raw.github.com/creationix/nvm/master/install.sh | sh
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  1559  100  1559    0     0   2049      0 --:--:-- --:--:-- --:--:--  2186
You need to install git - visit http://git-scm.com/downloads
or, use install-gitless.sh instead.

As pointed out on the message, there is a "gitless" version but it is giving me back an error:

vagrant@raring-base:~$ curl https://raw.github.com/creationix/nvm/master/install-gitless.sh | sh
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  1447  100  1447    0     0   1756      0 --:--:-- --:--:-- --:--:--  1864
sh: 3: Syntax error: "(" unexpected

Seems that it is the only thin we'll need to do is install git prior to installing nvm. I'll proceed with the merge and will update the code accordingly. Thanks a lot!

fgrehm commented 11 years ago

Silly me, we always install git :P