From: Junio C Hamano <junkio@cox.net>
To: Andy Parkins <andyparkins@gmail.com>
Cc: git@vger.kernel.org
Subject: Re: [PATCH] Allow the ident attribute to include a length specifier
Date: Thu, 17 May 2007 12:55:25 -0700 [thread overview]
Message-ID: <7vr6pf8bte.fsf@assigned-by-dhcp.cox.net> (raw)
In-Reply-To: <200705171453.32093.andyparkins@gmail.com> (Andy Parkins's message of "Thu, 17 May 2007 14:53:29 +0100")
Andy Parkins <andyparkins@gmail.com> writes:
>> There is no change to existing behaviour. With no parameter, the
>> expansion is all 40 hex digits.
>
> Were there any thoughts on this patch? I think it might have got drowned in
> the noise I made about $ident$ -> $Id$.
I was kind of hoping that I can avoid the decision and also I
can avoid having to remember it, by waiting long enough and
either seeing nobody on the list bring it up again, in which
case we can safely forget it, or somebody asks "what happened to
this, I want it too!", in which case the original submitter
would resubmit after the dust settles after 1.5.2, possibly with
a necessary rebase and clean-up. I'll see which one is the case
in the coming few weeks ;-).
prev parent reply other threads:[~2007-05-17 19:55 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-05-14 13:05 [PATCH] Allow the ident attribute to include a length specifier Andy Parkins
2007-05-14 13:29 ` Andy Parkins
2007-05-14 13:37 ` [PATCH] Use $Id$ as the ident attribute keyword rather than $ident$ to be consistent with other VCSs Andy Parkins
2007-05-14 14:19 ` Johannes Sixt
2007-05-14 14:39 ` Andy Parkins
2007-05-14 14:54 ` Junio C Hamano
2007-05-14 22:24 ` Andy Parkins
2007-05-15 0:17 ` Junio C Hamano
2007-05-15 2:03 ` Junio C Hamano
2007-05-15 8:14 ` Andy Parkins
2007-05-15 8:54 ` Junio C Hamano
2007-05-15 10:34 ` Andy Parkins
2007-05-17 13:53 ` [PATCH] Allow the ident attribute to include a length specifier Andy Parkins
2007-05-17 19:55 ` Junio C Hamano [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=7vr6pf8bte.fsf@assigned-by-dhcp.cox.net \
--to=junkio@cox.net \
--cc=andyparkins@gmail.com \
--cc=git@vger.kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.