From: Daniel Jacobowitz <drow@false.org>
To: Nick Roberts <nickrob@snap.net.nz>
Cc: gdb-patches@sources.redhat.com
Subject: Re: MI testsuite failures [PATCH]
Date: Wed, 10 Jan 2007 20:23:00 -0000 [thread overview]
Message-ID: <20070110202321.GA12861@nevyn.them.org> (raw)
In-Reply-To: <17828.37170.793413.362882@kahikatea.snap.net.nz>
On Wed, Jan 10, 2007 at 08:09:38PM +1300, Nick Roberts wrote:
> This doesn't seem a sensible way to do it as eventually we'll have massive
> duplication (multiplication?). Note that the manual claims that GDB supports
> mi1 but there don't appear to be any tests for it. I think:
>
> 1) mi-*exp should be tests that work for all mi`N' where N > 1, and could be
> run for each MI interpreter.
>
> 2) mi2-*exp tests that work for mi2 but fail for mi3, currently none.
>
> 3) mi3-*exp tests that work for mi3 but fail for mi2, currently none.
Running a single .exp file for multiple MI interpreters would be a bit
tricky, but we could probably do it. You'd have to wrap most of them
in a function and call it twice, I guess.
I don't feel strongly about this if you want to change it.
> Also if mi3 becomes the new mi, we presumably should advise frontend
> developers to specify mi2, otherwise existing frontends might get a nasty
> surprise when the new features of mi3 appear.
I haven't thought about it much. They'll appear at least one release
before we change -i=mi though.
> Hmm, what is this incompatible change anyway?
Quoting. I posted an analysis of command line quoting issues to the
gdb list around the middle of last year; I intend to make every single
MI command handle quoting the way the manual says MI ought to, but this
will change the behavior of various commands (the directory and file
related ones, mainly, but -gdb-set will probably be affected too).
--
Daniel Jacobowitz
CodeSourcery
next prev parent reply other threads:[~2007-01-10 20:23 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-01-07 23:34 MI testsuite failures Nick Roberts
2007-01-08 5:53 ` MI testsuite failures [PATCH] Nick Roberts
2007-01-08 17:11 ` Daniel Jacobowitz
2007-01-08 22:33 ` Nick Roberts
2007-01-08 22:45 ` Daniel Jacobowitz
2007-01-08 23:27 ` Nick Roberts
2007-01-09 14:26 ` Daniel Jacobowitz
2007-01-09 22:18 ` Nick Roberts
2007-01-09 22:50 ` Daniel Jacobowitz
2007-01-10 7:09 ` Nick Roberts
2007-01-10 20:23 ` Daniel Jacobowitz [this message]
2007-01-08 7:44 ` MI testsuite failures Vladimir Prus
2007-01-08 8:15 ` Nick Roberts
2007-01-08 9:40 ` Vladimir Prus
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=20070110202321.GA12861@nevyn.them.org \
--to=drow@false.org \
--cc=gdb-patches@sources.redhat.com \
--cc=nickrob@snap.net.nz \
/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