]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commit - gdb/testsuite/ChangeLog
Testsuite: Ensure changing directory does not break the log file
authorAlan Hayward <alan.hayward@arm.com>
Wed, 6 Mar 2019 09:52:08 +0000 (09:52 +0000)
committerAlan Hayward <alan.hayward@arm.com>
Wed, 6 Mar 2019 09:52:08 +0000 (09:52 +0000)
commitc3734e093aab1cea90a76b107cdda4a5870dd1b8
treea57fbe6009b8e2601c283f90ee332a8d52b114e4
parent219d6836e939d2cd1a8c5574bfa3a101daa31cbb
Testsuite: Ensure changing directory does not break the log file

get_compiler_info switches to a new log file before checking the compiler
to ensure the checks are not logged. Afterwards it restores back to using
the original log file. However, the logfile uses a relative path name -
if the current test has changed the current directory then all further
output for the test will be lost.  This can confuse the code that collates
the main gdb.log file at the end of a FORCE_PARALLEL run.

fullpath-expand.exp calls gdb_compile after changing the current directory.

The "Ensure stack protection is off for GCC" patch added a call to
get_compiler_info from inside of gdb_compile, causing log file collection
to break for FORCE_PARALLEL runs.

The ideal solution would be to ensure the log file is always created using
an absolute path name. However, this is set at multiple points in
Makefile.in and in some instances just relies on dejagnu common code to set
the log file directory to "."

The simpler and safer solution is to override the builtin cd function. The
new function checks the current log file and if the path is relative, then
it resets the logging using an absolute path. Finally it calls the builtin
cd.  This ensures get_compiler_info (and any other code) can correctly
backup and restore the current log file.

gdb/testsuite/ChangeLog:

* lib/gdb.exp (builtin_cd): rename of cd.
(cd): Override builtin.
gdb/testsuite/ChangeLog
gdb/testsuite/lib/gdb.exp