Improved Trac post commit hook for subversion with bash completion

[nslater@…]

The Trac post commit hook in the contrib directory could be improved.

I have attached a reworked version of the script it calls along with

• A bash completion script so that the script may be called interactively by the user.
• A DocBook file that provides the man/info page for the command.

[anonymous]

I am unable to add attachments to the ticket because of a database error.

[anonymous]

Attachments live here:

​http://www.cs.rpi.edu/~kantrn/trac/5230/

P.S. Thanks coderanger.

[sid]

Thanks for the files.

Generally (e.g. see #5402), updates to the commit hook scripts are not integrated since the scripts are provided as an example of better integration of Trac and SVN, and are expected to be customized to fit a team needs.

However, this might be worth applying to the default commit hook script. Any feedback from developers?

[Christian Boos]

Well, this looks like a total rewrite. I think given that we ask people to take the script and customize it to their needs, we should only take in small incremental improvements or bug fixes. Doing things that way is especially important when there's a need to propagate a bug fix (e.g. r5309).

So it would be wontfix from my side, at least for now. We may re-evaluate that later when integrating further improvements to the hooks, like #6135, #4746 etc.

[Emmanuel Blot]

Replying to cboos:

So it would be wontfix from my side, at least for now. We may re-evaluate that later when integrating further improvements to the hooks, like #6135, #4746 etc.

-1 as well.

I'll rather see these various scripts either be stored on trac-hacks.org or attached to a dedicated page on trac.edgewall.org. Hook scripts are really specific to the way the Trac+SVN pair is used in a project.

One or more examples could be provided the same way as sample-worflows are with 0.11.

BTW, is there a DocBook-to-TracWiki converter available? It could be nice to document the sample script as a Wiki page and promote the work nslater has done.

[Christian Boos]

Milestone 1.0 deleted

[Remy Blank]

The trac-post-commit-hook has been replaced by the CommitTicketUpdater component in 0.12.