[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 1/5] Tests defs: don't let useless variables leak in test scripts
From: |
Stefano Lattarini |
Subject: |
[PATCH 1/5] Tests defs: don't let useless variables leak in test scripts. |
Date: |
Mon, 15 Nov 2010 18:21:16 +0100 |
User-agent: |
KMail/1.13.3 (Linux/2.6.30-2-686; KDE/4.4.4; i686; ; ) |
* tests/defs ($priv_check_temp, $overwrite_status, $ro_dir_temp,
$create_status, $r2h, $libtool_found, $gettext_found, $aclocaldir,
$extra_includes): Unset once they've served their purpose.
---
ChangeLog | 7 +++++++
tests/defs | 5 +++++
2 files changed, 12 insertions(+), 0 deletions(-)
From cadeda85ac9c55bee84a280ee1bfdb4e5ce78479 Mon Sep 17 00:00:00 2001
From: Stefano Lattarini <address@hidden>
Date: Fri, 12 Nov 2010 14:52:51 +0100
Subject: [PATCH 1/5] Tests defs: don't let useless variables leak in test
scripts.
* tests/defs ($priv_check_temp, $overwrite_status, $ro_dir_temp,
$create_status, $r2h, $libtool_found, $gettext_found, $aclocaldir,
$extra_includes): Unset once they've served their purpose.
---
ChangeLog | 7 +++++++
tests/defs | 5 +++++
2 files changed, 12 insertions(+), 0 deletions(-)
diff --git a/ChangeLog b/ChangeLog
index 634a3ed..8c5bf41 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2010-11-12 Stefano Lattarini <address@hidden>
+
+ Tests defs: don't let useless variables leak in test scripts.
+ * tests/defs ($priv_check_temp, $overwrite_status, $ro_dir_temp,
+ $create_status, $r2h, $libtool_found, $gettext_found, $aclocaldir,
+ $extra_includes): Unset once they've served their purpose.
+
2010-11-11 Stefano Lattarini <address@hidden>
Tests defs: improve messages for skipped tests.
diff --git a/tests/defs b/tests/defs
index 82c3b5b..e5f54ba 100644
--- a/tests/defs
+++ b/tests/defs
@@ -233,6 +233,7 @@ do
echo "$me: cannot drop file write permissions" >&2
exit 77
fi
+ unset priv_check_temp overwrite_status
;;
perl-threads)
if test "$WANT_NO_THREADS" = "yes"; then
@@ -258,6 +259,7 @@ do
echo "$me: cannot drop directory write permissions" >&2
exit 77
fi
+ unset ro_dir_temp create_status
;;
rst2html)
# Try the variants that are tried in check.am.
@@ -269,6 +271,7 @@ do
echo "$me: no proper rst2html program found" >&2
exit 77
done
+ unset r2h
;;
runtest)
# DejaGnu's runtest program. We rely on being able to specify
@@ -375,6 +378,8 @@ case " $required " in
;;
esac
ACLOCAL="$ACLOCAL -Wno-syntax -I $top_testsrcdir/m4 $extra_includes -I
$aclocaldir"
+ unset libtool_found gettext_found
+ unset extra_includes aclocaldir
;;
esac
--
1.7.1