Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Stan Shebs <shebs@apple.com>
To: Kevin Buettner <kevinb@cygnus.com>
Cc: Andrew Cagney <ac131313@cygnus.com>,
	Michael Elizabeth Chastain <chastain@cygnus.com>,
	gdb-patches@sources.redhat.com
Subject: Re: ChangeLog fixes
Date: Mon, 05 Mar 2001 15:44:00 -0000	[thread overview]
Message-ID: <3AA424A9.FAE161DB@apple.com> (raw)
In-Reply-To: <1010305222742.ZM5518@ocotillo.lan>

Kevin Buettner wrote:
> 
> For real historians, I would imagine that it's incredibly tricky to
> figure out whether such corrections are a rewriting of history in the
> pejorative sense or are truly accurate corrections.  Our historical
> texts are the ChangeLog entries, and for these, we (or some of us
> anyway) have the luxury of being able to check the veracity of the
> ChangeLog entries by means of the CVS repositories.

I have a little familiarity with what real historians do, and this
process is called "textual editing".  For instance, most of the Greek
and Roman literature exists today only as manuscript copies made by monks,
the oldest of those still a thousand years separated from the originals.
In the worst cases, a monk might not know the language of the text he
was copying and make serious nonsensical mistakes, then the next
monk would try to fix them while making his copy, and so today we end
up with a dozen manuscripts, all inconsistent with each other.  Even
worse, in some cases the mistakes completely change the sense of a
passage, and there are learned papers debating which interpretation
is more plausible, based on context ("internal evidence") or other
documents, archaeology, etc ("external evidence").

> Now to specifics...  The bulk of Michael's ChangeLog fixes concern the
> use of the ``*'' wildcard.  He left the original author's comment the
> same; all he did was expand the ``*'' to list all of those files which
> were actually affected.  As I understand it, he verified the accuracy
> of his rewrites by checking against CVS.

This seems like a reasonable thing, but beware of ChangeLogs that
predate CVS.  Since there are many secret files mentioned in the
internal RH repositories, of necessity he must have done that
additional edit of wildcard expansions as well.  So I think the
change is fine, but as the GNU coding standard mentions, the ChangeLog
is still just an approximation to what actually happened.

Because of the likelihood of compounding errors by rewriting old
ChangeLogs, in general I would recommend resisting the temptation
to try to improve them.  (I've had to hold myself back many times.)

Stan


      parent reply	other threads:[~2001-03-05 15:44 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-03-04 20:38 Michael Elizabeth Chastain
2001-03-05 12:34 ` Andrew Cagney
     [not found]   ` <1010305222742.ZM5518@ocotillo.lan>
2001-03-05 15:43     ` Andrew Cagney
2001-03-05 15:44     ` Stan Shebs [this message]

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=3AA424A9.FAE161DB@apple.com \
    --to=shebs@apple.com \
    --cc=ac131313@cygnus.com \
    --cc=chastain@cygnus.com \
    --cc=gdb-patches@sources.redhat.com \
    --cc=kevinb@cygnus.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