Re: a bug about format-patch of multibyte characters comment

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, Feb 13, 2011 at 10:48:58AM +0100, Johannes Sixt wrote:

> On Sonntag, 13. Februar 2011, Jeff King wrote:
> > On Sat, Feb 12, 2011 at 07:13:15PM +0900, xiaozhu wrote:
> > > Subject: [PATCH]
> > > =?UTF-8?q?=E6=97=A5=E6=9C=AC=E8=AA=9E=E3=81=8C=E5=A4=A7=E4=B8=88=E5=A4=AB
> > > =20=E6=94=B9=E8=A1=8C=E3=81=99=E3=82=8B?=
> >
> > Yeah, this is wrong. There should be a whitespace indentation in a
> > multi-line header, or the whole thing should be on one line. The newline
> > in your commit subject is apparently leaking through, and it should be
> > qp-encoded.
> 
> Isn't it wrong that format-patch (and --pretty=email) does any quoting in the 
> first place? Isn't it the task of the MUA (git-send-email) to do the quoting?

I dunno. I guess it depends on how you view the output of format-patch.
Is it an mbox containing rfc2822-valid messages? Then it ought to be
quoted.

Certainly when opening the output in an editor, it is prettier to have
it unquoted. But what do MUAs expect when opening such an mbox? mutt
seems to handle unencoded utf8 just fine, but I don't know about other
MUAs.

> For example, when I import a format-patch generated patch into a mail message, 
> I don't want to see:
> 
>  From: =?UTF-8?q?Joh=C3=A4nnes=20S=C3=BCxt?= <me@localhost>
> 
> but rather:
> 
>  From: JohÃnnes SÃxt <me@localhost>
> 
> I know this is a bit late, but this really comes as a surprise. (I've never 
> had to pay attention to this behavior in the past...)

I can see how that would be annoying if your workflow is to paste
format-patch output into an email. But it has been that way literally
for years (I just tried v1.5.0, and format-patch produces
rfc2047-encoded headers). So yes, you are a bit late. :)

-Peff
--
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


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]