The git am documentation talks about “mailboxes”. I suppose these
contain messages in Internet Mail syntax. Is git am supposed to decode
MIME?
I'm asking because I have a message whose body is encoded as
quoted-printable, but git am does not parse the patch contained in it.
If git am is supposed to deal with this, I'll dig deeper and try to
figure out where things go wrong.
Thanks,
Florian
On Wed, Oct 04, 2017 at 10:44:31AM +0200, Florian Weimer wrote:
The git am documentation talks about “mailboxes”. I suppose these contain
messages in Internet Mail syntax. Is git am supposed to decode MIME?
I'm asking because I have a message whose body is encoded as
quoted-printable, but git am does not parse the patch contained in it.
If git am is supposed to deal with this, I'll dig deeper and try to figure
out where things go wrong.
Yes, it should. I just double-checked with the toy patch patch below,
and it correctly extracted the quoted-printable from the commit message
and patch, as well as in the headers.
-- >8 --
From peff@peff.net Wed Oct 4 05:21:57 2017
Date: Wed, 4 Oct 2017 05:21:55 -0400
From: =?utf-8?Q?=C3=81ccented N=C3=A1me?= <redacted>
To: Jeff King <redacted>
Subject: [PATCH] add 8bit content
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable
This commit message also has s=C3=B3me 8-bit characters which
will need qp-encoding.
Signed-off-by: =C3=81ccented N=C3=A1me <redacted>
---
file | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/file b/file
index d95f3ad..d39c7fc 100644
--- a/file
+++ b/file
@@ -1 +1 @@
-content
+8-bit c=C3=B3ntent
--=20
2.14.2.1117.g65a3442612
On 10/04/2017 11:25 AM, Jeff King wrote:
On Wed, Oct 04, 2017 at 10:44:31AM +0200, Florian Weimer wrote:
quoted
The git am documentation talks about “mailboxes”. I suppose these contain
messages in Internet Mail syntax. Is git am supposed to decode MIME?
I'm asking because I have a message whose body is encoded as
quoted-printable, but git am does not parse the patch contained in it.
If git am is supposed to deal with this, I'll dig deeper and try to figure
out where things go wrong.
Yes, it should. I just double-checked with the toy patch patch below,
and it correctly extracted the quoted-printable from the commit message
and patch, as well as in the headers.
It took me a while, but I know think the message is simply corrupted.
It's encoded with quoted-printable, and that looks correct, but it ends
with:
@@ -5137,11 +5114,13 @@ __libc_mallopt (int param_number, int value)
if (__malloc_initialized < 0)
ptmalloc_init ();
__libc_lock_lock (av->mutex);
- /* Ensure initialization/consolidation */
- malloc_consolidate (av);=20
LIBC_PROBE (memory_mallopt, 2, param_number, value);
=20
+ /* We must consolidate main arena before changing max_fast
+ (see definition of set_max_fast). */
+ malloc_consolidate (av);
+
switch (param_number)
{
case M_MXFAST:=
The “=” masks the final newline, and that doesn't decode into a valid
diff hunk. The file being patched continues after that, so it's not
even the “\ No newline at end of file” case.
So in short, there is no Git bug here, and I just failed to interpret
the “git am” diagnostics correctly:
Applying: Improve malloc initialization sequence
error: corrupt patch at line 342
Patch failed at 0001 Improve malloc initialization sequence
The copy of the patch that failed is found in: .git/rebase-apply/patch
Line 342 refers to the file in .git/rebase-apply/patch, not the original
input, and it took me a while to figure that out.
Thanks,
Florian