Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Simon Marchi <simark@simark.ca>
To: Andrew Burgess <aburgess@redhat.com>, gdb-patches@sourceware.org
Subject: Re: [PATCH] gdb/testsuite: fix failure in gdb.server/fetch-exec-and-args.exp
Date: Thu, 15 Jan 2026 16:13:14 -0500	[thread overview]
Message-ID: <f059e227-7d65-4302-8488-4b4a60e666c7@simark.ca> (raw)
In-Reply-To: <19b4a897d945398861334de10c7c2c3367ecec3a.1768420854.git.aburgess@redhat.com>

On 1/14/26 3:01 PM, Andrew Burgess wrote:
> Bug PR gdb/33792 reported a gdb.server/fetch-exec-and-args.exp FAIL
> when using the native-gdbserver board:
> 
>   FAIL: gdb.server/fetch-exec-and-args.exp: packet=on: set_remote_exec=false: test_server_with_no_exec: show remote exec-file
> 
> The actual test output looks like this:
> 
>   (gdb) show remote exec-file
>   The remote exec-file is unset, using automatic value "/tmp/build/gdb/testsuite/outputs/gdb.server/fetch-exec-and-args/fetch-exec-and-args".
>   (gdb) FAIL: gdb.server/fetch-exec-and-args.exp: packet=on: set_remote_exec=false: test_server_with_no_exec: show remote exec-file
> 
> This test actually fails with native-gdbsever and
> native-extended-gdbserver boards.  The problem is that these boards
> clear the sysroot.
> 
> This exact test has the following conditions:
> 
>   + The qExecAndArgs is in use (see 'packet=on').
> 
>   + We're not explicitly doing 'set remote exec-file ...' (see
>     'set_remote_exec=false').
> 
>   + The test starts gdbserver without an executable (see
>     'test_server_with_no_exec').
> 
>   + And because of the native-gdbsever board, the sysroot is "".
> 
> What this means is that GDB knows that gdbserver doesn't have an
> executable thanks to qExecAndArgs, the user hasn't set an executable
> for GDB to use when starting a new inferior, but GDB does know that
> GDB and gdbserver can see the same filesystem due to the sysroot
> setting.  GDB will then automatically use the current executable as
> the remote executable name.  The test script doesn't expect this case,
> and so the test fails.
> 
> Fix this by adjusting the script to expect the 'using automatic value
> ...' text when appropriate.
> 
> I also extended the test_server_with_no_exec proc to take a new flag
> 'clear_sysroot', we now run the test with the sysroot set to 'target:'
> and with the sysroot set to "", even when using the 'unix' board.
> 
> Additionally, I ran the test through check-all-boards and found one
> additional failure, when using --host_board=local-remote-host-native
> and --target_board=local-remote-host-native.  In this case GDB copies
> the executable to the remote host, which changes its filename.  When
> the filename appears in the 'using automatic value ...' text, I was
> expecting the filename assuming a local host.
> 
> I could fix this, but it doesn't seem worth the extra complexity for
> this one test, so I've just set the test to be skipped for that one
> configuration.
> 
> Now, when using check-all-boards, I'm seeing no failures.
> 
> Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=33792

It appears to fix the failure on my side.  Thanks for the quick fix and
for improving the test.

Approved-By: Simon Marchi <simon.marchi@efficios.com>

Simon

  reply	other threads:[~2026-01-15 21:13 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-10-06 15:07 [PATCH] gdb: use current executable for 'remote exec-file' in some cases Andrew Burgess
2025-10-06 15:43 ` Eli Zaretskii
2025-10-09 16:17 ` Simon Marchi
2025-10-11 13:34 ` [PATCH 0/2] Auto setting of 'remote exec-file' Andrew Burgess
2025-10-11 13:34   ` [PATCH 1/2] gdb/remote: replace use of std::pair with an actual struct Andrew Burgess
2025-10-11 13:46     ` Simon Marchi
2025-10-12  8:57       ` Andrew Burgess
2025-10-12 12:05         ` Simon Marchi
2025-10-12 13:13           ` Andrew Burgess
2025-11-19 10:32     ` Andrew Burgess
2025-10-11 13:34   ` [PATCH 2/2] gdb: use current executable for 'remote exec-file' in some cases Andrew Burgess
2025-10-11 14:45     ` Eli Zaretskii
2025-11-07 20:46     ` Simon Marchi
2025-11-11 15:59       ` Andrew Burgess
2026-01-14  1:17         ` Simon Marchi
2026-01-14 16:48           ` Andrew Burgess
2026-01-14 20:01           ` [PATCH] gdb/testsuite: fix failure in gdb.server/fetch-exec-and-args.exp Andrew Burgess
2026-01-15 21:13             ` Simon Marchi [this message]
2026-01-22 15:31               ` Andrew Burgess
2025-11-19 10:32     ` [PATCH 2/2] gdb: use current executable for 'remote exec-file' in some cases Andrew Burgess

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=f059e227-7d65-4302-8488-4b4a60e666c7@simark.ca \
    --to=simark@simark.ca \
    --cc=aburgess@redhat.com \
    --cc=gdb-patches@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox