Date: Fri, 15 Dec 2000 22:46:33 +0200
From: "Eli Zaretskii" <eliz AT is DOT elta DOT co DOT il>
Sender: halo1 AT zahav DOT net DOT il
To: "Tim Van Holder" <tim DOT van DOT holder AT pandora DOT be>
Message-Id: <1225-Fri15Dec2000224633+0200-eliz@is.elta.co.il>
X-Mailer: Emacs 20.6 (via feedmail 8.3.emacs20_6 I) and Blat ver 1.8.6
CC: djgpp-workers AT delorie DOT com
In-reply-to: <NBBBIOJKJBNCHJBEKHLOOEFHCCAA.tim.van.holder@pandora.be>
Subject: Re: MS-DOS path support in CVS
References:  <NBBBIOJKJBNCHJBEKHLOOEFHCCAA DOT tim DOT van DOT holder AT pandora DOT be>
Reply-To: djgpp-workers AT delorie DOT com
Errors-To: nobody AT delorie DOT com
X-Mailing-List: djgpp-workers AT delorie DOT com
X-Unsubscribes-To: listserv AT delorie DOT com
Precedence: bulk

> From: "Tim Van Holder" <tim DOT van DOT holder AT pandora DOT be>
> Date: Fri, 15 Dec 2000 17:37:52 +0100
> 
> Of these, I would prefer the first to be the default, with the third
> a viable alternative. I consider the second option to be too broken,
> but am willing to include it if necessary.

I agree.  I also think there should be a way for the user to override
the default, for each operation.

In addition, we could make the I/O mode be sensitive to the repository
location: a remote repository is probably almost always a Unix box.

> > A "cvs diff" needs to deal with CRLF/NL conversions automatically for
> > text files.
> This is a difficult issue. You'd want the result of the diff to work
> on all platforms. As such it should preserve EOL conventions, and be
> written in binary mode. Only mode 1 handles this cleanly, as all files
> use binary I/O anyway.

Things are not that bad: latest versions of GNU Patch have a --binary
option which strips the CRs from the diffs, before applying them.
Using --binary, I can apply on Unix diffs generated on a DOS machine,
and it all works.

CVS can use the same code in its internal Patch operation.