diff options
author | Eric Blake <eblake@redhat.com> | 2011-05-24 10:04:46 -0600 |
---|---|---|
committer | Eric Blake <eblake@redhat.com> | 2011-05-24 11:05:24 -0600 |
commit | 866bbaee7bf64b5f54b2b3d566552782dfb514dd (patch) | |
tree | e7ee45af564f7601fabf68a8b2372254b84ce490 | |
parent | 6dc42e2d25df9c84b335062bad9beb0a7319647b (diff) | |
download | gnulib-866bbaee7bf64b5f54b2b3d566552782dfb514dd.tar.gz |
tests: fix logic bug in init.sh
If the shell test loop first finds a marginal then a good shell, the
variable $gl_set_x_corrupts_stderr is still set to true and needlessly
drops $VERBOSE logging.
* tests/init.sh: (gl_set_x_corrupts_stderr_): Clear for successful
shell.
Signed-off-by: Eric Blake <eblake@redhat.com>
-rw-r--r-- | ChangeLog | 10 | ||||
-rw-r--r-- | tests/init.sh | 5 |
2 files changed, 12 insertions, 3 deletions
@@ -1,3 +1,9 @@ +2011-05-24 Eric Blake <eblake@redhat.com> + + tests: fix logic bug in init.sh + * tests/init.sh: (gl_set_x_corrupts_stderr_): Clear for successful + shell. + 2011-05-24 Jim Meyering <meyering@redhat.com> utimensat: do not reference an out-of-scope buffer @@ -5,8 +11,8 @@ declared in an inner scope, yet "times" would be dereferenced outside the scope in which "ts" was valid. * lib/utimensat.c (rpl_utimensat) [__linux__]: Move the declaration - of ts[2] "out/up", so that the use of aliased "times" (via "times = ts;") - does not end up referencing an out-of-scope "ts" + of ts[2] "out/up", so that the use of aliased "times" (via + "times = ts;") does not end up referencing an out-of-scope "ts" opendir-safer.c: don't clobber errno; don't close negative FD * lib/opendir-safer.c (opendir_safer): diff --git a/tests/init.sh b/tests/init.sh index 71c6516962..294dcdd0f6 100644 --- a/tests/init.sh +++ b/tests/init.sh @@ -167,7 +167,10 @@ else st_=$? # $re_shell_ works just fine. Use it. - test $st_ = 10 && break + if test $st_ = 10; then + gl_set_x_corrupts_stderr_=false + break + fi # If this is our first marginally acceptable shell, remember it. if test "$st_:$marginal_" = 9: ; then |