[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.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]