Re: rsync 5.x breakage

2002-06-18 Thread David O'Brien
On Tue, Jun 18, 2002 at 10:21:32AM +0200, Oliver Braun wrote: > BTW, why have you added BUILD_DEPENDS= perl:${PORTSDIR}/lang/perl5 to > net/rsync? Only for the patches? Correct, for the use of "perl -pi -e". To Unsubscribe: send mail to [EMAIL PROTECTED] with "unsubscribe freebsd-current" in th

Re: rsync 5.x breakage

2002-06-18 Thread Oliver Braun
* David O'Brien <[EMAIL PROTECTED]> [2002-06-18 05:32]: > On Sun, Jun 16, 2002 at 02:30:29PM +0200, Oliver Braun wrote: > > The problem is that sed(1) on -current fails with "sed -i.bak file", if > > file.bak already exists, but perl does not. > Please file a PR about this. done > > Since net/r

Re: rsync 5.x breakage

2002-06-16 Thread Kris Kennaway
On Sun, Jun 16, 2002 at 02:30:29PM +0200, Oliver Braun wrote: > ==> ${REINPLACE_CMD} has different semantics on -current and -stable > > Maintainer only using -stable won't be able to get that error and fix it > _before_ submitting. Yeah, I really don't like this. At the very minimum there nee

Re: rsync 5.x breakage

2002-06-16 Thread Oliver Braun
* Doug Barton <[EMAIL PROTECTED]> [2002-06-16 20:20]: > Oliver Braun wrote: > > The problem is that sed(1) on -current fails with "sed -i.bak file", if > > file.bak already exists, but perl does not. Since net/rsync/Makefile > > uses 3 ${REINPLACE_CMD}s on one file (rsync.h), I have removed the >

Re: rsync 5.x breakage

2002-06-16 Thread Doug Barton
Oliver Braun wrote: > The problem is that sed(1) on -current fails with "sed -i.bak file", if > file.bak already exists, but perl does not. Since net/rsync/Makefile > uses 3 ${REINPLACE_CMD}s on one file (rsync.h), I have removed the > backup file with ${RM} file.bak between the calls. > > ==> $

Re: rsync 5.x breakage

2002-06-16 Thread Oliver Braun
[CC ports@ and current@] * Kris Kennaway <[EMAIL PROTECTED]> [2002-06-16 13:02]: > Looks like the ${REINPLACE_CMD} patch didn't work: > http://bento.freebsd.org/errorlogs/5-latest/rsync-2.5.5_1.log Fixed in PR ports/39365. But I am not very happy with that fix. The problem is that sed(1) on -