On Mon, Sep 7, 2009 at 11:50 PM, Yuki KODAMA<endflow....@gmail.com> wrote:
>>> class CommentLine(LineType):
>>>    regex = re.compile(r'^(?P<csep>[;#]|[rR][eE][mM])'
>>>                       r'(?P<comment>.*)$')
>>
>> That's regrettable, and it doesn't appear to be configurable.
>
> Yep, I disappointed this when I saw...
>
>>> However, you may know, the alias 'remote' is valid in Mercurial.
>>> So I suppose we shouldn't use iniparse module for [paths] section.
>>> To acquire paths from HGRC file, we can use
>>>
>>> self.repo.ui.configitems('paths')
>>
>> That is certainly an option but I worry about other places that might
>> get caught by this.  Other than monkey patching iniparse, I don't see
>> a lot of good options.
>
> Ah, indeed. If I call it, I will also get paths from Global HGRC file.
> However, I think unofficial patching is a final option, too.
> Um... how about using "config.py" module of mercurial directly?
> It has "config.read()" function.

That would be ok on the default branch, since we require at least hg 1.3.

> "ui" has also similar function "readconfig", but it doesn't work what
> I expected in my trying.

There will be other side effects as well.  Since iniparse is treating
'rem' lines as comments, it will not update them if you change their
values.  It will create new lines for every change you make.

--
Steve Borho

------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with 
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
Tortoisehg-develop mailing list
Tortoisehg-develop@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tortoisehg-develop

Reply via email to