Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Nick Roberts <nickrob@snap.net.nz>
Cc: gdb-patches@sources.redhat.com
Subject: Re: [PATCH] MI Doco
Date: Wed, 14 Jun 2006 18:30:00 -0000	[thread overview]
Message-ID: <u4pyn4mku.fsf@gnu.org> (raw)
In-Reply-To: <17551.10938.599796.89658@kahikatea.snap.net.nz> (message from 	Nick Roberts on Wed, 14 Jun 2006 09:14:34 +1200)

> From: Nick Roberts <nickrob@snap.net.nz>
> Date: Wed, 14 Jun 2006 09:14:34 +1200
> Cc: gdb-patches@sources.redhat.com
> 
>  > >  > Now please post only the new parts, the ones I've never seen.  Please
>  > >  > don't commit anything before that.
>  > > 
>  > > I'm not sure where this is leading because at some stage I will want to
>  > > commit the parts that you have already seen and approved.  It would be
>  > > easier for me to commit those now and submit a patch for the new parts,
>  > > so that there is no more confusion about what you are approving.  Would
>  > > that be OK?
>  > 
>  > I no longer have a clear idea what do you mean by ``those'' patches
>  > that you want to commit.  
> 
> No, but I have a clear idea about what you have approved and what you may not
> have seen.  You could choose to trust me.

If I somehow sounded like I didn't trust you, I apologize: I never
meant anything like that.  Of course I trust you; I just need to see
the parts I didn't yet review.  That is all I asked for above: that
you point me to those unseen parts, or post them separately, whatever
is more convenient.

>  > You responded with this message:
>  > 
>  >   http://sourceware.org/ml/gdb-patches/2006-06/msg00117.html
>  > 
>  > where you seemed to say that the patch included in the message was the
>  > previous one, with my comments fixed and some node reordering.  I
>  > approved the reordering, which I thought to be the only new part (in
>  > http://sourceware.org/ml/gdb-patches/2006-06/msg00121.html).  If the
>  > patch from msg00117.html included something other than what I thought
>  > it did, please tell me where to look to find the new parts.
> 
> Clearly they're in msg00117.html, more specifically everything after and
> including "@subsubheading A Bad Command".

That was the clue I was missing.  So everything after this
@subsubheading is the stuff I should review, right?  I will do that
soon.

Thanks.


  reply	other threads:[~2006-06-14 18:30 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-06-02  0:23 Nick Roberts
2006-06-02  9:05 ` Eli Zaretskii
2006-06-02 10:23   ` Nick Roberts
2006-06-02 10:44     ` Eli Zaretskii
2006-06-03 23:57       ` Nick Roberts
2006-06-04 19:32         ` Eli Zaretskii
2006-06-09 19:07         ` Eli Zaretskii
2006-06-09 19:12           ` Daniel Jacobowitz
2006-06-09 19:52             ` Eli Zaretskii
2006-06-09 19:57               ` Daniel Jacobowitz
2006-06-10  7:32                 ` Eli Zaretskii
2006-07-01 12:35                   ` [commit] Fix MI prompt in manual (was: MI Doco) Eli Zaretskii
2006-06-09 19:58               ` [PATCH] MI Doco Bob Rossi
2006-06-09 23:27           ` Nick Roberts
2006-06-10  7:51             ` Eli Zaretskii
2006-06-10 21:40               ` Nick Roberts
2006-06-11  3:31                 ` Eli Zaretskii
2006-06-11 10:47                   ` Nick Roberts
2006-06-11 19:27                     ` Eli Zaretskii
2006-06-11 21:42                       ` Nick Roberts
2006-06-12  3:36                         ` Eli Zaretskii
2006-06-12  6:19                           ` Nick Roberts
2006-06-13  3:39                             ` Eli Zaretskii
2006-06-13  4:46                               ` Nick Roberts
2006-06-13 17:26                                 ` Eli Zaretskii
2006-06-13 21:15                                   ` Nick Roberts
2006-06-14 18:30                                     ` Eli Zaretskii [this message]
2006-06-14 21:05                                       ` Nick Roberts
2006-06-15  6:56                                         ` Eli Zaretskii
2006-06-30  2:13                                           ` Nick Roberts
2006-06-30 12:09                                             ` Eli Zaretskii
2006-06-30 14:12                                               ` 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=u4pyn4mku.fsf@gnu.org \
    --to=eliz@gnu.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