Small patch to keyword extension ('module' object has no attribute '_parse')

[Christian Ebert]

* Marcin Kasperski on Tuesday, November 06, 2007 at 11:51:15 +0100
>> I mailed you personally, but perhaps the message got spam
>> trapped.
> 
> No, it was sunk deep below horde of Bugzilla messages.
> Plus I live in earlier timezone ;-)

Of course!

>>> b) If you load system libraries, it is not quite so. Here some module
>>> loads system dispatch module which does not offer what we want.
>> 
>> I have no such module here.
> 
> Of course, this is some 3rd party module. But some mercurial
> module (not yours, something from modules your module loads)
> has some construct which causes this module to be loaded
> if present.

I don't like things being done behind my back ;)

>>> (/var/lib/python-support/python2.4/dispatch/ is a directory installed
>>> by python-dispatch package,
>>> http://packages.debian.org/stable/python/python-dispatch)
>> 
>> Seems to debian specific.
> 
> The only 'specific' thing is that I have it installed (not sure why,
> btw).  I suspect that similar problems could happen if I had any
> python module named dispatch/something.py. Or if I installed
> RuleDispatch from http://peak.telecommunity.com/

Yeah, at some time I already had the suspicion that dispatch is
quite a common name.

>> Could you do me a favor and test the following patch instead of
>> yours?
> 
> Works correctly.

Uploaded a version with improved comments thanks to your help at
<http://www.blacktrash.org/hg/hgkeyword/rev/80a300088654>.

c
-- 
In der WIRklICHkeit steht das WIR vor dem ICH.
--Rainald Goetz