Re: bitkeeper comments

From: Jakob Oestergaard
Date: Mon Sep 01 2003 - 12:25:01 EST


On Mon, Sep 01, 2003 at 09:59:58AM -0700, Linus Torvalds wrote:
>
> On Mon, 1 Sep 2003, Christoph Hellwig wrote:
> >
> > But we're not going to retroactively censor commit messages.
>
> Actually, I do that all the time. In fact, it was I who asked Larry to add
> the "bk comment" command in to make it easy to do so.
>
> The thing is, it's hard to do after the message has already gone out into
> the public - but I fix up peoples email commentary by hand both in the
> email and often later after it has hit my BK tree too. I try to fix
> obvious typos, and just generally make the things more readable.
>
> And if the comment was wrong, then it should be fixed. Not because of any
> "censorship", but because it's misleading if the comment says it fixes
> something it doesn't fix - and that might make people overlook the _real_
> thing the change does.

There is an important difference.

If I send you a mail saying "X" and you change it to say "Y" and put "Y"
in the source tree, fine. It was a mail between us, noone except you
and me will know. If I think it's wrong, maybe I can make you submit
"X" to the source tree instead, with an explanation.

Everything that was ever publicly visible, stays publicly visible, even
with the the revised comments, thanks to the revision history.

But changing the source tree revision history retroactively, that's bad.
It defies the purpose of revision control itself.

The source tree is a public record. People will remember "this said 'Y'
I'm sure, but now it says 'X', why is that?" - and noone can answer.
History forgotten.

It's your call, but I can certainly see why some feel bad about this.

After all, we wouldn't want to edit the e-mail archives to remove all
trace of what happened, either, would we? ;)

Cheers,

--
................................................................
: jakob@xxxxxxxxxxxxx : And I see the elder races, :
:.........................: putrid forms of man :
: Jakob Østergaard : See him rise and claim the earth, :
: OZ9ABN : his downfall is at hand. :
:.........................:............{Konkhra}...............:
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/