diff options
author | Jeff King <peff@peff.net> | 2011-05-26 16:53:38 -0400 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2011-05-26 14:13:38 -0700 |
commit | 5b38456ec7bd0229bb35146ab8a905c4b63daeec (patch) | |
tree | 5551f430345d99a0e6fe5d707ee7e084ab77c305 /t/t4152-am-subjects.sh | |
parent | t: test subject handling in format-patch / am pipeline (diff) | |
download | tgif-5b38456ec7bd0229bb35146ab8a905c4b63daeec.tar.xz |
mailinfo: always clean up rfc822 header folding
Without the "-k" option, mailinfo will convert a folded
subject header like:
Subject: this is a
subject that doesn't
fit on one line
into a single line. With "-k", however, we assumed that
these newlines were significant and represented something
that the sending side would want us to preserve.
For messages created by format-patch, this assumption was
broken by a1f6baa (format-patch: wrap long header lines,
2011-02-23). For messages sent by arbitrary MUAs, this was
probably never a good assumption to make, as they may have
been folding subjects in accordance with rfc822's line
length recommendations all along.
This patch now joins folded lines with a single whitespace
character. This treats header folding purely as a syntactic
feature of the transport mechanism, not as something that
format-patch is trying to tell us about the original
subject.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t4152-am-subjects.sh')
-rwxr-xr-x | t/t4152-am-subjects.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/t/t4152-am-subjects.sh b/t/t4152-am-subjects.sh index 7222c06b80..37e5c0361c 100755 --- a/t/t4152-am-subjects.sh +++ b/t/t4152-am-subjects.sh @@ -58,7 +58,7 @@ test_expect_success 'long subject preserved (format-patch | am)' ' test_expect_success 'long subject preserved (format-patch -k | am)' ' check_subject long-k ' -test_expect_failure 'long subject preserved (format-patch -k | am -k)' ' +test_expect_success 'long subject preserved (format-patch -k | am -k)' ' check_subject long-k -k ' |