On 11/07/2014 01:37 PM, Steve Holme wrote:
> I'm sure Daniel will review this

Great.

>> Revert commit curl-7_39_0~160 (Cmake: Avoid cycle directory dependencies,
>> 2014-08-22) and add a comment explaining the purpose of the original code.
>
> I would recommend referencing the original commit in commit comments via a
> shortened version of the SHA-1 commit ID (I find that 9 or 10 characters
> are sufficient) so that tools can hyperlink to it as well as making a human
> search easier.

I'm happy to use whatever style this community prefers. FYI, git.git
switched from using the short sha1 to $(git describe --contains $sha1),
and I was given exactly the opposite advice when sending patches there
with short sha1s in the commit messages. IIUC the reasons include:

* References relative to tags tell the human about when the change
  was made relative to the project history.

* They would also survive a future rewrite when Git switches to something
  other than sha1.

The viewers can always be improved to recognize such references and make
them into links.

-Brad

-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette: http://curl.haxx.se/mail/etiquette.html
Received on 2014-11-07