Commit 9d6b6c5c authored by T0mK0's avatar T0mK0 Committed by Robby Russell
Browse files

speed up chruby plugin by eliminating(as much as possible) calls to brew (#6080)

* speed up chruby plugin by eliminating(as much as possible) calls to brew

* speed up aws plugin by eliminating(as much as possible) calls to brew
parent 47039f64
_homebrew-installed() { _homebrew-installed() {
type brew &> /dev/null type brew &> /dev/null
_xit=$?
if [ $_xit -eq 0 ];then
# ok , we have brew installed
# speculatively we check default brew prefix
if [ -h /usr/local/opt/awscli ];then
_brew_prefix="/usr/local/opt/awscli"
else
# ok , it is not default prefix
# this call to brew is expensive ( about 400 ms ), so at least let's make it only once
_brew_prefix=$(brew --prefix awscli)
fi
return 0
else
return $_xit
fi
} }
_awscli-homebrew-installed() { _awscli-homebrew-installed() {
brew list awscli &> /dev/null [ -r $_brew_prefix/libexec/bin/aws_zsh_completer.sh ] &> /dev/null
} }
export AWS_HOME=~/.aws export AWS_HOME=~/.aws
...@@ -28,7 +43,7 @@ function aws_profiles { ...@@ -28,7 +43,7 @@ function aws_profiles {
compctl -K aws_profiles asp compctl -K aws_profiles asp
if _homebrew-installed && _awscli-homebrew-installed ; then if _homebrew-installed && _awscli-homebrew-installed ; then
_aws_zsh_completer_path=$(brew --prefix awscli)/libexec/bin/aws_zsh_completer.sh _aws_zsh_completer_path=$_brew_prefix/libexec/bin/aws_zsh_completer.sh
else else
_aws_zsh_completer_path=$(which aws_zsh_completer.sh) _aws_zsh_completer_path=$(which aws_zsh_completer.sh)
fi fi
......
...@@ -16,12 +16,28 @@ ...@@ -16,12 +16,28 @@
# rvm and rbenv plugins also provide this alias # rvm and rbenv plugins also provide this alias
alias rubies='chruby' alias rubies='chruby'
_homebrew-installed() { _homebrew-installed() {
whence brew &> /dev/null whence brew &> /dev/null
_xit=$?
if [ $_xit -eq 0 ];then
# ok , we have brew installed
# speculatively we check default brew prefix
if [ -h /usr/local/opt/chruby ];then
_brew_prefix="/usr/local/opt/chruby"
else
# ok , it is not default prefix
# this call to brew is expensive ( about 400 ms ), so at least let's make it only once
_brew_prefix=$(brew --prefix chruby)
fi
return 0
else
return $_xit
fi
} }
_chruby-from-homebrew-installed() { _chruby-from-homebrew-installed() {
[ -r $(brew --prefix chruby) ] &> /dev/null [ -r _brew_prefix ] &> /dev/null
} }
_ruby-build_installed() { _ruby-build_installed() {
...@@ -64,8 +80,8 @@ _chruby_dirs() { ...@@ -64,8 +80,8 @@ _chruby_dirs() {
} }
if _homebrew-installed && _chruby-from-homebrew-installed ; then if _homebrew-installed && _chruby-from-homebrew-installed ; then
source $(brew --prefix chruby)/share/chruby/chruby.sh source $_brew_prefix/share/chruby/chruby.sh
source $(brew --prefix chruby)/share/chruby/auto.sh source $_brew_prefix/share/chruby/auto.sh
_chruby_dirs _chruby_dirs
elif [[ -r "/usr/local/share/chruby/chruby.sh" ]] ; then elif [[ -r "/usr/local/share/chruby/chruby.sh" ]] ; then
source /usr/local/share/chruby/chruby.sh source /usr/local/share/chruby/chruby.sh
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment