Age | Commit message (Collapse) | Author | Files | Lines |
|
Address an execution race in `close_wait_program' and use `catch' in
killing pending force-kills issued there in the recovery of a stuck test
case, in case the force-kill sequence has completed before the command
to kill the sequence had a chance to run, so that no error is thrown and
a testsuite run does not get interrupted early like:
PASS: gcc.c-torture/execute/postmod-1.c -O0 (test for excess errors)
Executing on remote-localhost: .../gcc/testsuite/gcc/postmod-1.exe (timeout = 15)
spawn [open ...]
WARNING: program timed out
ERROR: tcl error sourcing .../gcc/testsuite/gcc.c-torture/execute/execute.exp.
ERROR: child process exited abnormally
while executing
"exec sh -c "exec > /dev/null 2>&1 && kill -9 $exec_pid""
(procedure "close_wait_program" line 57)
invoked from within
"close_wait_program $spawn_id $pid wres"
(procedure "local_exec" line 104)
[...]
"uplevel #0 source .../gcc/testsuite/gcc.c-torture/execute/execute.exp"
invoked from within
"catch "uplevel #0 source $test_file_name""
testcase .../gcc/testsuite/gcc.c-torture/execute/execute.exp completed in 196 seconds
=== gcc Summary ===
# of expected passes 1
-- therefore not letting `execute.exp' continue (here with the GCC `c'
testsuite invoked with `execute.exp=postmod-1.c' for 8 compilation and 8
execution tests).
The completion of the force-kill sequence would have to happen in the
window between the `wait' command has returned, which would at worst
happen as a result of the final `kill -9' command in the sequence, and
the `kill -9 $exec_pid' command issued here, and the `sleep 5' command
issued at the end of the force-kill sequence makes the likelihood of
such a scenario low, but this might still happen with a loaded host
system and there is no drawback from using `catch' here, so let's do it.
* lib/remote.exp (close_wait_program): Use `catch' in killing
pending force-kills.
Signed-off-by: Maciej W. Rozycki <macro@wdc.com>
|
|
This commit closes the window during which srcdir was controlled by the
testsuite local init file even if specified on the command line.
Allowing this override causes problems with Automake when the source
directory was set using a relative file name.
|
|
|
|
|
|
|
|
|
|
Previously, two procedures were defined between setting logname and
reporting its value if verbosity is selected. This does not change
program flow, but will make the code easier to examine in the future.
|
|
|
|
This was causing testsuite/runtest.main/stats.exp to fail unless the
DejaGnu sources were in a directory named dejagnu.
|
|
The previous merge duplicated the "target_link procedure" node.
|
|
|
|
|
|
|
|
|
|
|
|
checking a local empty target_info array due to lacking global target_info.
|
|
|
|
compiler is given.
|
|
|
|
|
|
|
|
|
|
board_info array.
|
|
|
|
|
|
argument list for the remote_exec procedure.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
filename once using 'file join'.
Signed-off-by: Ben Elliston <bje@gnu.org>
|
|
spacing so the Emacs move-by-sentence commands work correctly.
Signed-off-by: Ben Elliston <bje@gnu.org>
|
|
file" throughout.
|
|
formatting to fix overlapping columns and revise text slightly.
Signed-off-by: Ben Elliston <bje@gnu.org>
|
|
Tcl "switch" command instead of the deprecated "case" command.
Signed-off-by: Ben Elliston <bje@gnu.org>
|
|
|
|
the structure of the "runtests" argument.
Signed-off-by: Ben Elliston <bje@gnu.org>
|
|
|
|
for verifying output of clone_output.
* testsuite/runtest.libs/default_procs.tcl (send_error): Roll into
template using store_test_output.
(send_log): Likewise.
(send_user): Likewise.
(clear_test_output): New proc.
(store_test_output): New proc.
* lib/framework.exp (clone_output): Actually send errors to the
error stream; bug found while improving testsuite.
Signed-off-by: Ben Elliston <bje@gnu.org>
|
|
lines from child process correctly.
Signed-off-by: Ben Elliston <bje@gnu.org>
|
|
* Makefile.am (clean-local): Add target.
(clean-local-check): Add target; mark as PHONY.
(commands_DATA): Add "report-card" scripts.
(dist_man_MANS): Add dejagnu-report-card.1 and split.
(DEJATOOL): Add "report-card" tool.
(TESTSUITE_FILES): Add testsuite for "report-card" tool.
* commands/report-card.awk: New command script.
* doc/dejagnu.texi (Invoking dejagnu report card): New node.
* doc/dejagnu-report-card.1: New man page.
* testsuite/lib/bohman_ssd.exp: New file.
* testsuite/lib/report-card.exp: New file.
* testsuite/report-card.all/onetest.exp: New file.
* testsuite/report-card.all/passes.exp: New file.
Signed-off-by: Ben Elliston <bje@gnu.org>
|
|
(CLEANFILES): Use this instead.
* Makefile.in: Regenerate.
|
|
* configure: Regenerate.
|
|
* config.sub: Likewise.
|
|
* configure: Regenerate.
|
|
POSIX does not clearly require sed(1) to support that feature.
Signed-off-by: Ben Elliston <bje@gnu.org>
|