mblaze, a Maildir-focused command line mail client
 help / color / mirror / code / Atom feed
From: Leah Neukirchen <le...@vuxu.org>
To: Wolfgang Jenkner <wjen...@inode.at>
Cc: mbl...@googlegroups.com
Subject: Re: [PATCH] Fix non-POSIX sed usage
Date: Wed, 12 Jul 2017 21:12:26 +0200	[thread overview]
Message-ID: <87mv89o4f9.fsf@vuxu.org> (raw)
In-Reply-To: <ERNX3CSOLA.2VZMT4QLSH7WK@inode.at> (Wolfgang Jenkner's message of "Wed, 12 Jul 2017 20:22:53 +0200")

Wolfgang Jenkner <wjen...@inode.at> writes:

> Leah Neukirchen <le...@vuxu.org> wrote:
>> > Subject: [PATCH] Fix non-POSIX sed usage
>> 
>> Thanks, applied!
>
> Thank you.
>
> There's still a point about the 'References' header which puzzles me:
>
> mrep puts two spaces between 'References:' and the first message-id (at
> line 148 and line 17 in mcom).  So, if the message-id is sufficiently
> long (>= 78 - 11 - 2) it's actually qp-encoded, due to how print_header
> in mmime.c works.
>
> (This affects, e.g., gmail, which seems to produce message-ids of length
> 68.)
>
> Is this intended?

No, it's a bug.  References: must not use qp-encoding, but I didn't
want to add blacklisting yet...

But since 67c0a3b17c17 print_header should use up to 997 chars now, no?

Oh, I see.  The initial space breaks it.  Fixed in 73d80d8.

-- 
Leah Neukirchen  <le...@vuxu.org>  http://leah.zone

      reply	other threads:[~2017-07-12 19:12 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-07-07 23:50 Wolfgang Jenkner
2017-07-08 18:49 ` Leah Neukirchen
2017-07-12 18:22   ` Wolfgang Jenkner
2017-07-12 19:12     ` Leah Neukirchen [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87mv89o4f9.fsf@vuxu.org \
    --to="le..."@vuxu.org \
    --cc="mbl..."@googlegroups.com \
    --cc="wjen..."@inode.at \
    --subject='Re: [PATCH] Fix non-POSIX sed usage' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Code repositories for project(s) associated with this inbox:

	https://git.vuxu.org/mblaze/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).