Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

minor fixes to ry::install #16

Merged
merged 2 commits into from

2 participants

perlguy Jeanine Adkisson
perlguy

ruby-build and internal routines share need for -z $url and both can make use of $dir.

when using the internal routines (no ruby-build available) an empty $name was causing "src" to appear in the installed version list...

Jeanine Adkisson
Owner

Neat. That does make it easier to read. Thanks!

Jeanine Adkisson jneen merged commit ef82caf into from
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on May 16, 2012
  1. perlguy

    minor fixes to ry::install

    perlguy authored
  2. perlguy

    fix to the fix...

    perlguy authored
This page is out of date. Refresh to see the latest.
Showing with 4 additions and 4 deletions.
  1. +4 −4 bin/ry
8 bin/ry
View
@@ -119,14 +119,14 @@ ry::install() {
# if you've got ruby-build, and you've named one of its recipes,
# use that instead :)
+ [[ -z "$url" ]] && abort "no URL given. usage: ry install <url> <name>"
+ [[ -z "$name" ]] && name="$url"
+ local dir="$RY_LIB/rubies/$name"
if exists? ruby-build \
&& ruby-build --definitions | grep -qx "$url"; then
- [[ -z "$name" ]] && name="$url"
- ruby-build "$url" "$RY_LIB/rubies/$name"
+ ruby-build "$url" "$dir"
else
- [[ -z "$url" ]] && abort "no URL given. usage: ry install <name> <url>"
- local dir="$RY_LIB/rubies/$name"
local logpath="/tmp/ry.log"
# create build directory
Something went wrong with that request. Please try again.