git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] Use $Id$ as the ident attribute keyword rather than $ident$ to be consistent with other VCSs

From
Junio C Hamano <junkio@cox.net>
Date
May 14, 2007, 14:54 UTC
Message-ID
<7v646vo3pn.fsf@assigned-by-dhcp.cox.net>
In-Reply-To
<46486FE6.16A82D9A@eudaptics.com>
Johannes Sixt <J.Sixt@eudaptics.com> writes:
Show 8 quoted lines
> Andy Parkins wrote:
>> I think that bzr uses $Id$; Mercurial has examples hooks for $Id$;
>> monotone has $Id$ on its wishlist.  I can't think of a good reason not
>> to stick with the de-facto standard and call ours $Id$ instead of
>> $ident$.
>
> I very much agree. I wondered why it was named $ident$ in the first
> place. Now that I'm not alone, I thought I'd throw my 2 cents in...

My take when I did the $ident$ stuff on this issue was quite the opposite. CVS "$Id$" means quite a different thing (pathname, per-file revision number, date, and status) and it would not be right to overwrite it with $ident$ which does not record any of those "context sensitive" information.

I did not think other systems making that mistake was not an excuse for us to do so, but on the other hand, if the users of those other systems are happy to lose the information from CVS then perhaps the users do want $Id$.

Obviously I do not care much about this feature and I have not look at Andy's patch too deeply yet, but in any case I think the inverse conversion needs to be modified to match it, if it hasn't been done so.

Previous: Andy ParkinsNext: Andy Parkins
Message 6 of 14 in “Allow the ident attribute to include a length specifier”
  1. Allow the ident attribute to include a length specifierAndy Parkins, May 14, 2007
  2. Andy ParkinsMay 14, 2007
  3. Use $Id$ as the ident attribute keyword rather than $ident$ to be consistent with other VCSsAndy Parkins, May 14, 2007
  4. Johannes SixtMay 14, 2007
  5. Andy ParkinsMay 14, 2007
  6. Junio C HamanoMay 14, 2007
  7. Andy ParkinsMay 14, 2007
  8. Junio C HamanoMay 15, 2007
  9. Junio C HamanoMay 15, 2007
  10. Andy ParkinsMay 15, 2007
  11. Junio C HamanoMay 15, 2007
  12. Andy ParkinsMay 15, 2007
  13. Andy ParkinsMay 17, 2007
  14. Junio C HamanoMay 17, 2007

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.