Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Fix configure for custom ruby versions #12

Merged
merged 1 commit into from

2 participants

@zhm

It was passing the local name on to configure which was causing an error. It should exclude $1 when running configure. There might be a better way to handle this, but this made it work for me.

@zhm zhm Fix arguments passed to configure for `ry install`
It was passing the local name on to `configure`
which was causing an error. It should exclude
`$1` when running configure.
f1049d9
@jneen
Owner

Awesome, thanks!

@jneen jneen merged commit c155626 into jneen:master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Mar 23, 2012
  1. @zhm

    Fix arguments passed to configure for `ry install`

    zhm authored
    It was passing the local name on to `configure`
    which was causing an error. It should exclude
    `$1` when running configure.
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  bin/ry
View
2  bin/ry
@@ -169,7 +169,7 @@ ry::build() {
if [[ -x "./configure" ]]; then
log "running ./configure --prefix=\"$install_prefix\"..."
- ./configure --prefix="$install_prefix" "$@"
+ ./configure --prefix="$install_prefix" "${@:2}"
else
log "no ./configure script found, skipping..."
fi
Something went wrong with that request. Please try again.