[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: patch: do not emit trailing blanks in context header
From: |
Paul Eggert |
Subject: |
Re: patch: do not emit trailing blanks in context header |
Date: |
Tue, 07 Nov 2006 09:28:05 -0800 |
User-agent: |
Gnus/5.1008 (Gnus v5.10.8) Emacs/21.4 (gnu/linux) |
Ralf Wildenhues <address@hidden> writes:
> AFAICS, Solaris 10 patch is _not_ resilient against missing trailing blanks.
> (It does have a few other quirks with unified diffs that GNU patch does not.)
I vaguely recall having lots of problems with Solaris 'patch', even
aside from -u.
> FWIW, I really wonder whether it's not way easier to adjust the modern
> editor to not mistakenly flag trailing white space stemming from diff
> output, rather than force a file format change upon old tools.
I'm starting to wonder the same thing: we're shaking out more buggy
patch-readers than I thought we would.
> (Simply ignoring lines matching '^[\t ]*- $' would probably be
> enough for all practical purposes.)
Sorry, I don't follow this point. The lines in question contain a
single space, and that doesn't match the pattern you give.