[binutils-gdb] [gdb/testsuite] Disable styling in host board local-remote-host.exp

[Tom de Vries]

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=17c68d98f74b98704bf5a934dd6a0356a1e08802

commit 17c68d98f74b98704bf5a934dd6a0356a1e08802
Author: Tom de Vries <tdevries@suse.de>
Date:   Fri Oct 14 19:59:26 2022 +0200

    [gdb/testsuite] Disable styling in host board local-remote-host.exp
    
    With test-case gdb.server/stop-reply-no-thread.exp and host board
    local-remote-host.exp, I run into:
    ...
    Breakpoint 1, ^[[33mmain^[[m () at ^[[32mstop-reply-no-thread.c^[[m:21^M
    21        ^[[01;34mreturn^[[m ^[[35m0^[[m^[[31m;^[[m^M
    (gdb) FAIL: gdb.server/stop-reply-no-thread.exp: to_disable=: t_nonstop=off: \
      continue to main
    ...
    
    The problem is that styling is enabled, and that is causing a regexp mismatch.
    
    With native, styling is disabled in default_gdb_init by doing
    'setenv TERM "dumb"', but that only has effect because the build (where we
    execute runtest, and consequently the setenv) and the host (where we execute
    gdb) are the same.  For this host board however, gdb executes on a remote
    host, and the setenv has no effect.
    
    We could try to make some generic way to set TERM on the host, but for the
    purposes of this test-case it seems sufficient to just add:
    ...
    set GDBFLAGS "${GDBFLAGS} -iex \"set style enabled off\""
    ...
    so let's go with that for now.
    
    Tested on x86_64-linux.

Diff:
---
 gdb/testsuite/boards/local-remote-host.exp | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/gdb/testsuite/boards/local-remote-host.exp b/gdb/testsuite/boards/local-remote-host.exp
index e57af97a62e..8d69a568f23 100644
--- a/gdb/testsuite/boards/local-remote-host.exp
+++ b/gdb/testsuite/boards/local-remote-host.exp
@@ -30,3 +30,5 @@ proc ${board}_spawn { board cmd } {
     set board_info($board,fileid) $spawn_id
     return $spawn_id
 }
+
+set GDBFLAGS "${GDBFLAGS} -iex \"set style enabled off\""