Commit Graph

236 Commits (5a4c097aac5a2b90072378cafa0a04be9c88185f)

Author SHA1 Message Date
Ron Korving 5a4c097aac Install script no longer depends on git to be installed.
Also fixed a bug where a literal "\n" would be injected into the bash profile.
2013-06-14 13:00:52 +09:00
Tim Caswell 5c6af59fcd Merge pull request #236 from donovanjamesking/master
Fixes issue 232 and "nvm ls" returning "N/A" when it shouldn't
2013-06-10 06:36:16 -07:00
Tim Caswell 72fd8c970a Merge pull request #229 from em-/testsuite-fixes
Testsuite fixes
2013-06-10 06:34:35 -07:00
Derek Keller d1d53ea979 Updated fix for issues #232 and #244
Fixes issues #232 and #244 where "nvm ls" returns inconsistent results depending on the number of node versions installed.  This updated fix uses the find and sort commands only.  Tested in bash v3.2.48 and v4.2.25, and zsh v4.3.11.
2013-06-09 21:44:46 -05:00
Emanuele Aina 04dba10b9a Fix the setup/teardown urchin test scripts under zsh
The v* glob expansions were failing with an error under zsh, just set
nullglob/NULL_GLOB and make both bash and zsh happier.
2013-06-07 23:18:50 +02:00
Emanuele Aina 8486a0da2b Fix the 'nvm deactivate' test
`nvm deactivate` does not really remove the nvm command, it just remove
the entries from $PATH and $MANPATH previously adde with `nvm use`.
2013-06-07 23:18:50 +02:00
Tim Caswell b2b29ff5c5 Merge pull request #243 from lgw4/master
Changed git download protocol from git:// to https:// to work around firewalls.
2013-06-03 18:57:57 -07:00
Chip Warden 6c57b275db Changed git protocol from git:// to https:// to work around firewalls. 2013-06-03 17:05:30 -05:00
Tim Caswell 7cc6f37e69 Merge pull request #242 from koenpunt/patch-1
Return 1 so exit status is not null on error
2013-06-03 14:18:03 -07:00
Koen Punt 0f05655ee6 Return 1 so exit status is not null on error 2013-06-03 22:26:52 +03:00
Tim Caswell 6b932e59c3 Remove column since many common environments don't have it. 2013-06-03 10:08:13 -04:00
Tim Caswell 66633de1a9 Merge pull request #235 from okuryu/fix-install
Remove `-e` option in `install.sh` script
2013-05-13 13:01:37 -07:00
Donovan James King bc04ed3ece Fixes issue 232
Fixes issue 232 that occurs in at least bash v4.2.25 where "nvm ls" returns "N/A" no matter how many versions of node have been installed.  The fix uses a combination of the find, tr and basename commands instead of basename only.  Tested in bash v3.2.48 and v4.2.25 and zsh v4.3.11.
2013-05-13 14:35:36 -05:00
Ryuichi Okumura 9205ff8001 Remove `-e` option in `install.sh` script
The `sh` actually doesn't support `-e` option.
2013-05-14 00:05:53 +09:00
Tim Caswell 367ae728fb Merge pull request #231 from miguelcoba/patch-1
Drop invalid -a flag for basename.
2013-05-09 15:02:22 -07:00
Miguel Cobá 335456d008 Drop invalid -a flag for basename.
basename doesn't accept options in bash. This causes the VERSIONS variable get the 'N/A' value for $ nvm ls command.

From basename man page:
NAME
       basename - strip directory and suffix from filenames

SYNOPSIS
       basename NAME [SUFFIX]
       basename OPTION

DESCRIPTION
       Print NAME with any leading directory components removed.  If specified, also remove a trailing SUFFIX.

       --help display this help and exit

       --version
              output version information and exit



coreutils version used in Linux Mint 14 Nadia.

GNU coreutils 8.12.197-032bb                                               September 2011                                                               BASENAME(1)

