bug-rcs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

PATCH rcs 5.7 - support for CVS/CVSNT commitid phrase extension


From: Mark D. Baushke
Subject: PATCH rcs 5.7 - support for CVS/CVSNT commitid phrase extension
Date: Sun, 12 Feb 2006 22:15:25 -0800

Package: rcs
Version: 5.7
Severity: wishlist
Tags: patch

Recent versions of CVS and CVSNT add a new phrase `commitid ...;' to
attach a unique commitid to the delta section of all files committed at
the same time. This allows users to better be able to identify the set
of files that were changed as a unit.

Production versions of RCS 5.7 do not know what to do with this
information and generate a message to STDERR about it such as the
following:

% rlog CVSROOT/config,v > /dev/null
rlog: CVSROOT/config,v: warning: Unknown phrases like `commitid ...;' are 
present.
%

The standard workaround is for users to use the -q switch to avoid
getting these messages on STDERR.

% rlog -q CVSROOT/config,v > /dev/null
%

However, some users may not desire to use the -q switch and instead
might even want to see the commitid information as a part of the rlog
output.

A patch against rcs-5.7 to teach it how to handle this new phrase and to
use it for its own commits is included in the CVS source repository
contrib directory and is presented here for your consideration.

The URL is

http://cvs.savannah.nongnu.org/viewcvs/*checkout*/ccvs/contrib/rcs-5.7-commitid.patch?rev=HEAD&root=cvs

The attached patch is a snapshot of version 1.9 of this file.

I hope you find the patch to be useful enough to incorporate into a future
revision of rcs.

If you would rather that the RCS ci command not actively add the new
phrase commitid to deltas, then I can resubmit this patch with the
changes to src/ci.c and src/rcsbase.h to remove the changes which
generate a unique commitid for a new checkin.

Please advise if you think there will ever be an RCS 5.8 release which
would pick up this patch. I suspect that there are some other patches
(security-related in nature) which might be submitted if you open the
door to a new release.

        Thank you,
        -- Mark

Attachment: rcs-5.7-commitid.patch
Description: http://cvs.savannah.nongnu.org/viewcvs/*checkout*/ccvs/contrib/rcs-5.7-commitid.patch?rev=1.9&root=cvs

Attachment: pgpFpy1Stqca1.pgp
Description: PGP signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]