This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH v2 6/6] 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, 18 Jan 2017 18:42:59 +0200
- Subject: Re: [PATCH v2 6/6] Implement proper "startup-with-shell" support on gdbserver
- Authentication-results: sourceware.org; auth=none
- References: <1482464361-4068-1-git-send-email-sergiodj@redhat.com> <20170118153605.4610-1-sergiodj@redhat.com> <20170118153605.4610-7-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>, Eli Zaretskii <eliz@gnu.org>,
> Luis Machado <lgustavo@codesourcery.com>,
> Sergio Durigan Junior <sergiodj@redhat.com>
> Date: Wed, 18 Jan 2017 10:36:05 -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.
Thanks, the documentation parts are approved.