Tickets :: [#6786] Automatically link cvs commits to tickets

6.0.0-beta1

7/17/25

Summary	Automatically link cvs commits to tickets
Queue	Horde.org Servers
Queue Version	CVS Server
Type	Enhancement
State	Resolved
Priority	1. Low
Owners	jan (at) horde (dot) org
Requester	chuck (at) horde (dot) org
Created	05/29/2008 (6258 days ago)
Due
Updated	09/06/2009 (5793 days ago)
Assigned	06/04/2008 (6252 days ago)
Resolved	09/06/2009 (5793 days ago)
Milestone
Patch	No

09/06/2009 05:22:14 PM	Jan Schneider	Assigned to Jan Schneider Taken from Ben Klang State ⇒ Resolved

09/06/2009 05:19:36 PM	CVS Commit	Comment #11	Reply to this comment
Changes have been made in CVS for this ticket: Test for ~~bug #6786~~. New line. http://cvs.horde.org/diff.php/CVSROOT/cfg_local.pm?rt=horde&r1=1.54&r2=1.55&ty=u

06/04/2008 06:06:52 PM	Chuck Hagenbuch	Comment #10	Reply to this comment
I'd prefer not to have the line-number changes in the ticket, just the log message and links to Chora. Also, if you're in here, we should strip the leading spaces from the Chora links (and the log message if applicable), either globally, or from the Whups comment, because when posted to Whups, it results in some weird wrapping in the emails.

06/04/2008 05:41:09 PM	Jan Schneider	Comment #9	Reply to this comment
Not from me. Just be sure to strip out the "Bug:" and "Submitted by:" lines. We might also want to add some prefix for other lines that should not go into the ticket comment, like "#".

06/04/2008 05:16:46 PM	Ben Klang	Comment #8	Reply to this comment
Before I make this change is there any objection to having the CVS commit message become part of the ticket?

06/04/2008 03:05:34 PM	Chuck Hagenbuch	Comment #7	Reply to this comment
the code is in horde_post_commit, in CVSROOT/cfg_local.pm

06/04/2008 03:05:18 PM	Chuck Hagenbuch	Comment #6 Assigned to Ben Klang State ⇒ Assigned Taken from Chuck Hagenbuch	Reply to this comment
sure, just change what's passed into the addComment call...

06/04/2008 12:27:13 PM	Ben Klang	Comment #5	Reply to this comment
This is great, thanks for doing this! Is it possible to get the CVS commit message into the ticket history as well?

05/31/2008 09:00:44 PM	Chuck Hagenbuch	Comment #4 State ⇒ Resolved	Reply to this comment
Done.

05/31/2008 08:59:28 PM	CVS Commit	Comment #3	Reply to this comment
Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/CVSROOT/cfg_local.pm?r1=1.49&r2=1.50&ty=u

05/31/2008 08:55:44 PM	CVS Commit	Comment #2	Reply to this comment
http://cvs.horde.org/diff.php/CVSROOT/cfg_local.pm?r1=1.48&r2=1.49&ty=u

05/29/2008 10:45:48 PM	Chuck Hagenbuch	Comment #1 Priority ⇒ 1. Low Type ⇒ Enhancement Summary ⇒ Automatically link cvs commits to tickets Queue ⇒ Horde.org Servers Assigned to Chuck Hagenbuch Milestone ⇒ Patch ⇒ No State ⇒ Assigned	Reply to this comment
We have a svn script for this; since we're not currently moving off of CVS, though, we should enable it on our existing infrastructure.