From owner-freebsd-hackers@FreeBSD.ORG Fri Aug 1 21:33:36 2008 Return-Path: Delivered-To: freebsd-hackers@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 66CEB1065675; Fri, 1 Aug 2008 21:33:36 +0000 (UTC) (envelope-from lulf@freebsd.org) Received: from bene2.itea.ntnu.no (bene2.itea.ntnu.no [IPv6:2001:700:300:3::57]) by mx1.freebsd.org (Postfix) with ESMTP id A6E308FC1D; Fri, 1 Aug 2008 21:33:35 +0000 (UTC) (envelope-from lulf@freebsd.org) Received: from localhost (localhost [127.0.0.1]) by bene2.itea.ntnu.no (Postfix) with ESMTP id EE0B590005; Fri, 1 Aug 2008 23:33:33 +0200 (CEST) Received: from nobby.studby.ntnu.no (unknown [IPv6:2001:700:300:3::184]) by bene2.itea.ntnu.no (Postfix) with ESMTP id 4CB6990001; Fri, 1 Aug 2008 23:33:33 +0200 (CEST) Date: Fri, 1 Aug 2008 23:33:15 +0200 From: Ulf Lilleengen To: freebsd-current@freebsd.org Message-ID: <20080801213315.GA6002@nobby.studby.ntnu.no> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="YiEDa0DAkWCtVeE4" Content-Disposition: inline User-Agent: Mutt/1.5.18 (2008-05-17) X-Virus-Scanned: Debian amavisd-new at bene2.itea.ntnu.no Cc: freebsd-hackers@freebsd.org, freebsd-wip@freebsd.org Subject: [Call for testers] New cvsmode for csup patch X-BeenThere: freebsd-hackers@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Technical Discussions relating to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 01 Aug 2008 21:33:36 -0000 --YiEDa0DAkWCtVeE4 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hello again, Even in these new subversion days, I'm not giving up :) Thanks for the feed= back I got on the previous patches. It brought some issues with the patch that s= hould now be fixed. It took some time, but I've been priotitizing other projects. However, here are a list of the changes: - Someone experienced segfaults when doing an update. This was because using wrong format specifier to proto_printf. - Some parts of the protocol was missing, such as support for the Z command= , and supporting U in cvsmode. This is now implemented. - The rcsfile implementation would use huge amounts of memory when running diffs. Lately, it was easier to pinpoint where this happened due to huge = diff chains, showing clearly where the memory was lost. This is now fixed, and= csup seems to have a stable memory usage. Please report if you see otherwise. - Some simplifications and removal of code that is not needed. - Added an update of the usr.bin/csup Makefile. Thanks to kris@ for pointing this out. I hope all of you that tried the previous patch will try this one. Your pre= vious feedback was valuable in finding out where I should look to fix these probl= ems. Patches can be found for CURRENT: http://people.freebsd.org/~lulf/patches/csup/csup_2008-08-01_CURRENT.diff and 7-STABLE: http://people.freebsd.org/~lulf/patches/csup/csup_2008-08-01_RELENG_7.diff Any feedback welcome! --=20 Ulf Lilleengen --YiEDa0DAkWCtVeE4 Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.9 (FreeBSD) iEYEARECAAYFAkiTgRoACgkQCILg8nMIdCVULwCfdQfIh93S8syPhoBsLkSY8bXp Y8gAnRuskBTwvzHrr68ItIp/Ho2aQ3RZ =Xonr -----END PGP SIGNATURE----- --YiEDa0DAkWCtVeE4--