Commit e73dd2cd authored by Andrew Grangaard's avatar Andrew Grangaard
Browse files

virtualenvwrapper plugin cleanup

* removes cd override by using chpwd_functions
* removes subshell call to which by using $+commands array and
  c param expansion to find in PATH
* zsh love!
parent 5d6a06bd
wrapsource=`which virtualenvwrapper_lazy.sh` virtualenvwrapper='virtualenvwrapper_lazy.sh'
if (( $+commands[$virtualenvwrapper] )); then
if [[ -f "$wrapsource" ]]; then source ${${virtualenvwrapper}:c}
source $wrapsource
if [[ ! $DISABLE_VENV_CD -eq 1 ]]; then if [[ ! $DISABLE_VENV_CD -eq 1 ]]; then
# Automatically activate Git projects' virtual environments based on the # Automatically activate Git projects's virtual environments based on the
# directory name of the project. Virtual environment name can be overridden # directory name of the project. Virtual environment name can be overridden
# by placing a .venv file in the project root with a virtualenv name in it # by placing a .venv file in the project root with a virtualenv name in it
function workon_cwd { function workon_cwd {
...@@ -40,11 +39,17 @@ if [[ -f "$wrapsource" ]]; then ...@@ -40,11 +39,17 @@ if [[ -f "$wrapsource" ]]; then
fi fi
} }
# New cd function that does the virtualenv magic # Append workon_cwd to the chpwd_functions array, so it will be called on cd
function cd { # http://zsh.sourceforge.net/Doc/Release/Functions.html
builtin cd "$@" && workon_cwd # TODO: replace with 'add-zsh-hook chpwd workon_cwd' when oh-my-zsh min version is raised above 4.3.4
} if (( ${+chpwd_functions} )); then
if (( $chpwd_functions[(I)workon_cwd] == 0 )); then
set -A chpwd_functions $chpwd_functions workon_cwd
fi
else
set -A chpwd_functions workon_cwd
fi
fi fi
else else
print "zsh virtualenvwrapper plugin: Cannot find virtualenvwrapper_lazy.sh. Please install with \`pip install virtualenvwrapper\`." print "zsh virtualenvwrapper plugin: Cannot find ${virtualenvwrapper}. Please install with \`pip install virtualenvwrapper\`."
fi fi
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