This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH v4 5/5] Implement proper "startup-with-shell" support on gdbserver
- From: Eli Zaretskii <eliz at gnu dot org>
- To: Sergio Durigan Junior <sergiodj at redhat dot com>
- Cc: gdb-patches at sourceware dot org, palves at redhat dot com, lgustavo at codesourcery dot com
- Date: Wed, 08 Mar 2017 17:48:37 +0200
- Subject: Re: [PATCH v4 5/5] Implement proper "startup-with-shell" support on gdbserver
- Authentication-results: sourceware.org; auth=none
- References: <20170208032257.15443-1-sergiodj@redhat.com> <20170308052931.25398-1-sergiodj@redhat.com> <20170308052931.25398-6-sergiodj@redhat.com>
- Reply-to: Eli Zaretskii <eliz at gnu dot org>
> From: Sergio Durigan Junior <sergiodj@redhat.com>
> Cc: Pedro Alves <palves@redhat.com>, Luis Machado <lgustavo@codesourcery.com>, Sergio Durigan Junior <sergiodj@redhat.com>
> Date: Wed, 8 Mar 2017 00:29:31 -0500
>
> This patch implements the proper support for the "startup-with-shell"
> feature on gdbserver. A new packet is added, QStartupWithShell, and
> it is sent on initialization. If the host sends a
> "QStartupWithShell:1", it means the inferior shall be started using a
> shell. If the host sends a "QStartupWithShell:0", it means the
> inferior shall be started without using a shell. Any other value is
> considered an error.
>
> There is no way to remotely set the shell that will be used by the
> target to start the inferior. In order to do that, the user must
> start gdbserver while providing a shell via the $SHELL environment
> variable. The same is true for the host side.
>
> The "set startup-with-shell" setting from the host side is used to
> decide whether to start the remote inferior using a shell. This same
> setting is also used to decide whether to use a shell to start the
> host inferior; this means that it is not really possible to start the
> inferior using different mechanisms on target and host.
>
> A documentation patch is included, along with a new testcase for the
> feature.
The NEWS part is OK.
Thanks.