Re: [PATCH] Add forgotten setup_iconv() call so that daemon-side iconv works.

2008-02-06 Thread Wayne Davison
On Tue, Jan 29, 2008 at 10:00:06PM -0500, Matt McCutchen wrote: > I put the call after the "if (write_batch < 0) dry_run = 1;" test, just > as in main. Thanks, Matt! This is now committed. ..wayne.. -- To unsubscribe or change options: https://lists.samba.org/mailman/listinfo/rsync Before posti

Re: [PATCH] Add forgotten setup_iconv() call so that daemon-side iconv works.

2008-02-02 Thread Jochen Reinwand
Hi Matt, thanks for the patch! I applied the patch to rsync-3.0.0pre8 and rsync-HEAD-20080127-2251GMT, but the new setup_iconv doesn't seem to work. Trying to connect with parameter --iconv set, the daemon writes the following to the syslog and closes the connection: iconv_open("UTF-8", "iso8

[PATCH] Add forgotten setup_iconv() call so that daemon-side iconv works.

2008-01-29 Thread Matt McCutchen
--- I put the call after the "if (write_batch < 0) dry_run = 1;" test, just as in main. I think a single-use daemon will call both this setup_iconv and the one in main, but I don't care: the one in main will have no effect because the !am_server test will fail and the !iconv_opt return will occur.