No idea if this flag exists on FreeBSD or other unixes (if they use gnu coreutils shouldn't be any problem)
2013-05-09 13:50:12 -04:00
Tim Caswell eade411422 Merge pull request #198 from ArtemGordinsky/master
Adding a newline before the source string in the ".bash_profile"
2013-05-09 06:39:01 -07:00
Tim Caswell cca6edfc5c Merge pull request #203 from em-/master
Don't trust the environment and explicitly ask for the builtin `cd'
2013-05-06 20:36:11 -07:00
Emanuele Aina 42915fc21c Reduce usage of `cd' to avoid issues with shell aliases
Since we run in interactive context the builtin `cd' may have been aliased to
print some additional info when invoked, thus it's safer for us to avoid
parsing the output from subshells which calls it.

For instance in .bash_profile I have the following function to redefine
`cd' such that it will list the destination directory contents:
cd () { builtin cd "$@" && ls -F --color=auto }
2013-05-03 17:44:02 +01:00
Emanuele Aina f1b484ee87 Silence stderr warning on `nvm uninstall' with no configured alias 2013-05-03 17:42:14 +01:00
Tim Caswell 6c3708d337 Merge pull request #195 from gpad/patch-1
Resolved Issue #194
2013-04-24 09:03:36 -07:00
Tim Caswell 91a7aa8611 Merge pull request #222 from jnlin/master
Added FreeBSD support
2013-04-24 09:02:12 -07:00
jnlin db7494494c Add FreeBSD support 2013-04-24 15:42:10 +08:00
Tim Caswell 01189e144c Merge pull request #200 from croach/issue-199
Fixes Issue 199
2013-03-07 15:02:22 -08:00
Tim Caswell 3fe12f0a49 Merge pull request #202 from hugojosefson/recommend-version-0.8
Recommend version 0.8 in README.markdown.
2013-03-04 06:54:23 -08:00
Hugo Josefson e4acd63991 Clarify difference between `nvm ls` and `nvm ls-remote`. 2013-03-04 12:54:42 +01:00
Hugo Josefson 3ce23aa726 Recommend version 0.8 in README.markdown.
Automatically install the latest available 0.8.x version.
2013-03-04 12:50:55 +01:00
Christopher Roach 205117bed4 Fixes issue 199
Fixes issue 199 where the bash completion was not working in ZSH
because the builtin bash command `complete` was not found. The fix
executes the `bashcompinit` command which creates the `complete`
function for the Z shell.
2013-03-01 10:55:58 -08:00
Artem 9c7da6cec3 Adding a newline before the source string in the ".bash_profile"
Resolved Issue #197
2013-02-27 11:50:08 +02:00
Artem 1088b4b00f Ignoring the ".DS_Store" files on OS X 2013-02-27 11:43:05 +02:00
GPad 5811cb9e73 Resolved Issue #194
Issue #194
2013-02-11 19:23:25 +01:00
Tim Caswell 0ec339e040 Merge pull request #192 from gpad/nvmrc
Use .nvmrc if present
2013-01-25 11:50:11 -08:00
GPad 7226e5fb98 read the .nvmrc if present 2013-01-25 19:13:40 +01:00
GPad bc9c82cb9f read version from .nvmrc if not specified 2013-01-22 13:28:13 +01:00
Tim Caswell f605581c7d return not exit 2013-01-21 12:07:54 -06:00
Tim Caswell 87a76af398 Merge pull request #191 from koenpunt/patch-1
Add exit code, fix #190
2013-01-21 08:51:14 -08:00
Koen Punt 70e1b81ad3 Add exit code, fix #190 2013-01-21 17:26:16 +01:00
Tim Caswell e145978e73 Merge pull request #189 from kt3k/feature/hyphen-s-option-2
Add -s option to force install from source
2013-01-14 05:59:47 -08:00
Yosiya Hinosawa a6be969403 changed shasums file url because in v0.8.6 .txt.asc file is not available. 2013-01-13 16:13:20 +09:00
Yosiya Hinosawa 184f641730 fixed the condition for the binary package availability. 2013-01-13 16:10:50 +09:00
Yoshiya Hinosawa 7dc0827eaa minor modifications. 2013-01-12 14:43:04 +09:00
Yoshiya Hinosawa 278fd031b0 added test for installation with -s option. 2013-01-12 14:42:41 +09:00
Yoshiya Hinosawa 7013372867 added -s (install from source) option. 2013-01-12 14:42:28 +09:00
Yosiya Hinosawa 881178db01 fixed syntax error. 2013-01-12 14:42:13 +09:00
Yoshiya Hinosawa 1845a4a5a9 added the description about -s option. 2013-01-12 14:40:41 +09:00
Tim Caswell bc508802e5 Update README.markdown 2012-12-12 09:48:26 -06:00
Tim Caswell 0a4d3a2944 Merge pull request #181 from andrwj/master
fixed 'bash command not found sha1sum' in Mac OS
2012-12-08 12:59:52 -08:00
A.J 5596054445 fixed wrong condition checking for 'shasum' (I'm using Mac OS 10.8.2)
line 188, `which shasum > /dev/null 2>&1` will be replaced by:

	'/usr/bin/shasum > /dev/null 2>&1'

but, `/usr/bin/shasum` needs filename argument which is ommited and the
test results always 'false'.
2012-12-09 05:17:13 +09:00
Tim Caswell 68f7dc9cb3 Merge pull request #171 from cou929/support_sha1sum_command
support platforms which does not have shasum command
2012-12-04 17:46:04 -08:00
Tim Caswell 9c3af112ca Merge pull request #175 from Marsup/fix-172
Fix #172 on ZSH
2012-12-04 17:45:25 -08:00