Re: Fw: [PATCH] Documentation: networking: device drivers: Remove stray asterisks

From: Linus Torvalds
Date: Sat Nov 30 2019 - 20:20:35 EST


On Sat, Nov 30, 2019 at 5:03 PM Jonathan Corbet <corbet@xxxxxxx> wrote:
>
> Attached. The patch itself was not an attachment, but it was
> base64-encoded.

Ok, so presumably git removed the CRLF from the email, but then the
base64 encoded part had another set of CRLF.

And when I try to apply that patch (in a test-tree reset to commit
facd86390be2, so I think the patch should apply) I see the CRLF in
.git/rebase-apply/patch, but then I get

error: patch failed: Documentation/networking/device_drivers/intel/e100.rst:1
error: Documentation/networking/device_drivers/intel/e100.rst: patch
does not apply

for every hunk. I assume that's because the CR part doesn't match the old code.

But my git version is d9f6f3b619 ("The first batch post 2.24 cycle")
and some private patches.

So the problem might be limited to only some versions. I'm surprised,
though - when git applies patches, it really wants the surrounding
lines to match exactly. The extra CR at the end of the lines should
have made that test fail.

Do you use some special options for git? Like --whitespace=nowarn or
--3way or something like that?

Linus