Edgewall Software
Modify

Opened 10 years ago

Closed 8 years ago

Last modified 8 years ago

#6041 closed defect (fixed)

trac-post-commit-hook doen't work (nor error message)

Reported by: Andy andy@… Owned by: Christian Boos
Priority: low Milestone: 0.12-multirepos
Component: general Version: 0.10.4
Severity: minor Keywords: multirepos documentation
Cc:
Release Notes:
API Changes:

Description

Hello there.

I just updated my Ubuntu machine from Edgy Eft to Feisty Fawn what updated Python from 2.4 to 2.5. Now I have troubles with the hooks.

With Python2.5 the post-commit-hook subversion returns the following error: {{ MERGE von '/svn/xyz': 200 OK (https://example.com) }}

Using Python2.4, subversion doesn't return an error, but the referenced ticket is untouched.

When calling the trac-post-commit-hook manually, there is no error at all, no matter if using python2.4 or 2.5

Attachments (0)

Change History (6)

comment:1 Changed 10 years ago by anonymous

I have a similar problem. When I call the post-commit-hook manually (from teh command line) it works fine, but when a simple commit occurs nothing happens. What could be the reason for this?

comment:2 Changed 10 years ago by Christian Boos

Keywords: documentation added
Milestone: 0.12
Priority: highlow
Severity: normalminor

The svn hooks are run without any environment variable set, in contrast with running them from the command-line. That's a frequent cause of problems.

Keeping open for 0.12, so that we can eventually improve the documentation for those scripts.

comment:3 Changed 10 years ago by quimper.daniel@…

I had the same problem (doen't work (nor error message)) with 0.10 when trying to test on the command line.

Found my problem

Version 0.10 and up (maybe before that too) has deprecate all options except the -p and -r. If you use the other options, they are ignored.

So you have to test on the command line with a revision number which exist and has the correct log.

comment:4 Changed 9 years ago by Christian Boos

Milestone: 0.130.12
Owner: changed from Jonas Borgström to Christian Boos
Status: newassigned

The new hook (for now in source:sandbox/multirepos/contrib/trac-svn-hook) is better documented and warns against such issues.

comment:5 Changed 8 years ago by Christian Boos

Keywords: multirepos added
Resolution: fixed
Status: assignedclosed

Still have the trac-svn-hook.cmd to finish, but that's tracked in #6370.

comment:6 Changed 8 years ago by Christian Boos

Milestone: 0.120.12-multirepos

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Christian Boos.
The resolution will be deleted.
to The owner will be changed from Christian Boos to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .

 
Note: See TracTickets for help on using tickets.