summaryrefslogtreecommitdiff
path: root/HACKING
diff options
context:
space:
mode:
authorGreg Troxel <gdt@ir.bbn.com>2010-11-03 07:20:38 -0400
committerGreg Troxel <gdt@ir.bbn.com>2010-11-03 07:20:38 -0400
commitd7a9779742748f0658be22664349669438e69af1 (patch)
treedbd2886e08dfd83312ed46028d8cb5d6f579d55f /HACKING
parentf3bd05011ceb5173c2b01d2d51616fabe4860f57 (diff)
infrastructure: Clarify commit message format.
Note 54/72 line length rules, and that this is intended to play nice with "git log --oneline".
Diffstat (limited to 'HACKING')
-rw-r--r--HACKING8
1 files changed, 6 insertions, 2 deletions
diff --git a/HACKING b/HACKING
index 5a207274..ca3ad52e 100644
--- 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