<div class="gmail_quote">On Sun, Feb 7, 2010 at 6:11 AM, Christian Ebert <span dir="ltr"><<a href="mailto:blacktrash@gmx.net">blacktrash@gmx.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Hi Kevin,<br>
<br>
* Christian Ebert on Wednesday, February 03, 2010 at 09:05:13 +0000<br>
<div class="im">> * Kevin Grover on Saturday, January 30, 2010 at 14:53:38 -0600<br>
>> # HG changeset patch<br>
>> # User Kevin O. Grover <<a href="mailto:kevin@kevingrover.net">kevin@kevingrover.net</a>><br>
>> # Date 1264179036 28800<br>
>> # Node ID 02ffdeb799f73cc3c1a04627586cf3039dc5b058<br>
>> # Parent e898bc7810ad08affd99d4420c89165252db194c<br>
>> Use os.sep as path separator in keywords.<br>
<br>
</div>Usually the message for extensions is like:<br>
<br>
keyword: use os.sep as path separator in default keywordmaps<br>
<div class="im"><br>
>> diff -r e898bc7810ad -r 02ffdeb799f7 hgext/keyword.py<br>
>> --- a/hgext/keyword.py Thu Jan 21 13:26:26 2010 +0100<br>
>> +++ b/hgext/keyword.py Fri Jan 22 08:50:36 2010 -0800<br>
>> @@ -82,7 +82,7 @@<br>
>> from mercurial.lock import release<br>
>> from mercurial.node import nullid<br>
>> from mercurial.i18n import _<br>
>> -import re, shutil, tempfile<br>
>> +import re, shutil, tempfile, os<br>
>><br>
>> commands.optionalrepo += ' kwdemo'<br>
>><br>
>> @@ -115,9 +115,9 @@<br>
>> 'RCSfile': '{file|basename},v',<br>
>> 'RCSFile': '{file|basename},v', # kept for backwards compatibility<br>
>> # with hg-keyword<br>
>> - 'Source': '{root}/{file},v',<br>
>> + 'Source': '{root}%s{file},v' % (os.sep, ),<br>
>> 'Id': '{file|basename},v {node|short} {date|utcdate} {author|user}',<br>
>> - 'Header': '{root}/{file},v {node|short} {date|utcdate} {author|user}',<br>
>> + 'Header': '{root}%s{file},v {node|short} {date|utcdate} {author|user}' % (os.sep, ),<br>
<br>
</div>you will have to break that line.<br>
<div class="im"><br>
>> }<br>
>><br>
>> def __init__(self, ui, repo):<br>
><br>
> ugh, yes, thank you.<br>
><br>
> But I think<br>
><br>
> 'Source': '{root}%s{file},v' % os.sep,<br>
><br>
> is safe enough and simpler.<br>
<br>
</div>test-keyword should not really be problem (it filters the output<br>
of kwdemo), so why don't you go ahead and resubmit this.<br>
Otherwise I might do it ;-)<br>
<div><div></div><div class="h5"><br>
c<br>
--<br>
\black\trash movie _C O W B O Y_ _C A N O E_ _C O M A_<br>
Ein deutscher Western/A German Western<br>
<br>
--->> <a href="http://www.blacktrash.org/underdogma/ccc.php" target="_blank">http://www.blacktrash.org/underdogma/ccc.php</a><br>
</div></div></blockquote></div><br><br>I was traveling when I worked on the other part. I'm now back and won't have time to play with this again until the weekend (at the soonest). Feel free to make the changes and resubmit it.<br>
<br>Thanks.<br><br>