On 10/07/2018 08:24 PM, Simon Marchi wrote:

> Thanks for this, fixes to make the testsuite run more cleanly is always 
> appreciated.
> 
> I just did a quick smoke check, and noticed some errors when running on 
> x86:
> 
>    Running 
> /home/emaisin/src/binutils-gdb/gdb/testsuite/gdb.trace/signal.exp ...
>    ERROR: no fileid for elxacz23q12
> 
> (elxacz23q12 is my hostname)
> 
> This looks like a check done a bit too early.  Can you make sure that 
> these tests:
> 
> $ make check TESTS="gdb.trace/*.exp"
> $ make check TESTS="gdb.trace/*.exp" 
> RUNTESTFLAGS="--target_board=native-gdbserver"
> $ make check TESTS="gdb.trace/*.exp" 
> RUNTESTFLAGS="--target_board=native-extended-gdbserver"
> 
> Don't have significant differences before/after your patch (on 
> x86/Linux)?  If you don't see any difference, maybe the problem is on my 
> side, then I'll look into it.

Thanks, I tried that and made a couple tweaks to the patch to fix those 
errors.  Is this version OK?

-Sandra