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

From: Linus Torvalds <torvalds@linux-foundation.org>
Date: Dec 1, 2019, 01:20 UTC
Message-ID: <CAHk-=wj8tNhu76yxShwOfwVKk=qWznSFkAKyQfu6adcV8JzJkQ@mail.gmail.com>
In-Reply-To: <20191130180301.5c39d8a4@lwn.net>

On Sat, Nov 30, 2019 at 5:03 PM Jonathan Corbet <corbet@lwn.net> 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