From owner-freebsd-cvsweb@FreeBSD.ORG Fri Nov 12 18:50:50 2004 Return-Path: Delivered-To: freebsd-cvsweb@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id D51C116A4CE for ; Fri, 12 Nov 2004 18:50:50 +0000 (GMT) Received: from smtp2.pp.htv.fi (smtp2.pp.htv.fi [213.243.153.35]) by mx1.FreeBSD.org (Postfix) with ESMTP id 96A7E43D39 for ; Fri, 12 Nov 2004 18:50:50 +0000 (GMT) (envelope-from scop@FreeBSD.org) Received: from [62.78.134.122] (cs78134122.pp.htv.fi [62.78.134.122]) by smtp2.pp.htv.fi (Postfix) with ESMTP id 472A8296B79 for ; Fri, 12 Nov 2004 20:50:49 +0200 (EET) From: Ville =?ISO-8859-1?Q?Skytt=E4?= To: freebsd-cvsweb@freebsd.org In-Reply-To: <20041112092735.GA28354@terreactive.ch> References: <20041112092735.GA28354@terreactive.ch> Content-Type: text/plain Organization: FreeBSD Message-Id: <1100285448.8353.18.camel@bobcat.mine.nu> Mime-Version: 1.0 X-Mailer: Ximian Evolution 1.4.6 (1.4.6-2) Date: Fri, 12 Nov 2004 20:50:48 +0200 Content-Transfer-Encoding: 7bit Subject: Re: cvsweb patch (supports now annotate for binaries) X-BeenThere: freebsd-cvsweb@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS Web maintenance mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 12 Nov 2004 18:50:50 -0000 On Fri, 2004-11-12 at 11:27, Chris Straessle wrote: > I've needed annotate support also for binaries. the reason: > I make backups from several systems with cvs, but i alwas set > the 'b' flag when i check in the files, beucase if I backup a > script file, or any files with a cvs header, i don't wan't > cvs to change the $id$ field. (is not really a change, its > just a backup) The -ko keyword expansion mode in cvs is for this purpose. > if I wanna watch these files with the fine webinterface, > cvsweb I can't get an annotate... so I've written a small > patch for it. maybe you like it too and implement this in the > next release of cvsweb. :-) I'm afraid I don't like it. Some effort has been put into cvsweb lately so that it would do the right thing wrt. binary files; this patch would essentially revert a subset of it, and in an uglyish way. Use -ko, not -kb for files you want annotations for instead. "cvs admin -ko" should do the trick for already committed files. One more minor note in case you still need the patch: > print CVS_IN "UseUnchanged\n"; > + print CVS_IN "Argument -F\n"; # to support annotate for binaries > print CVS_IN "Argument -r\n"; That shouldn't be necessary, see @annotate_options in cvsweb.conf instead.