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

David Blaikie dblaikie at gmail.com
Thu Nov 15 14:30:39 PST 2012


On Thu, Nov 15, 2012 at 2:20 PM, Jordan Rose <jordan_rose at apple.com> wrote:
> Hi, everyone. I was comparing our commit messages with another commit list I'm on, and I realized (not for the first time) that our subject lines are less than useful. Why don't we have the first line of our commit messages in the subject lines? Compare:
>
> [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).

Also, having this format would mean we would be less likely to go over
the subject character limit that causes gmail to truncate subjects &
break threading...

So for both/all those reasons, I'd love this.

> Is there a reason we currently prefer files to log messages?
> Jordan
> _______________________________________________
> cfe-dev mailing list
> cfe-dev at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev




More information about the llvm-dev mailing list