ag/cpython-source-deps
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix dejagnu test support for --tool_opts

Browse Source 
Right now it concatenates it with the existing options and then appends
it to that list, fix it to simply append it as is, same as it is done
with the other variables.

Tested by running the following command which includes gcc options:
  $ make check RUNTESTFLAGS="--tool_opts '-Werror'"

Without this patch, all the tests fail. With it, the test succeed.
Inspecting the logs shows that -Werror was indeed used when compiling
the test sources.
This commit is contained in:
Filipe Brandenburger
2015-10-19 15:33:23 -07:00
parent 6ba2a49e22
commit 89b760503e
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
testsuite/lib/libffi.exp
View File

@@ -182,7 +182,7 @@ proc libffi_target_compile { source dest type options } {
# TOOL_OPTIONS must come first, so that it doesn't override testcase # TOOL_OPTIONS must come first, so that it doesn't override testcase
# specific options. # specific options.
if [info exists TOOL_OPTIONS] { if [info exists TOOL_OPTIONS] {
lappend options [concat "additional_flags=$TOOL_OPTIONS" $options]; lappend options "additional_flags=$TOOL_OPTIONS"
} }
# search for ffi_mips.h in srcdir, too # search for ffi_mips.h in srcdir, too