setupenv.bash misses to declare some variables as local, cluttering shell env

[mp@…]

Simple fix:

--- setupenv.bash.orig  2011-08-03 23:14:52.000000000 +0200
+++ setupenv.bash       2011-08-03 23:13:11.000000000 +0200
@@ -32,8 +32,9 @@
 function export_path() {
     local binpath="/opt/local/bin"
     local sbinpath="/opt/local/sbin"
-
     local IFS=":"
+    local p
+
     for p in $PATH; do
         if [ "$p" == "$binpath" ]; then
             binpath=""
@@ -54,13 +55,14 @@
 }
 
 function export_manpath() {
-    mpath="/opt/local/share/man"
+    local mpath="/opt/local/share/man"
+    local IFS=":"
+    local p
 
     if [ -z "$MANPATH" ]; then
         return
     fi
 
-    local IFS=":"
     for p in $MANPATH; do
         if [ "$p" == "$mpath" ]; then
             mpath=""

[jmroot (Joshua Root)]

Please remember to use WikiFormatting, or better, just add patches as attachments.

[mp@…]

Cc Me!

[neverpanic (Clemens Lang)]

You don't need to cc yourself.

Fixed in r90601. I didn't use the patch, because setupenv.bash is created by autoconf from setupenv.bash.in and the patch didn't apply against setupenv.bash.in.