]> git.puffer.fish Git - matthieu/frr.git/commitdiff
infrastructure: Clarify commit message format.
authorGreg Troxel <gdt@ir.bbn.com>
Wed, 3 Nov 2010 11:20:38 +0000 (07:20 -0400)
committerGreg Troxel <gdt@ir.bbn.com>
Wed, 3 Nov 2010 11:20:38 +0000 (07:20 -0400)
Note 54/72 line length rules, and that this is intended to play nice
with "git log --oneline".

HACKING

diff --git a/HACKING b/HACKING
index 5a207274d4059af0f114ee75f50fda008945dfa9..ca3ad52eb7309ff364835691844ad68476a0b2ff 100644 (file)
--- a/HACKING
+++ b/HACKING
@@ -125,8 +125,8 @@ COMMIT MESSAGES
 
 The commit message should provide:
 
-* A suitable one-line summary as the very first line of the message, in the
-  form:
+* A suitable one-line summary followed by a blank line as the very
+  first line of the message, in the form:
 
   topic: high-level, one line summary
 
@@ -143,6 +143,8 @@ to provide a short description of the general intent of the patch, in terms
 of the problem it solves and how it achieves it, to help reviewers
 understand. 
 
+The one-line summary must be limited to 54 characters, and all other
+lines to 72 characters.
 
 The reason for such itemised commit messages is to encourage the author to
 self-review every line of the patch, as well as provide reviewers an index
@@ -162,6 +164,8 @@ zebra: Enhance frob FSM to detect loss of frob
 * frob.c: (frob_change) set/clear DOWN appropriately on state change.
 * bar.c: (barinate) Check frob for DOWN state.
 
+Note that the commit message format follows git norms, so that "git
+log --oneline" will have useful output.
 
 HACKING THE BUILD SYSTEM