Skip site navigation (1)Skip section navigation (2)
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>