[LLVMdev] [cfe-dev] RFC: put commit messages in *-commits subject lines?

NAKAMURA Takumi geek4civic at gmail.com
Thu Nov 15 18:12:55 PST 2012


2012/11/16 Chris Lattner <clattner at apple.com>:
>
> On Nov 15, 2012, at 2:28 PM, Chandler Carruth <chandlerc at google.com> wrote:
>
>>> [cfe-commits] r167788 - in /cfe/trunk: lib/Analysis/CFG.cpp test/Analysis/dead-stores.cpp
>>>
>>> with
>>>
>>> [cfe-commits] r167788 - Fix bad CFG construction bug when handling C++ 'try' statements.
>>>
>>> The first gives me just enough information (in my abbreviated mesage list window) to see that there was a change to Clang's Analysis library; if I look at the whole subject, I can see what files were changed (or the first few, at least). But the second actually tells me what changed, and whether or not it affects me.
>>>
>>> Having this form will also encourage people to put a useful summary in the first line of their commit, which is useful for anyone viewing history (especially git users, where the one-line summary is very common).
>>>
>>> Is there a reason we currently prefer files to log messages?
>>
>> I suspect not. I would much prefer your suggestion. If you can
>> implement the fix to our scripts (and get Tanya or Chris or Anton to
>> give you access), I would be thrilled.
>
> I'm in favor of it.  Of course, the truly awesomest thing would be something like:
>
>     [cfe-commits] r167788 - [lib/Analysis] Fix bad CFG construction bug when handling C++ 'try' statements.
>
> Even if it required some heuristics and horrible hackery in the commit script, it would be worth it.

+1, awesome. Regardless of putting [path/to/dir], I prefer a summary line.

I know a few person tends to put blank line in 1st line, to see blank
line with git --oneline, though.
It could be improved if he saw blank line from *-commits. :)

...Takumi




More information about the llvm-dev mailing list