Date: Fri, 12 Nov 2004 20:50:48 +0200 From: Ville =?ISO-8859-1?Q?Skytt=E4?= <scop@FreeBSD.org> To: freebsd-cvsweb@freebsd.org Subject: Re: cvsweb patch (supports now annotate for binaries) Message-ID: <1100285448.8353.18.camel@bobcat.mine.nu> In-Reply-To: <20041112092735.GA28354@terreactive.ch> References: <20041112092735.GA28354@terreactive.ch>
next in thread | previous in thread | raw e-mail | index | archive | help
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.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1100285448.8353.18.camel>