On Friday 2007 January 26 15:49, Bill Lear wrote:
> I presume that I send the patch text to the list with the subject line
> that starts with [PATCH], some sensible topic (e.g., "document --check
> option to diff"), and then a short body of explanation and then,
> following my signature, the patch itself.
git-format-patch will make you an email of the correct form. Personally I do
this (this is the IMAP server version, but it's similar for mbox)
* make myself a branch from current master
* write patch/patches, test (yeah, right ;-))
* git-format-patch --stdout HEAD^ to check it looks good
* git-format-patch --stdout HEAD^ | git-imap-send
* Go to Drafts mailbox, open it up, add any additional comments for the
mailing list only underneath the "---" and before the diffstat. Make sure
you turn word wrap off as soon as you open the mail. Word wrapping will
ruin the patch.
* Send.
* Wait for huge feeling of disappointment because your patch is junk and
gets savaged by the git-gurus (maybe this step is just for me though).
To make this all easy and automatic I also have the following in
my .git/config:
[format]
headers = "To: git@xxxxxxxxxxxxxxx\n"
And the following in my ~/.gitconfig:
[imap]
Folder = "INBOX.Drafts"
Tunnel = "ssh -q mailhost /usr/bin/imapd ./IMAPmd 2> /dev/null"
And the following in my .git/hooks/commit-msg (make sure you set it
executable) to automatically add the "Signed-Off-By" line to your commits:
SOB=$(git var GIT_AUTHOR_IDENT | sed -n 's/^\(.*>\).*$/Signed-off-by: \1/p')
grep -qs "^$SOB" "$1" || echo "$SOB" >> "$1"
I've also enabled the .git/hooks/pre-commit hook (chmod a+x) to make git check
my patches before they get committed.
git-format-patch makes git mailing list friendly patches that match the
guidelines given in Documentation/SubmittingPatches
Hope that helps.
Andy
--
Dr Andy Parkins, M Eng (hons), MIEE
andyparkins@xxxxxxxxx
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html