From: Linus Torvalds <torvalds@linux-foundation.org>
To: Bjorn Helgaas <helgaas@kernel.org>,
Peter Grayson <jpgrayson@gmail.com>,
Catalin Marinas <catalin.marinas@gmail.com>
Cc: linux-pci@vger.kernel.org,
"Linux Kernel Mailing List" <linux-kernel@vger.kernel.org>,
"Lorenzo Pieralisi" <lorenzo.pieralisi@arm.com>,
"Christian König" <christian.koenig@amd.com>,
"Aaro Koskinen" <aaro.koskinen@iki.fi>,
"Andy Shevchenko" <andriy.shevchenko@linux.intel.com>,
"Boris Ostrovsky" <boris.ostrovsky@oracle.com>,
"Juergen Gross" <jgross@suse.com>,
"Alex Deucher" <alexander.deucher@amd.com>,
"David Airlie" <airlied@linux.ie>
Subject: Re: [GIT PULL] PCI fixes for v4.15
Date: Tue, 23 Jan 2018 17:04:22 -0800 [thread overview]
Message-ID: <CA+55aFwpmqZUnD=3U23c-r89XmyncoXQzK7+b8AyuQ9+soC19g@mail.gmail.com> (raw)
In-Reply-To: <20180124004603.GH5317@bhelgaas-glaptop.roam.corp.google.com>
On Tue, Jan 23, 2018 at 4:46 PM, Bjorn Helgaas <helgaas@kernel.org> wrote:
> On Tue, Jan 23, 2018 at 12:49:22PM -0800, Linus Torvalds wrote:
>>
>> I suspect you're editing your mailbox, or you're using some broken
>> mailer setup, or something (using some broken "export" function?),
>> because you're corrupting Christian's name all the time:
>
> I'm not editing my mailbox, but I am using stgit. The "From:" header
> uses encoded-word syntax for Christian's name, and I think stgit is
> not decoding that when applying the patch.
Ahh. Ok, that would explain it.
> When I save the patch email message from mutt (or gmail, for that
> matter), I see:
>
> From: "=?UTF-8?q?Christian=20K=C3=B6nig?=" ...
Yes. That's the standard locale encoding format for header values
(mainly subject and author), since the
Content-Type: text/plain; charset=utf-8
only applies to the *body* of the email.
So then the email handling tools have to handle that correctly, and as
you noticed, "git am" does so:
> Applying that patch with "git am" results in the correct
>
> Author: Christian König <ckoenig.leichtzumerken@gmail.com
But apparently stgit does not:
> Applying the identical patch with "stg import -M" results in
>
> Author: =?UTF-8?q?Christian=20K=C3=B6nig?= <ckoenig.leichtzumerken@gmail.com>
I'm adding Peter Grayson and Catalin Marinas to the participants,
since they seem to be the main stgit developers.
Presumably there is either a flag for this (which should presumably be
the default). Or if not, maybe stgit can just be fixed?
[ I go off looking at the stgit sources. ]
Oddly, stgit seems to even have a *testcase* for this:
- t/t1800-import/email-qp:
From: Inge =?utf-8?q?Str=C3=B6m?= <inge@power.com>
and that test-case harkens back to 2006. I wonder why it doesn't seem
to work for you and Christian.
Bjorn, maybe you can send Catalin an example mbox?
Linus
next prev parent reply other threads:[~2018-01-24 1:04 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-01-23 19:56 [GIT PULL] PCI fixes for v4.15 Bjorn Helgaas
2018-01-23 20:49 ` Linus Torvalds
2018-01-23 20:53 ` Linus Torvalds
2018-01-24 0:46 ` Bjorn Helgaas
2018-01-24 1:04 ` Linus Torvalds [this message]
2018-01-24 7:17 ` Andy Shevchenko
2018-01-24 16:20 ` Bjorn Helgaas
2018-01-24 16:40 ` Linus Torvalds
2018-01-24 17:27 ` Christian König
2018-01-24 20:25 ` Peter Grayson
2018-01-24 20:48 ` Linus Torvalds
2018-01-24 16:45 ` Peter Grayson
-- strict thread matches above, loose matches on Subject: below --
2018-01-12 20:22 Bjorn Helgaas
2017-12-14 23:34 Bjorn Helgaas
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='CA+55aFwpmqZUnD=3U23c-r89XmyncoXQzK7+b8AyuQ9+soC19g@mail.gmail.com' \
--to=torvalds@linux-foundation.org \
--cc=aaro.koskinen@iki.fi \
--cc=airlied@linux.ie \
--cc=alexander.deucher@amd.com \
--cc=andriy.shevchenko@linux.intel.com \
--cc=boris.ostrovsky@oracle.com \
--cc=catalin.marinas@gmail.com \
--cc=christian.koenig@amd.com \
--cc=helgaas@kernel.org \
--cc=jgross@suse.com \
--cc=jpgrayson@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=lorenzo.pieralisi@arm.com \
/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
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).