Opened 9 years ago

Closed 9 years ago

#1556 closed task (fixed)

post-commit hooks for signalling PostGIS SVN commits to trac

Reported by: strk Owned by: sac@…
Priority: normal Milestone:
Component: SysAdmin Keywords:
Cc: wildintellect, martin

Description

They'd be needed to allow for automatic cross-referencing of commits from tickets and ticket updates, see https://trac.osgeo.org/postgis/ticket/3330

I see MetaCRS requested this 7 years ago (#279) so I wonder if it wouldn't be easier to just enable SVN->trac communication for all projects at once

Change History (6)

comment:1 by strk, 9 years ago

The GIT equivalent of this ticket is currently on the trac/git sandbox instance: https://trac.osgeo.org/gittest/ticket/2

comment:2 by neteler, 9 years ago

Maybe I don't get the question but the following is there:

neteler@tracsvn:~$ grep commit-email.pl /var/www/svn/repos/postgis/hooks/post-commit
/var/www/svn/hookscripts/commit-email.pl "$REPOS" "$REV" --from svn_postgis@osgeo.org postgis-commits@postgis.refractions.net &

(likewise for many other OSGeo projects on that server)

comment:3 by strk, 9 years ago

Markus, I think a trac signaling hook is also needed for the "ticket updater", like:

trac-admin /var/www/trac/postgis changeset added "$REPOS" "$REV"

At least, I had to add that from my local tests with a git repo, and I confirm an SVN commit log in PostGIS was unable to update the referenced ticket even if the related trac feature is activated, probably due to the lack of this trac signaling.

comment:4 by strk, 9 years ago

Summary: post-commit hooks for PostGIS SVNpost-commit hooks for signalling PostGIS SVN commits to trac

comment:5 by strk, 9 years ago

btw, the comit-email.pl hook is probably to be reviewed, see #352:comment10

comment:6 by strk, 9 years ago

Resolution: fixed
Status: newclosed

Done, see https://trac.osgeo.org/postgis/ticket/3330#comment:4

Shall anyone want to same for their SVN repo, please point them to me :) [ with a ticket, please ]

Note: See TracTickets for help on using tickets.