Comments on: Mercurial trac commit hook http://madssj.com/blog/2008/10/13/mercurial-trac-commit-hook/ Various articles about programming and systems administration. Sat, 11 Jun 2011 14:50:13 +0000 hourly 1 http://wordpress.org/?v=3.3.1 By: Pascalhttp://madssj.com/blog/2008/10/13/mercurial-trac-commit-hook/#comment-34 Pascal Thu, 26 Nov 2009 18:01:25 +0000 http://swag.dk/blog/?p=56#comment-34 <p>Thanks for this plugin.</p> <p>I have try this plugin without success with Trac0.12-multirepos I have this error : Traceback (most recent call last): File "/usr/lib/python2.4/site-packages/trachook.py", line 210, in hook trac_hook.update(c, r, url) File "/usr/lib/python2.4/site-packages/trachook.py", line 88, in update self.env.href = Href(url) TypeError: 'str' object is not callable error: changegroup hook raised an exception: 'str' object is not callable</p> <p>???</p> Thanks for this plugin.

I have try this plugin without success with Trac0.12-multirepos I have this error : Traceback (most recent call last): File “/usr/lib/python2.4/site-packages/trachook.py”, line 210, in hook trac_hook.update(c, r, url) File “/usr/lib/python2.4/site-packages/trachook.py”, line 88, in update self.env.href = Href(url) TypeError: ‘str’ object is not callable error: changegroup hook raised an exception: ‘str’ object is not callable

???

]]> By: mathhttp://madssj.com/blog/2008/10/13/mercurial-trac-commit-hook/#comment-33 math Fri, 30 Oct 2009 14:35:07 +0000 http://swag.dk/blog/?p=56#comment-33 <p>Hmm, just copying the trachook.py to the place gave me an permission error, that normal users which checkin doesn't have the permission to change /srv/trac/myProject files. How did you solved this??</p> Hmm, just copying the trachook.py to the place gave me an permission error, that normal users which checkin doesn’t have the permission to change /srv/trac/myProject files. How did you solved this??

]]> By: hyperhttp://madssj.com/blog/2008/10/13/mercurial-trac-commit-hook/#comment-32 hyper Tue, 04 Aug 2009 09:32:48 +0000 http://swag.dk/blog/?p=56#comment-32 <p>I kept getting import errors on trachook, eve though i could import successfully into a python shell. I think it had something to do with my PYTHONPATH not being the same as the webserver user's one.</p> <p>I fixed the import problem by using a more explicit syntax: In ~/.hgrc:</p> <p>[extensions] trachook = /full/path/to/trachook.py</p> <p>then in your repository-specific hgrc, follow the syntax exactly as shown in the original entry.</p> I kept getting import errors on trachook, eve though i could import successfully into a python shell. I think it had something to do with my PYTHONPATH not being the same as the webserver user’s one.

I fixed the import problem by using a more explicit syntax: In ~/.hgrc:

[extensions] trachook = /full/path/to/trachook.py

then in your repository-specific hgrc, follow the syntax exactly as shown in the original entry.

]]> By: Mads Sülau Jørgensenhttp://madssj.com/blog/2008/10/13/mercurial-trac-commit-hook/#comment-31 Mads Sülau Jørgensen Tue, 07 Apr 2009 11:59:18 +0000 http://swag.dk/blog/?p=56#comment-31 <p>I've givin you write acess to the repository, should you want to maintain it (better than moving it imo) :)</p> I’ve givin you write acess to the repository, should you want to maintain it (better than moving it imo) :)

]]> By: Marcus Lindblomhttp://madssj.com/blog/2008/10/13/mercurial-trac-commit-hook/#comment-30 Marcus Lindblom Tue, 07 Apr 2009 11:49:53 +0000 http://swag.dk/blog/?p=56#comment-30 <p>For others who end up on this page, I've cloned Mads repo (as he knows) and changed some bits. It's now available here and I'll try to maintain it until the Trac-devs integrate a hook into trac releases. </p> <p>http://bitbucket.org/marcusl/mercurial-trac-hook/</p> <p>Comnments welcome. :)</p> For others who end up on this page, I’ve cloned Mads repo (as he knows) and changed some bits. It’s now available here and I’ll try to maintain it until the Trac-devs integrate a hook into trac releases.

http://bitbucket.org/marcusl/mercurial-trac-hook/

Comnments welcome. :)

]]> By: Mads Sülau Jørgensenhttp://madssj.com/blog/2008/10/13/mercurial-trac-commit-hook/#comment-29 Mads Sülau Jørgensen Thu, 05 Feb 2009 09:04:26 +0000 http://swag.dk/blog/?p=56#comment-29 <p>I've pushed a new version of the hook now - just changed the count() to len(). Let me know if it should break something (it shouldn't).</p> <p><p>Thanks Marcus!</p></p> I’ve pushed a new version of the hook now – just changed the count() to len(). Let me know if it should break something (it shouldn’t).

Thanks Marcus!

]]> By: Marcus Lindblomhttp://madssj.com/blog/2008/10/13/mercurial-trac-commit-hook/#comment-28 Marcus Lindblom Tue, 03 Feb 2009 08:13:16 +0000 http://swag.dk/blog/?p=56#comment-28 <p>Another fix. Starting at v1.1, Mercurial's changelog do not have a count() function anymore. Rather, you need to do use the len() built-in function. </p> <p>I've patched my version for this. Some logic is probably needed to make it work for different versions of Hg.</p> Another fix. Starting at v1.1, Mercurial’s changelog do not have a count() function anymore. Rather, you need to do use the len() built-in function.

I’ve patched my version for this. Some logic is probably needed to make it work for different versions of Hg.

]]> By: Marcus Lindblomhttp://madssj.com/blog/2008/10/13/mercurial-trac-commit-hook/#comment-27 Marcus Lindblom Wed, 14 Jan 2009 08:55:25 +0000 http://swag.dk/blog/?p=56#comment-27 <p>I had to move to hg-ssh in cygwin to make things work, so then I got:</p> <p>TypeError: '_demandmod' object is not iterable</p> <p>I just disabled demandimport at the top of trachook.py, and this made things work again:</p> <p>from mercurial import demandimport demandimport.disable()</p> <p>One could probably be more selective in the disabling, but this seem to work for now.</p> I had to move to hg-ssh in cygwin to make things work, so then I got:

TypeError: ‘_demandmod’ object is not iterable

I just disabled demandimport at the top of trachook.py, and this made things work again:

from mercurial import demandimport demandimport.disable()

One could probably be more selective in the disabling, but this seem to work for now.

]]> By: Mads Sülau Jørgensenhttp://madssj.com/blog/2008/10/13/mercurial-trac-commit-hook/#comment-26 Mads Sülau Jørgensen Mon, 27 Oct 2008 20:05:01 +0000 http://swag.dk/blog/?p=56#comment-26 <p>There, updated at bitbucket. I've added an re.I to the match, which should fix it.</p> There, updated at bitbucket. I’ve added an re.I to the match, which should fix it.

]]> By: Mads Sülau Jørgensenhttp://madssj.com/blog/2008/10/13/mercurial-trac-commit-hook/#comment-25 Mads Sülau Jørgensen Mon, 27 Oct 2008 14:55:23 +0000 http://swag.dk/blog/?p=56#comment-25 <p>That's something that the orginal hook does as well. I'll update my version later tonight, as it's a very annoying "feature".</p> <p>Thanks.</p> That’s something that the orginal hook does as well. I’ll update my version later tonight, as it’s a very annoying “feature”.

Thanks.

]]>