From 5b1dae570aee8f2917c082a172b0268e7cfc2682 Mon Sep 17 00:00:00 2001 From: Stefano Lattarini Date: Sun, 5 Aug 2012 12:14:19 +0200 Subject: tests: avoid tons of spurious failures on NetBSD * t/ax/am-test-lib.sh (process_requirements): Set the '$am_tool' variable to the empty string before trying to unset it; otherwise, we might be attempting to unset an already-unset variable, which (together with the presence of the 'errexit' shell flag) causes spurious failures at least with the /bin/sh shell from NetBSD 5.1. This was actually causing the great majority of the Automake tests (all those not using a "required=..." declaration) to fail spuriously on that platform! Signed-off-by: Stefano Lattarini --- t/ax/am-test-lib.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 't/ax/am-test-lib.sh') diff --git a/t/ax/am-test-lib.sh b/t/ax/am-test-lib.sh index ee5606822..e69a9046b 100644 --- a/t/ax/am-test-lib.sh +++ b/t/ax/am-test-lib.sh @@ -740,7 +740,7 @@ process_requirements () for am_tool in $*; do require_tool $am_tool done - unset am_tool + am_tool=; unset am_tool # We might need extra macros, e.g., from Libtool or Gettext. case " $required " in *\ libtool*) . ./t/libtool-macros.dir/get.sh;; -- cgit v1.2.1