From: Daniel Jacobowitz <drow@false.org>
To: Jim Blandy <jimb@red-bean.com>
Cc: gdb@sourceware.org
Subject: Re: Quoting, backslashes, CLI and MI
Date: Wed, 22 Feb 2006 18:11:00 -0000 [thread overview]
Message-ID: <20060222180137.GA27535@nevyn.them.org> (raw)
In-Reply-To: <8f2776cb0602220939i5189212ds8fb249747851cf72@mail.gmail.com>
On Wed, Feb 22, 2006 at 09:39:55AM -0800, Jim Blandy wrote:
> > Similar problems apply to the other listed MI commands. For instance,
> > -exec-arguments ARGS; should it take a single string which is then
> > split by buildargv into a vector, or should it take freeform text which
> > is then split into an argument vector? Well, right now it takes a
> > literal string, since it just passes the text to CLI "set args".
> > That's saved as a string and then passed to create_inferior as a
> > string, and eventually passed directly to a shell in the fork-child.c
> > case. So, as un-MI-like as it is, I think I'd have to leave this one
> > alone for now - it's just too big a can of worms!
>
> You'd have to rework a lot of code to make the quoting happen earlier.
Actually, there's two different possibilities here, and I think I
focused on the wrong one.
1. We want -exec-arguments to take MI-quoted individual arguments,
which are then passed as argv elements to the program.
2. We want -exec-arguments to take a single MI-quoted argument,
which is the value to set the argument string to, for the target
and/or shell to handle however they deem appropriate.
Doing (1) would require a lot of reworking, but more-or-less-kind-of
preserve the current semantics, for common cases - it would probably be
incompatible at the edge cases. Doing (2) would be a clear MI
interface change, but the result seems somewhat sensible. WDYT?
Then there's the question of what to do with CLI "set args". People
use this today and it is passed literally to the program, without any
interpretation of quotes or escapes - if you want that to happen you
wait for the shell to do it. I don't think that we can really change
that - we can bump the interface version on MI, but we can't really
bump it on our CLI users' fingers :-)
So CLI "set args" will need to continue being unescaped, one way or
another.
--
Daniel Jacobowitz
CodeSourcery
next prev parent reply other threads:[~2006-02-22 18:01 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-02-22 4:30 Daniel Jacobowitz
2006-02-22 4:35 ` Paul Koning
2006-02-22 19:57 ` Daniel Jacobowitz
2006-02-22 21:57 ` Paul Koning
2006-02-23 4:25 ` Mark Kettenis
2006-02-25 1:30 ` Eli Zaretskii
2006-02-22 4:40 ` Eli Zaretskii
2006-02-22 5:24 ` Daniel Jacobowitz
2006-02-22 19:30 ` Eli Zaretskii
2006-02-22 20:59 ` Daniel Jacobowitz
2006-02-22 17:39 ` Jim Blandy
2006-02-22 18:01 ` Daniel Jacobowitz
2006-02-22 18:05 ` Jim Blandy
2006-02-22 18:11 ` Daniel Jacobowitz [this message]
2006-02-22 19:24 ` Andrew STUBBS
2006-02-22 19:28 ` Daniel Jacobowitz
2006-02-22 19:50 ` Eli Zaretskii
2006-02-22 19:34 ` Eli Zaretskii
2006-02-22 19:53 ` Daniel Jacobowitz
2006-02-23 11:13 ` Eli Zaretskii
2006-02-22 19:25 ` Eli Zaretskii
2006-02-22 19:51 ` Daniel Jacobowitz
2006-02-23 4:32 ` Eli Zaretskii
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=20060222180137.GA27535@nevyn.them.org \
--to=drow@false.org \
--cc=gdb@sourceware.org \
--cc=jimb@red-bean.com \
/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