[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: commitinfo documentation
From: |
Derek Robert Price |
Subject: |
Re: commitinfo documentation |
Date: |
Tue, 19 Nov 2002 12:57:14 -0500 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.0.1) Gecko/20020823 Netscape/7.0 |
Fred L. Drake, Jr. wrote:
It would also be very helpful if more information was available to
scripts run via commitinfo, such as previous/new revision numbers, the
branch being affected, etc. The branch information would be
especially helpful since it can reasonably play a part in
authorization checks for commits, which is part of the stated
intention of commitinfo scripts.
This has been under discussion for a long time, and though this is asked
for frequently, no one seems to be willing to help test even though I've
already written a patch. You can find more information, including the
patch, here:
<http://ccvs.cvshome.org/issues/show_bug.cgi?id=60>
Unfortunately, the patch was written to apply to 1.11, so it might take
a little tweaking to make it work with 1.11.2. Like I said, I've been
trying to get help on this for awhile. The patch seems to work well,
but it was a pretty major overhaul and I don't want ot commit it to the
main tree until it gets tested on a few more systems. The second patch
attached to issue #60 is the same one with some extra code to make more
information available to the taginfo hook. The method should be
extensible to expanding the information available to other hooks.
If you are really interested, there's a link in the issue above to:
<http://ccvs.cvshome.org/issues/show_bug.cgi?id=25>
That issue discusses an even more complete overhaul and contains some
patches on the way to that end, but they require more work than issue #60.
Derek
--
*8^)
Email: derek@ximbiot.com
Get CVS support at <http://ximbiot.com>!
--
You want my advice? Go back to Bulgaria.
- Humphrey Bogart as Rick, _Casablanca_