mirror of
https://github.com/git/git
synced 2024-08-25 02:35:46 +00:00
Merge branch 'jk/test-send-sh-x-trace-elsewhere' into maint
Running tests with '-x' option to trace the individual command executions is a useful way to debug test scripts, but some tests that capture the standard error stream and check what the command said can be broken with the trace output mixed in. When running our tests under "bash", however, we can redirect the trace output to another file descriptor to keep the standard error of programs being tested intact. * jk/test-send-sh-x-trace-elsewhere: test-lib: set BASH_XTRACEFD automatically
This commit is contained in:
commit
68a6e976a8
6
t/README
6
t/README
|
@ -84,9 +84,9 @@ appropriately before running "make".
|
|||
|
||||
-x::
|
||||
Turn on shell tracing (i.e., `set -x`) during the tests
|
||||
themselves. Implies `--verbose`. Note that this can cause
|
||||
failures in some tests which redirect and test the
|
||||
output of shell functions. Use with caution.
|
||||
themselves. Implies `--verbose`. Note that in non-bash shells,
|
||||
this can cause failures in some tests which redirect and test
|
||||
the output of shell functions. Use with caution.
|
||||
|
||||
-d::
|
||||
--debug::
|
||||
|
|
|
@ -322,6 +322,19 @@ else
|
|||
exec 4>/dev/null 3>/dev/null
|
||||
fi
|
||||
|
||||
# Send any "-x" output directly to stderr to avoid polluting tests
|
||||
# which capture stderr. We can do this unconditionally since it
|
||||
# has no effect if tracing isn't turned on.
|
||||
#
|
||||
# Note that this sets up the trace fd as soon as we assign the variable, so it
|
||||
# must come after the creation of descriptor 4 above. Likewise, we must never
|
||||
# unset this, as it has the side effect of closing descriptor 4, which we
|
||||
# use to show verbose tests to the user.
|
||||
#
|
||||
# Note also that we don't need or want to export it. The tracing is local to
|
||||
# this shell, and we would not want to influence any shells we exec.
|
||||
BASH_XTRACEFD=4
|
||||
|
||||
test_failure=0
|
||||
test_count=0
|
||||
test_fixed=0
|
||||
|
|
Loading…
Reference in a new issue