From owner-cvs-all Sat Apr 20 20:24:48 2002 Delivered-To: cvs-all@freebsd.org Received: from w250.z064001178.sjc-ca.dsl.cnc.net (w250.z064001178.sjc-ca.dsl.cnc.net [64.1.178.250]) by hub.freebsd.org (Postfix) with SMTP id C383437B41C for ; Sat, 20 Apr 2002 20:24:44 -0700 (PDT) Received: (qmail 23178 invoked by uid 1000); 21 Apr 2002 03:25:05 -0000 Date: Sat, 20 Apr 2002 20:24:43 -0701 From: Jos Backus To: rsync@lists.samba.org Cc: cvs-all@FreeBSD.ORG Subject: Re: rsync breaks on FreeBSD without -O2?(fwd from grog@FreeBSD.org) PR 36998 Message-ID: <20020421032505.GA23136@lizzy.catnook.com> Reply-To: jos@catnook.com Mail-Followup-To: rsync@lists.samba.org, cvs-all@FreeBSD.ORG References: <20020420193712.O10865@va.samba.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20020420193712.O10865@va.samba.org> User-Agent: Mutt/1.3.28i Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On Sat, Apr 20, 2002 at 07:37:12PM -0700, Martin Pool wrote: > Apparently rsync breaks on FreeBSD if you turn off -O2 (which is the > default.) The breakage is apparently inside zlib. > > You'd have to think it was a compiler bug, but perhaps not. > > I haven't tried to reproduce it yet. Here's some mail from freebsd-cvs-all (<20020420171059.A593@xor.obsecurity.org>). Bruce Evans submitted this workaround which has been incorporated into the FreeBSD tree. I think we should do the same. I can do the honors :-) Jos On Sat, Apr 20, 2002 at 04:01:29PM -0700, Jos Backus wrote: > On Sat, Apr 20, 2002 at 01:22:16PM -0700, Kris Kennaway wrote: > > On Sat, Apr 20, 2002 at 01:19:47PM -0700, Anders Nordby wrote: > > > anders 2002/04/20 13:19:47 PDT > > > > > > Modified files: > > > net/rsync Makefile > > > Log: > > > Make port set -O2 in CFLAGS. Necessary for -z (compress) option to work > > > properly. > > > > Err, this sounds like the fault of the broken zlib code we had in the > > source tree, which was already fixed. > > rsync uses it's own slightly incompatible version of zlib. If you can point me > to a fix I can discuss the fix with the rsync folks and commit it. > > (If you don't hear from me it's because XO cut my DSL line - I'm in the > process of switching providers.) Here's the fix from bde which I applied to the libz in the tree: http://www.freebsd.org/cgi/cvsweb.cgi/src/lib/libz/infcodes.c.diff?r1=1.3&r2=1.4 Kris -- Jos Backus _/ _/_/_/ Santa Clara, CA _/ _/ _/ _/ _/_/_/ _/ _/ _/ _/ jos@catnook.com _/_/ _/_/_/ use Std::Disclaimer; To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message