I'll push this tomorrow: >From dd45755406861fc192284d7e2b28f3c33e0433d3 Mon Sep 17 00:00:00 2001 From: Jim Meyering <meyer...@redhat.com> Date: Sun, 7 Nov 2010 14:23:32 +0100 Subject: [PATCH] tests: don't disqualify FreeBSD 8.1's /bin/sh just yet,
because dash-0.5.6-2.fc14.x86_64 would also be disqualified. This reverts part of yesterday's commit 6c058b2d, "tests: avoid failure due to bug in FreeBSD 8.1's /bin/sh". Note that the offending aspect of those shells is not officially a bug, since "local" is not specified by POSIX. However, it is very unintuitive that prepending "local" to an existing, standards-compliant assignment would evoke such a fundamental change in semantics. * tests/init.sh: Remove snippet requiring sane "local" support. Upstream dash bug report: http://thread.gmane.org/gmane.comp.shells.dash/419 --- tests/init.sh | 4 ---- 1 files changed, 0 insertions(+), 4 deletions(-) diff --git a/tests/init.sh b/tests/init.sh index 0193cbe..a57de77 100644 --- a/tests/init.sh +++ b/tests/init.sh @@ -93,9 +93,6 @@ else fi # We require $(...) support unconditionally. -# We also require "local" support. The local-vs-IFS test is required to -# eliminate FreeBSD 8.1's /bin/sh, which would otherwise pass these tests -# and provoke opaque test failures in coreutils. # We require a few additional shell features only when $EXEEXT is nonempty, # in order to support automatic $EXEEXT emulation: # - hyphen-containing alias names @@ -118,7 +115,6 @@ fi # ? - not ok gl_shell_test_script_=' test $(echo y) = y || exit 1 -gl_local_test_(){ local s=$IFS; test -n "$s"; }; gl_local_test_ || exit 1 score_=10 if test "$VERBOSE" = yes; then test -n "$( (exec 3>&1; set -x; P=1 true 2>&3) 2> /dev/null)" && score_=9 -- 1.7.3.2.193.g78bbb