fix #50 by escaping rm before calling it

Author: dhellmann

README.txt

@@ -72,6 +72,16 @@ issues and features.
 Report bugs via the `bug tracker on BitBucket
 <http://bitbucket.org/dhellmann/virtualenvwrapper/>`__.
 
+Shell Aliases
+=============
+
+Since virtualenvwrapper is largely a shell script, it uses shell
+commands for a lot of its actions.  If your environment makes heavy
+use of shell aliases or other customizations, you may encounter
+issues.  Before reporting bugs in the bug tracker, please test
+*without* your aliases enabled.  If you can identify the alias causing
+the problem, that will help make virtualenvwrapper more robust.
+
 =======
 License
 =======

docs/en/history.rst

@@ -8,6 +8,8 @@ Release History
   - Integrate Manuel Kaufmann's patch to unset GREP_OPTIONS before
     calling grep.  Resolves :bbissue:`51`.
   - Escape ``$`` in regex to resolve :bbissue:`53`.
+  - Escape ``rm`` to avoid issues with aliases and resolve
+    :bbissue:`50`.
 
 2.2
 

docs/en/index.rst

@@ -212,6 +212,16 @@ issues and features.
 Report bugs via the `bug tracker on BitBucket
 <http://bitbucket.org/dhellmann/virtualenvwrapper/>`__.
 
+Shell Aliases
+=============
+
+Since virtualenvwrapper is largely a shell script, it uses shell
+commands for a lot of its actions.  If your environment makes heavy
+use of shell aliases or other customizations, you may encounter
+issues.  Before reporting bugs in the bug tracker, please test
+*without* your aliases enabled.  If you can identify the alias causing
+the problem, that will help make virtualenvwrapper more robust.
+
 =======
 License
 =======

tests/test_rmvirtualenv.sh

@@ -29,6 +29,16 @@ test_remove () {
     assertFalse "[ -d $WORKON_HOME/deleteme ]"
 }
 
+test_rm_aliased () {
+    mkvirtualenv "deleteme"
+    deactivate
+    alias rm='rm -i'
+    set -x
+    rmvirtualenv "deleteme"
+    set +x
+    unalias rm
+}
+
 test_no_such_env () {
     assertFalse "[ -d $WORKON_HOME/deleteme ]"
     assertTrue "rmvirtualenv deleteme"

virtualenvwrapper.sh

@@ -106,7 +106,7 @@ virtualenvwrapper_tempfile () {
         echo "ERROR: virtualenvwrapper could not create a temporary file name." 1>&2
         return 1
     fi
-    trap "rm -f '$file' >/dev/null 2>&1" EXIT
+    trap "\rm -f '$file' >/dev/null 2>&1" EXIT
     echo $file
     return 0
 }
@@ -131,7 +131,7 @@ virtualenvwrapper_run_hook () {
         fi
         source "$hook_script"
     fi
-    rm -f "$hook_script" >/dev/null 2>&1
+    \rm -f "$hook_script" >/dev/null 2>&1
     return $result
 }
 
@@ -223,7 +223,7 @@ rmvirtualenv () {
         return 1
     fi
     virtualenvwrapper_run_hook "pre_rmvirtualenv" "$env_name"
-    rm -rf "$env_dir"
+    \rm -rf "$env_dir"
     virtualenvwrapper_run_hook "post_rmvirtualenv" "$env_name"
 }