diff options
| author | Pádraig Brady <P@draigBrady.com> | 2016-11-22 01:23:22 +0000 |
|---|---|---|
| committer | Pádraig Brady <P@draigBrady.com> | 2016-11-22 20:04:25 +0000 |
| commit | 055b08ff7853c7c9ab791643047b1b163a227013 (patch) | |
| tree | 15d7a014a486c9c0189cef9511c7e7c7b0470620 /tests/rm | |
| parent | b572a286d5d24cdb1669baa3f867317bdb534b1a (diff) | |
| download | coreutils-055b08ff7853c7c9ab791643047b1b163a227013.tar.xz | |
tests: fix false fails due to passing env vars to returns_
On BSD /bin/sh it was seen that unexported env vars passed to
returns_() would not be propagated to the wrapped command.
* cfg.mk (sc_prohibit_env_returns): Add a syntax check to disallow.
* tests/misc/csplit-io-err.sh: Rearrange to export vars in a subshell.
* tests/rm/rm-readdir-fail.sh: Likewise.
* tests/misc/nohup.sh: Export and unset vars around returns_.
* tests/misc/printenv.sh: Likewise.
Reported by Assaf Gordon
Diffstat (limited to 'tests/rm')
| -rwxr-xr-x | tests/rm/rm-readdir-fail.sh | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/tests/rm/rm-readdir-fail.sh b/tests/rm/rm-readdir-fail.sh index d9d8efdb4..835d3424f 100755 --- a/tests/rm/rm-readdir-fail.sh +++ b/tests/rm/rm-readdir-fail.sh @@ -88,7 +88,8 @@ gcc_shared_ k.c k.so \ export READDIR_PARTIAL for READDIR_PARTIAL in '' '1'; do rm -f preloaded - (LD_PRELOAD=$LD_PRELOAD:./k.so returns_ 1 rm -Rf dir 2>>err) || fail=1 + (export LD_PRELOAD=$LD_PRELOAD:./k.so + returns_ 1 rm -Rf dir 2>>err) || fail=1 test -f preloaded || skip_ "internal test failure: maybe LD_PRELOAD doesn't work?" done |