Date: Sat, 10 Jan 1998 19:50:07 +0100 From: J Wunsch <j@uriah.heep.sax.de> To: =?iso-8859-1?Q?=E1=CE=C4=D2=C5=CA_=FE=C5=D2=CE=CF=D7?= <ache@nagual.pp.ru> 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: <19980110195007.31038@uriah.heep.sax.de> In-Reply-To: =?iso-8859-1?Q?=3CPine=2EBSF=2E3=2E96=2E980110190329=2E27770A-100000=40l?= =?iso-8859-1?Q?sd=2Erelcom=2Eeu=2Enet=3E=3B_from_=E1=CE=C4=D2=C5=CA_=FE?= =?iso-8859-1?Q?=C5=D2=CE=CF=D7_on_Sat=2C_Jan_10=2C_1998_at_07=3A11=3A31P?= =?iso-8859-1?Q?M_%2B0300?= References: <199801071350.AAA01279@word.smith.net.au> <Pine.BSF.3.96.980110190329.27770A-100000@lsd.relcom.eu.net>
next in thread | previous in thread | raw e-mail | index | archive | help
As Андрей Чернов wrote:
> Since Index: is dead for context diffs, here is a patch to correct ***/---
> pathnames generated by CVS (now using -L diff option like in rcsdiff).
>
> Since I not run CVS and don't know how to do its minimal setup, please
> test and check is it really fix the problem.
(Well, you could always test it on freefall. ;)
> + run_setup ("%s %s", DIFF, opts);
> + run_arg (set_diff_label (DEVNULL));
> + run_arg (set_diff_label (finfo->fullname));
> + run_arg (DEVNULL);
> + run_arg (fn);
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?
Also, i'm not sure, but IMHO the Index: lines should be dropped then.
They are obsolete at best, and confusing at worst.
--
cheers, J"org
joerg_wunsch@uriah.heep.sax.de -- http://www.sax.de/~joerg/ -- NIC: JW11-RIPE
Never trust an operating system you don't have sources for. ;-)
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?19980110195007.31038>