Permalink
Browse files

use -rbundler/setup instead of bundle exec ruby since it is faster an…

…d has the same effect (2.3s vs 4s)
  • Loading branch information...
grosser committed Oct 6, 2012
1 parent 17df49e commit cca1af185dea5d9344c1e82e827c0cf388ab1fed
Showing with 5 additions and 1 deletion.
  1. +5 −1 bundler-exec.sh
View
@@ -24,7 +24,11 @@ within-bundled-project()
run-with-bundler()
{
if bundler-installed && within-bundled-project; then
- bundle exec "$@"
+ if [ $command == "ruby" ]; then
+ ruby -rbundler/setup "$@"
+ else
+ bundle exec $command "$@"
+ fi
else
"$@"
fi

3 comments on commit cca1af1

@gma

This comment has been minimized.

Show comment Hide comment
@gma

gma Nov 12, 2012

Owner

I've just reverted this – $command isn't defined. My bad for not checking it properly…

Owner

gma replied Nov 12, 2012

I've just reverted this – $command isn't defined. My bad for not checking it properly…

@grosser

This comment has been minimized.

Show comment Hide comment
@grosser

grosser Nov 12, 2012

Contributor

sorry, my bad, I have an older version locally and just copy-pasted the changes I made :(
can you fix it by replacing $command by whatever store the command ?

Contributor

grosser replied Nov 12, 2012

sorry, my bad, I have an older version locally and just copy-pasted the changes I made :(
can you fix it by replacing $command by whatever store the command ?

@gma

This comment has been minimized.

Show comment Hide comment
@gma

gma Nov 12, 2012

Owner
Owner

gma replied Nov 12, 2012

Please sign in to comment.