Date: Sun, 11 Jan 1998 01:30:23 +0300 (MSK) From: =?KOI8-R?B?4c7E0sXKIP7F0s7P1w==?= <ache@nagual.pp.ru> To: Joerg Wunsch <joerg_wunsch@uriah.heep.sax.de> Cc: Mike Smith <mike@smith.net.au>, FreeBSD-current <current@FreeBSD.ORG>, peter@FreeBSD.ORG Subject: Re: CVS DIFF fix for review (-L added) Message-ID: <Pine.BSF.3.96.980111012612.3287B-100000@lsd.relcom.eu.net> In-Reply-To: <19980110195007.31038@uriah.heep.sax.de>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, 10 Jan 1998, J Wunsch wrote:
> I didn't look into the code yet, but i was hoping something like
>
> run_setup("%s %s -L %s -L %s %s", ...)
>
> would work?
Yes, it would be simpler. I write function keeping more complex labels
in mind, something like: <fname>\t<date> (with embedded spaces),
then find that it is not neccessary. I'll make simplified version of
the patch and send it ASAP.
> Also, i'm not sure, but IMHO the Index: lines should be dropped then.
> They are obsolete at best, and confusing at worst.
Index: lines still needed for ed-style diffs. Such case detection adds
more code.
--
Andrey A. Chernov
<ache@nietzsche.net>
http://www.nagual.pp.ru/~ache/
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.BSF.3.96.980111012612.3287B-100000>