<br><div class="gmail_quote">On Mon, Jun 18, 2012 at 8:39 AM, Vincent Untz <span dir="ltr"><<a href="mailto:vuntz@gnome.org" target="_blank">vuntz@gnome.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Hi Thomas,<br>
<br>
Le mercredi 13 juin 2012, à 21:22 +0100, Thomas Kluyver a écrit :<br>
<div class="im">> On 12 June 2012 23:01, Thomas Kluyver <<a href="mailto:thomas@kluyver.me.uk">thomas@kluyver.me.uk</a>> wrote:<br>
> > I've put the changes I've got so far on Github - feel free to pull from there:<br>
> ><br>
> > <a href="https://github.com/takluyver/pyxdg/tree/update" target="_blank">https://github.com/takluyver/pyxdg/tree/update</a><br>
><br>
> I've now got an account (thanks!), and I've pushed the changes to<br>
> freedesktop git. It's in a separate branch until someone can do a bit<br>
> of code review:<br>
><br>
> <a href="http://cgit.freedesktop.org/xdg/pyxdg/?h=update" target="_blank">http://cgit.freedesktop.org/xdg/pyxdg/?h=update</a><br>
<br>
</div>Given that it seemed nobody else was stepping up for maintainership of<br>
pyxdg, I think you can just trust yourself, merge the branch, do a<br>
release and wait for bug reports from early users ;-)<br>
<br>
Cheers,<br>
<div class="im HOEnZb"><br>
Vincent<br>
<br>
--<br>
Les gens heureux ne sont pas pressés.<br>
</div><div class="HOEnZb"><div class="h5">_______________________________________________<br>
xdg mailing list<br>
<a href="mailto:xdg@lists.freedesktop.org">xdg@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/xdg" target="_blank">http://lists.freedesktop.org/mailman/listinfo/xdg</a><br>
</div></div></blockquote></div><div><br></div><div>Hi Thomas</div><div><br></div><div>Your changes look good, you will also want to change the old-style classes to new-style. See about swapping the os.spawn* calls to subprocess as well.</div>

<div><br></div><div>There's also a lot of inconsistency issues to fix, especially wrt the API (through a single class you'll find methods using this_style(), thisOne() and ThatOne(), it's pretty awful).</div>
<div>
<br></div><div>I don't know how many pyxdg users there are, but the whole thing needs a serious cleanup at a backwards-incompatible level. </div><div><br clear="all">J. Leclanche<br><br></div>