| Message ID | m3y3kv10st.fsf@oc1027705133.ibm.com |
|---|---|
| State | New |
| Headers | show |
On 01/18/2018 06:25 PM, Andreas Arnez wrote: > In August 2017 the GDB test suite was changed to always add the compile > option "-fdiagnostics-color=never", see: > > https://sourceware.org/ml/gdb-patches/2017-08/msg00150.html > > Since this option is not understood by rustc, a commit from 09/2017 > dropped its use in that case: > > https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;a=commit;h=5eb5f850 > ("Don't use -fdiagnostics-color=never for rustc") > > But that change goes overboard and stops using the option for other > languages as well. Thus compiler diagnostics written into gdb.log may > contain colored output again. This is fixed. > > gdb/testsuite/ChangeLog: > > * lib/gdb.exp (gdb_compile): Re-enable use of > universal_compile_options for languages other than Rust. OK. Looks obvious to me. Thanks, Pedro Alves
On Thu, Jan 18 2018, Pedro Alves wrote: > On 01/18/2018 06:25 PM, Andreas Arnez wrote: [...] >> gdb/testsuite/ChangeLog: >> >> * lib/gdb.exp (gdb_compile): Re-enable use of >> universal_compile_options for languages other than Rust. > > OK. Looks obvious to me. Thanks, pushed. -- Andreas
diff --git a/gdb/testsuite/lib/gdb.exp b/gdb/testsuite/lib/gdb.exp index a4bde72..7ee3a70 100644 --- a/gdb/testsuite/lib/gdb.exp +++ b/gdb/testsuite/lib/gdb.exp @@ -3504,12 +3504,12 @@ proc gdb_compile {source dest type options} { # Add platform-specific options if a shared library was specified using # "shlib=librarypath" in OPTIONS. + set new_options {} if {[lsearch -exact $options rust] != -1} { # -fdiagnostics-color is not a rustcc option. } else { set new_options [universal_compile_options] } - set new_options {} set shlib_found 0 set shlib_load 0 foreach opt $options {
In August 2017 the GDB test suite was changed to always add the compile option "-fdiagnostics-color=never", see: https://sourceware.org/ml/gdb-patches/2017-08/msg00150.html Since this option is not understood by rustc, a commit from 09/2017 dropped its use in that case: https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;a=commit;h=5eb5f850 ("Don't use -fdiagnostics-color=never for rustc") But that change goes overboard and stops using the option for other languages as well. Thus compiler diagnostics written into gdb.log may contain colored output again. This is fixed. gdb/testsuite/ChangeLog: * lib/gdb.exp (gdb_compile): Re-enable use of universal_compile_options for languages other than Rust. --- gdb/testsuite/lib/gdb.exp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)