Patch: Accept CR/LF line breaks in exclude files
Dave Dykstra
dwd at bell-labs.com
Fri Feb 22 04:15:30 EST 2002
On Thu, Feb 21, 2002 at 11:10:50AM -0600, Dave Dykstra wrote:
> On Tue, Feb 19, 2002 at 04:30:53PM -0600, Jordan Russell wrote:
> > --exclude-from and --cvs-exclude currently do not accept exclude files that
> > use CR/LF (DOS-style) line breaks. It basically behaves as if the files are
> > empty. This is rather annoying if the exclude files are generated by Windows
> > systems (e.g. if you're using --cvs-exclude and rsync'ing the contents of a
> > Samba share that includes .cvsignore files).
> >
> > Attached is a simple 1-line patch (for 2.5.2) which "fixes" this; it strips
> > off any CR character at the end of the line. Could it be committed?
> >
> > Jordan Russell
>
> Are you talking about on Cygwin or on some native unix? I'm not sure it
> makes sense to modify rsync to strip off carriage returns from every file
> it reads. On the other hand, that makes more sense than doing it for just
> exclude files. On Cygwin it should, certainly, but I think the cygwin
> option to handle that automatically has already been enabled.
Oops, I should have read through all the messages first. Rsync is not
doing that on Cygwin, and Ville Herva's patch addresses that.
- Dave Dykstra
More information about the rsync
mailing list