<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Mar 6, 2015 at 4:15 PM, Mehdi Amini <span dir="ltr"><<a href="mailto:mehdi.amini@apple.com" target="_blank">mehdi.amini@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br>
> On Mar 6, 2015, at 4:08 PM, Robinson, Paul <<a href="mailto:Paul_Robinson@playstation.sony.com">Paul_Robinson@playstation.sony.com</a>> wrote:<br>
><br>
>> -----Original Message-----<br>
>> From: <a href="mailto:cfe-dev-bounces@cs.uiuc.edu">cfe-dev-bounces@cs.uiuc.edu</a> [mailto:<a href="mailto:cfe-dev-bounces@cs.uiuc.edu">cfe-dev-bounces@cs.uiuc.edu</a>] On<br>
>> Behalf Of Mehdi Amini<br>
>> Sent: Friday, March 06, 2015 1:49 PM<br>
>> To: Davide Italiano<br>
>> Cc: Clang Dev; LLVM Dev<br>
>> Subject: Re: [cfe-dev] [LLVMdev] Commit message policy?<br>
>><br>
>><br>
>>> On Mar 6, 2015, at 1:36 PM, Davide Italiano <<a href="mailto:davide@freebsd.org">davide@freebsd.org</a>> wrote:<br>
>>><br>
>>> On Fri, Mar 6, 2015 at 1:12 PM, Renato Golin <<a href="mailto:renato.golin@linaro.org">renato.golin@linaro.org</a>><br>
>> wrote:<br>
>>>> On 6 March 2015 at 20:59, Reid Kleckner <<a href="mailto:rnk@google.com">rnk@google.com</a>> wrote:<br>
>>>>> I think the only guideline we should have is that the first line<br>
>> should be<br>
>>>>> written as though it is an email subject, because it gets used for<br>
>> that. If<br>
>>>>> you write a long first line, then you get a long subject, and it looks<br>
>>>>> silly. If people want to embarrass themselves with strangely formatted<br>
>>>>> email, they it's on them. We don't need a specific hard or soft<br>
>> number.<br>
>>>><br>
>>>> Not many people care about the email subject already, that's why they<br>
>>>> keep using ridiculously long first lines.<br>
>>>><br>
>>>> IMO, "suggesting" to write short first lines is the same as not doing<br>
>>>> anything. Either we add a cap (say, 80 chars), or we don't do<br>
>>>> anything.<br>
>>>><br>
>>>> Chandler's other suggestion, tough, is interesting: to write up a bit<br>
>>>> about what a *good* message would be, so the people that were really<br>
>>>> interested, could do it "right" (tm).<br>
>>>><br>
>>><br>
>>> Another guideline I would like to propose for commit messages is that<br>
>>> of attaching to the commit a link to the code review, if any.<br>
>><br>
>> I believe it is documented here:<br>
>> <a href="http://llvm.org/docs/Phabricator.html#committing-a-change" target="_blank">http://llvm.org/docs/Phabricator.html#committing-a-change</a><br>
>><br>
>> Mehdi<br>
><br>
> That would be the norm for people doing reviews in Phabricator.<br>
> I think the suggestion is to do something similar for non-Phab reviews?<br>
<br>
</div></div>Oh, like a link to the email thread, for instance: <a href="http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20110314/117996.html" target="_blank">http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20110314/117996.html</a> ?<br></blockquote><div><br></div><div>I really wish our mail web ui was better.... personally, I only use it as a last resort. I'll turn first to my inbox, and then to gmane or <a href="http://marc.info">marc.info</a> or any of the other mail archives.<br><br>As such, the most useful thing is for people to reply to review threads with the revision at which the code landed. That way, I can find the commit from the review thread (just the named revisions), or the review thread for the commit (searching for the revision number turns up the review thread, and also any relevant post-commit review or reference to it). The community is actually pretty consistent about providing this and it's an awesome system IMO.<br><br></div><div>I would vastly prefer we try to encourage this existing convention more consistently, rather than trying to enforce putting links in commit messages. The existing convention provides a bidirectional link.<br></div><div><br></div><div>-- Sean Silva<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
As a Phabricator user, I haven’t thought about that, it makes sense I guess.<br>
<br>
Thanks for the clarification.<br>
<br>
—<br>
<span class="HOEnZb"><font color="#888888">Mehdi<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
</div></div></blockquote></div><br></div></div>