Revision ids in htmllog (was Re: [Bug 45677] Re: Push should create a README file in destination folder)

[Michael Ellerman]

On Sun, 2006-05-21 at 00:27 +0000, David Allouche wrote:
> I'm not sure this bug is the right place to comment on the htmllog
> behaviour, but I cannot thing of anything better.

How about the mailing list? :)

> Is it really necessary to show off the revision ids in the output of
> htmllog? Revision ids look messy, do not contain any human readable
> information that's not in the date or comitter, and are not normally
> needed to use bzr.
> 
> Making revision ids an implementation detail is a design goal of bzr,
> they should not be advertised this prominently.

Hmm. To be honest when I wrote the htmllog plugin it was a 10 minute job
and I didn't really think about it that much.

Having said that, I think it's a really cool feature to have unique ids
for each commit, and hiding from the user is stupid. Sticking them in
the htmllog means google can find them, and if someone else merges my
branch (and uses htmllog) I can find out that they've merged from me
purely with a web browser.

There's a developing practice in the kernel community that when people
commit a change which references another change, they include the git
commit id (sha hash). This is awesome because it means you can exactly
pinpoint the change they're talking about, rather than "the change to
foo.c three weeks ago ...". It's extra cool that gitk recognises git ids
in the changelog and makes them hyperlink to the referenced change.

</2c>

cheers

-- 
Michael Ellerman
IBM OzLabs

wwweb: http://michael.ellerman.id.au
phone: +61 2 6212 1183 (tie line 70 21183)

We do not inherit the earth from our ancestors,
we borrow it from our children. - S.M.A.R.T Person
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 191 bytes
Desc: This is a digitally signed message part
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20060522/259e1def/attachment.pgp