Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Joel Brobecker <brobecker@adacore.com>
Cc: gdb-patches@sourceware.org
Subject: Re: [non-stop] 01/10 Add "executing" property
Date: Wed, 18 Jun 2008 00:22:00 -0000	[thread overview]
Message-ID: <uve07hjda.fsf@gnu.org> (raw)
In-Reply-To: <20080617182050.GA3595@adacore.com>

> Date: Tue, 17 Jun 2008 14:20:50 -0400
> From: Joel Brobecker <brobecker@adacore.com>
> Cc: pedro@codesourcery.com, gdb-patches@sourceware.org
> 
> > Well, "Makefile.in (foo): Update." doesn't give any idea WHAT was
> > done, it only says WHERE it was done.
> 
> Actually, it does, at least as far I what I want to know (I want to
> know that the author did an update of the Makefile rules).

That's exactly what "cvs log" tells you as well.

> I don't remember checking exactly what dependencies were added or
> removed, but I don't mind using CVS for that.

It's a pain to do this with CVS, at least in the situations I have in
mind.  Imagine that you need to find all the changes in a certain part
of a specific rule since a very old version.  Perhaps you wonder why
that part of the rule has its present shape, or what other files were
modified together with that part of the rule, or who did those
changes.  "cvs annotate" only gives you the last version, and "cvs
log" doesn't say _what_ was changed.  The only way I know of is to use
"cvs diff -rVn -rVn+1" until you have all the history figured out and
find those changes that matter.  The fact that the ChangeLog tells you
the name of the rule is of little help, because you are still facing a
lot of changes in the same rule, even if you now can skip some of the
versions.

By contrast, if the ChangeLog states what was changed in the rule, you
can find the information you want or at least dramatically narrow the
number of version diffs you need to review just by looking up the
target's name in the log file, and then reading what was modified in
the rule.

I've been there quite a few times, and I came to appreciate a few
details in the log entry.


  reply	other threads:[~2008-06-17 20:58 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-06-15 21:04 Pedro Alves
2008-06-15 22:18 ` Eli Zaretskii
2008-06-16  1:42   ` Daniel Jacobowitz
2008-06-16  3:23     ` Eli Zaretskii
2008-06-16  6:45       ` ChangeLog entries [was Re: [non-stop] 01/10 Add "executing" property] Nick Roberts
2008-06-16  7:22     ` [non-stop] 01/10 Add "executing" property Vladimir Prus
2008-06-17  0:40       ` Eli Zaretskii
2008-06-17 17:10         ` Vladimir Prus
2008-06-17 20:38           ` Eli Zaretskii
2008-06-23 17:27       ` Tom Tromey
2008-06-17 16:14   ` Joel Brobecker
2008-06-17 18:24     ` Eli Zaretskii
2008-06-17 18:32       ` Joel Brobecker
2008-06-18  0:22         ` Eli Zaretskii [this message]
2008-06-17 18:34       ` Pedro Alves
2008-06-17 20:58         ` Eli Zaretskii
2008-06-23  1:13         ` Daniel Jacobowitz
2008-06-23 11:54           ` Vladimir Prus
2008-06-23 14:41             ` Pedro Alves
2008-06-23 18:22               ` Eli Zaretskii
2008-06-23 18:25                 ` Vladimir Prus
2008-06-23 19:52                   ` Pedro Alves
2008-06-24  1:13                     ` Eli Zaretskii
2008-06-23 18:14             ` Eli Zaretskii
2008-07-02  3:29           ` Pedro Alves
2008-06-25 19:02 ` Vladimir Prus
2008-06-25 19:13   ` Pedro Alves
2008-06-25 19:22     ` Daniel Jacobowitz
2008-06-26 13:37       ` 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=uve07hjda.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=brobecker@adacore.com \
    --cc=gdb-patches@sourceware.org \
    /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