This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] Make sure GDB uses a valid shell when starting the inferior and to perform the "shell" command
- From: Eli Zaretskii <eliz at gnu dot org>
- To: Andreas Schwab <schwab at linux-m68k dot org>
- Cc: sergiodj at redhat dot com, simon dot marchi at ericsson dot com, gdb-patches at sourceware dot org
- Date: Sat, 25 Jul 2015 11:08:53 +0300
- Subject: Re: [PATCH] Make sure GDB uses a valid shell when starting the inferior and to perform the "shell" command
- Authentication-results: sourceware.org; auth=none
- References: <1437761993-18758-1-git-send-email-sergiodj at redhat dot com> <55B2850D dot 6030306 at ericsson dot com> <87k2tp5q3g dot fsf at redhat dot com> <838ua52wmp dot fsf at gnu dot org> <87oaj148e7 dot fsf at igel dot home> <834mks3e5r dot fsf at gnu dot org> <m2twsszn8c dot fsf at linux-m68k dot org>
- Reply-to: Eli Zaretskii <eliz at gnu dot org>
> From: Andreas Schwab <schwab@linux-m68k.org>
> Cc: sergiodj@redhat.com, simon.marchi@ericsson.com, gdb-patches@sourceware.org
> the set of non-shells is finite, because they will match a line in
> /etc/shells.
I'm okay with looking up non-shells in /etc/shells, but that's not
what the proposed patch does. It just compares the shell's file name
with a list of known "invalid" shells. It doesn't even mention
/etc/shells, in fact, not even in the documentation parts of the
patch.