Properly scope local variables.

This is needed specifically in the case of update_hdlist() where a previous run
would set the ofdeps variable which would then be reused on the next run.

This is typically not a big problem but it did mean that standard media file-deps
file was used when updating debug media which seems incorrect.

1 files changed, 11 insertions, 10 deletions

diff --git a/functions b/functions
index 26fc6db..7f5f63c 100644
--- a/functions
+++ b/functions
@@ -25,7 +25,7 @@ fi
 
 function isingroup()
 {
-    grp="$1"
+    local grp="$1"
     for group in `groups`; do
         [ "$grp" = "$group" ] && return 0
     done
@@ -34,8 +34,8 @@ function isingroup()
 
 function inlist()
 {
-    k="$1"
-    list="$2"
+    local k="$1"
+    local list="$2"
     for e in $list; do
         [ "$k" = "$e" ] && return 0
     done
@@ -66,8 +66,9 @@ function valid_subsection()
 
 function update_hdlist()
 {
-    repository="$1"
-    fdeps="$repository/../../media_info/file-deps"
+    local ofdeps
+    local repository="$1"
+    local fdeps="$repository/../../media_info/file-deps"
     test -f "$fdeps" && ofdeps="--file-deps $fdeps"
     $dryrun /usr/bin/genhdlist2 -v --versioned --allow-empty-media $ofdeps "$repository"
 }
@@ -79,9 +80,9 @@ function update_distro_hdlist()
     else
         valid_release "$1" && valid_section "$2" && valid_subsection "$3" || return 1
     fi
-    release="$1"
-    section="$2"
-    subsection="$3"
+    local release="$1"
+    local section="$2"
+    local subsection="$3"
 
     for arch in $arches; do
         update_hdlist "$distribdir/$release/$arch/media/$section/$subsection"
@@ -92,7 +93,7 @@ function update_distro_hdlist()
 
 function update_common_MD5SUM()
 {
-    release="$1"
+    local release="$1"
     for arch in $arches; do
         pushd "$distribdir/$release/$arch/media/media_info"
         $dryrun /bin/sh -c "/usr/bin/md5sum hdlist_* synthesis.* >MD5SUM"
@@ -102,6 +103,6 @@ function update_common_MD5SUM()
 
 function mirror_repository()
 {
-    release="$1"
+    local release="$1"
     /usr/bin/rsync $mirror_rsync_options "$distribdir/$release/" "$finaldistribdir/$release/"
 }