Skip to content

Skip alias versions by default, add --aliases. #19

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Skip alias versions by default, add --aliases. #19

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
50e91f2
Skip alias versions, add --with-aliases option.
ab Mar 29, 2016
27ee84d
Remove extra shift in --verbose processing.
ab Mar 30, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,4 +22,5 @@ the output.
```
$ rbenv each bundle install
$ rbenv each -v rake test
$ rbenv each --with-aliases bundle check
```
78 changes: 54 additions & 24 deletions bin/rbenv-each
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,36 +2,60 @@
#
# Summary: Execute a command for each Ruby version
#
# Usage: rbenv each [-v] <command> [arg1 arg2...]
# Usage: rbenv each [options] <command> [arg1 arg2...]
Copy link
Member

@mislav mislav Mar 29, 2016

Choose a reason for hiding this comment

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

I wanted this to say [-va] instead of options but I realized that our option parsing doesn't support combining multiple shorthand flags together. So these docs will do for now.

#
# Executes a command for each Ruby version by setting RBENV_VERSION.
# Execute a command for each Ruby version by setting RBENV_VERSION.
# Failures are collected and reported at the end.
#
# -v Verbose mode. Prints a header for each ruby.
#
# Options:
# -h, --help Print this help message
# -a, --with-aliases Don't skip ruby versions that are aliases (symlinks)
# -v, --verbose Verbose mode: print a header for each ruby version
#
set -e
[ -n "$RBENV_DEBUG" ] && set -x

# Provide rbenv completions
case "$1" in
--complete )
echo --help
echo --verbose
exit
;;
-v | --verbose )
verbose=1
shift
;;
--help )
rbenv-help each
exit
;;
"" | -* )
rbenv-help --usage each >&2
exit 1
;;
esac
aliases=
verbose=

while [[ $1 == -* ]]; do
case "$1" in
--)
shift
break
;;

# Provide rbenv completions
--complete )
echo --help
echo --verbose
echo --with-aliases
exit
;;
-a | --with-aliases )
aliases=1
Copy link
Member

@mislav mislav Mar 29, 2016

Choose a reason for hiding this comment

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

Seems like this block needs a shift?

Copy link
Author

@ab ab Mar 30, 2016

Choose a reason for hiding this comment

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

It's after the esac. The -- block needed one specially because the break skips it.

EDIT: Oh I see, --verbose has an extra one.

Copy link
Member

@mislav mislav Mar 30, 2016

Choose a reason for hiding this comment

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

Yeah that's what got me confused.

;;
-v | --verbose )
verbose=1
shift
;;
-h | --help )
rbenv-help each
exit
;;
"" | -* )
rbenv-help --usage each >&2
exit 1
;;
esac
shift
done

if [ $# -lt 1 ]; then
rbenv-help --usage each >&2
exit 1
fi

GRAY=""
RED=""
Expand All @@ -49,7 +73,13 @@ failed_rubies=""

trap "exit 1" INT

for ruby in $(rbenv versions --bare); do
if [ -n "$aliases" ]; then
opts=
else
opts="--skip-aliases"
fi

for ruby in $(rbenv versions --bare $opts); do
if [ -n "$verbose" ]; then
header="===[$ruby]==================================================================="
header="${header:0:72}"
Expand Down