[Culling mike@xxxxxxxxxxxxxxx from CC] > As "git help imap-send" clearly says that this command is intended to be > used on output from format-patch (but it actually does not say "and > nothing else", which it should for your patch to make sense), I think the > code is allowed to assume _some_ fixed order of the header fields. It > would break people's existing abuse to use imap-send to stuff random > pieces of messages in mbox to their IMAP server, though. Yes, it would. Don't you think the bug it fixes justifies the break? Perhaps we should mention in the documentation that it accepts mailboxes where emails must have those three mandatory fields in that order? > But does format-patch _always_ start each message with these and only with > these fields in this order? Where would Message-Id:, References:, and Cc: > come when the user told format-patch to add them? Yes. From, Date, and Subject are the minimum set of fields for any general patch produced by git-format-patch, and they always appear in that order. For example, see this header: >From 20f0cf89fb9508f9e5197a3cf47a872053d4941d Mon Sep 17 00:00:00 2001 Message-Id: <20f0cf89fb9508f9e5197a3cf47a872053d4941d.1269224757.git.artagnon@xxxxxxxxx> In-Reply-To: <42ef3f8cebd4d34e24b1544036cd48f56110c2d4.1269224756.git.artagnon@xxxxxxxxx> References: <13132> <42ef3f8cebd4d34e24b1544036cd48f56110c2d4.1269224756.git.artagnon@xxxxxxxxx> From: Ramkumar Ramachandra <artagnon@xxxxxxxxx> Date: Sun, 21 Mar 2010 23:58:25 +0530 Subject: [PATCH 2/2] git-imap-send bugfix: Allow lines starting with "From " in body Cc: gitster@xxxxxxxxx Thanks, Ramkumar -- 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