fix lazy-loader function definitions under zsh; fixes #144

Author: dhellmann

docs/en/history.rst

@@ -7,6 +7,7 @@ dev
   - Switch to stevedore_ for plugin management
   - mkvirtualenv_help should use ``$VIRTUALENVWRAPPER_PYTHON`` instead
     of calling ``virtualenv`` directly (:bbissue:`148`).
+  - Fix issue with lazy-loader code under zsh (:bbissue:`144`).
 
 .. _stevedore: http://pypi.python.org/pypi/stevedore
 

tests/test_lazy.sh

@@ -1,12 +1,13 @@
-#!/bin/sh
 
 test_dir=$(cd $(dirname $0) && pwd)
 source "$test_dir/setup.sh"
 
 oneTimeSetUp() {
     rm -rf "$WORKON_HOME"
     mkdir -p "$WORKON_HOME"
+    [ ! -z "$ZSH_VERSION" ] && unsetopt shwordsplit
     source "$test_dir/../virtualenvwrapper_lazy.sh"
+    [ ! -z "$ZSH_VERSION" ] && setopt shwordsplit
 }
 
 oneTimeTearDown() {
@@ -22,6 +23,13 @@ function_defined_lazy() {
     name="$1"
     assertTrue "$name not defined" "type $name"
 	assertTrue "$name does not load virtualenvwrapper" "typeset -f $name | grep 'virtualenvwrapper_load'"
+    if [ "$name" = "mkvirtualenv" ]
+    then
+        lookfor="rmvirtualenv"
+    else
+        lookfor="mkvirtualenv"
+    fi
+	assertFalse "$name includes reference to $lookfor: $(typeset -f $name)" "typeset -f $name | grep $lookfor"
 }
 
 test_mkvirtualenv_defined_lazy() {

virtualenvwrapper_lazy.sh

@@ -20,12 +20,12 @@ function virtualenvwrapper_load {
 # Set up "alias" functions based on the API definition.
 function virtualenvwrapper_setup_lazy_loader {
     typeset venvw_name
-	for venvw_name in $_VIRTUALENVWRAPPER_API
+	for venvw_name in $(echo ${_VIRTUALENVWRAPPER_API})
 	do
 		eval "
 function $venvw_name {
 	virtualenvwrapper_load
-	$venvw_name \"\$@\"
+	${venvw_name} \"\$@\"
 }
 "
 	done