Re: tip: New "Link:" tag to replace "LKML-Reference:"

[H. Peter Anvin]

On 04/07/2011 07:54 AM, Peter Zijlstra wrote:
> 
> Can't we use a prepare-commit-msg or commit-msg hook to re-write the
> commit message to include a Link tag when one is missing, using
> something like:
> 
> Link: http://lkml.kernel.org/r/tip-$SHA@xxxxxxxxxxxxxx
> 
> And have the post-commit hook that sends the messages recognize this
> form and send the message using the correct msgid.
> 
> That way the commit includes a correct and clickable link, doesn't
> require magic knowledge and doesn't need the committer to do extra work.
> 

To summarize the problem I hinted at earlier:

1. The link that ends up in the commit message needs to be unique, or
   it is not going to be a working future-clickable link.
2. The link that ends up in the commit message CANNOT use the SHA1 of
   the commit itself, since the commit SHA1 is derived from the entire
   commit object including the message;
3. As a result, the only option is for the robot to change the
   Message-ID to something other than the SHA1 of the commit.  It is a
   bit unfortunate, but probably not disastrous.

	-hpa

-- 
H. Peter Anvin, Intel Open Source Technology Center
I work for Intel.  I don't speak on their behalf.

--
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/