Re: [PATCH][gdb/testsuite] Be quiet about untested dtrace-prob.exp

[Luis Machado <luis.machado@linaro.org>]

On 2/18/20 9:02 AM, Tom de Vries wrote:
> Hi,
> 
> When running gdb.base/dtrace-probe.exp, I get this on stdout/stderr:
> ...
> Running src/gdb/testsuite/gdb.base/dtrace-probe.exp ...
> gdb compile failed, ld: error in \
>    build/gdb/testsuite/outputs/gdb.base/dtrace-probe/dtrace-probe.o\
>    (.eh_frame); no .eh_frame_hdr table will be created
> ld: crt1.o: in function `_start':
> start.S:110: undefined reference to `main'
> ld: build/gdb/testsuite/outputs/gdb.base/dtrace-probe/dtrace-probe-p.o:\
>    (.SUNW_dof+0x88): undefined reference to `main'
> ld: build/gdb/testsuite/outputs/gdb.base/dtrace-probe/dtrace-probe-p.o:\
>    (.SUNW_dof+0xb8): undefined reference to `main'
> collect2: error: ld returned 1 exit status
> 
>                  === gdb Summary ===
> 
> nr of untested testcases         1

Odd. I'm running this natively on aarch64 and i don't see the same. I 
only get output in gdb.log.

> ...
> 
> There is no reason to be this verbose about the failure to compile.
> 
> Fix this by using quiet as additional option to gdb_compile in
> dtrace_build_usdt_test_program.  Note that the error message still occurs in
> gdb.log.
> 
> Tested on x86_64-linux.
> 
> OK for trunk?
> 
> Thanks,
> - Tom
> 
> [gdb/testsuite] Be quiet about untested dtrace-prob.exp
> 
> gdb/testsuite/ChangeLog:
> 
> 2020-02-18  Tom de Vries  <tdevries@suse.de>
> 
> 	* lib/dtrace.exp (dtrace_build_usdt_test_program): Use quiet as
> 	gdb_compile option.
> 
> ---
>   gdb/testsuite/lib/dtrace.exp | 6 ++++--
>   1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/gdb/testsuite/lib/dtrace.exp b/gdb/testsuite/lib/dtrace.exp
> index 9aed481f32..8f861c9de4 100644
> --- a/gdb/testsuite/lib/dtrace.exp
> +++ b/gdb/testsuite/lib/dtrace.exp
> @@ -52,7 +52,8 @@ proc dtrace_build_usdt_test_program {} {
>       }
>   
>       # 2. Compile testprogram.c.
> -    set options [list debug additional_flags=-I[file dirname $out_header_file]]
> +    set options [list debug quiet \
> +		     additional_flags=-I[file dirname $out_header_file]]
>       if {[gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}.o" object ${options}] != ""} {
>           return -1
>       }
> @@ -65,7 +66,8 @@ proc dtrace_build_usdt_test_program {} {
>       }
>   
>       # 4. Link everything together to get the test program.
> -    if {[gdb_compile "${binfile}.o ${binfile}-p.o" ${binfile} executable {debug}] != ""} {
> +    if {[gdb_compile "${binfile}.o ${binfile}-p.o" ${binfile} executable \
> +	     {debug quiet}] != ""} {
>           return -1
>       }
>   }
>
Then again, it wouldn't hurt to prevent noisy output. So, this LGTM.