[PATCH 1/2] libctf, testsuite: don't run without a suitable compiler

Jan Beulich jbeulich@suse.com
Wed Feb 3 09:14:01 GMT 2021


On 05.01.2021 20:52, Nick Alcock via Binutils wrote:
> We never actually check to see if the compiler supports CTF,
> or even if a suitable compiler exists.
> 
> libctf/ChangeLog
> 2021-01-05  Nick Alcock  <nick.alcock@oracle.com>
> 
> 	* Makefile.am (BASEDIR): New.
> 	(BFDDIR): Likewise.
> 	(check-DEJAGNU): Add development.exp to prerequisites.
> 	(development.exp): New.
> 	(CONFIG_STATUS_DEPENDENCIES): New.
> 	(EXTRA_DEJAGNU_SITE_CONFIG): Likewise.
> 	(DISTCLEANFILES): Likewise.
> 	* Makefile.in: Regenerated.
> 	* testsuite/lib/ctf-lib.exp (check_ctf_available): Return boolean.
> 	* testsuite/libctf-lookup/lookup.exp: Call check_ctf_available.
> 	* testsuite/libctf-regression/regression.exp: Likewise.

Is there any reason the same check didn't get added here to
testsuite/libctf-writable/writable.exp? If it was there, on an
older host I expect I wouldn't have run into

ERROR: (DejaGnu) proc "try {
	set run_output [exec "sh" "-c" "$command" "2>@1"]
	set status 0
    } trap CHILDSTATUS {results options} {
	set status [lindex [dict get $options -errorcode] 2]
	set run_output $results
    }" does not exist.
The error code is CHILDSTATUS 22593 1
The info on the error is:
child process exited abnormally
    while executing
"close file7"
    invoked from within
"catch "close $id" res"

because the Tcl there is 8.5, but "try" is available only
from 8.6. I think this dependency would also be nice to be
checked.

Jan


More information about the Binutils mailing list