<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Apr 4, 2014 at 5:24 PM, James Cameron <span dir="ltr"><<a href="mailto:quozl@laptop.org" target="_blank">quozl@laptop.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">It is so irritating that to review patches to Sugar one has to click<br>

on the link in the mail, which doesn't even get you to the patch, but<br>
instead takes you to a page you have to click twice on to see the<br>
code.  And since the pages are SSL over HTTP, the time to connect can<br>
be considerable.  This really favours developers with low latency<br>
internet.  ;-}<br>
<div class=""><br></div></blockquote><div><br></div><div>I know, modern times....</div><div><br></div><div>manuq taught me a useful github trick:</div><div><br></div><div>when you have a link like <a href="https://github.com/sugarlabs/sugar/pull/328">https://github.com/sugarlabs/sugar/pull/328</a></div>
<div>you can add .patch or .diff to the url </div><div>and get a clean patch </div><div>can be useful for you I imagine.</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div class="">
On Fri, Apr 04, 2014 at 12:04:25PM -0300, Gonzalo Odiard wrote:<br>
> I was exploring use the gi api for network manager,<br>
> instead of our old dbus api. <br>
> This was not done when we ported sugar to gtk3.<br>
><br>
> I have sent two pr [1] and [2] with the easy parts.<br>
<br>
</div>Reviewed-by: James Cameron <<a href="mailto:quozl@laptop.org">quozl@laptop.org</a>><br>
<br>
It is so much simpler.  I like it, please continue.<br></blockquote><div><br></div><div>Thanks. </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<br>
It reminds me of how silly the dbus api was when I first saw it.<br>
<div class=""><br>
> The last piece missing is jarabe/model/network.py and is a big<br>
> piece. I started to change it, but look like a big project,  and<br>
> will need a lot more work and extensive testing.<br>
<br>
</div>Yes, it will be some considerable work, but as you know, it can be<br>
broken into short periods of editing.  When I have to break such a<br>
conversion into multiple sessions, I add more careful methods to check<br>
the changes before commit.<br>
<span class=""><font color="#888888"><br></font></span></blockquote><div><br></div><div>I agree. Is a good opportunity to add testing to our networking code.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<span class=""><font color="#888888">
--<br>
James Cameron<br>
<a href="http://quozl.linux.org.au/" target="_blank">http://quozl.linux.org.au/</a><br>
_______________________________________________<br>
Sugar-devel mailing list<br>
<a href="mailto:Sugar-devel@lists.sugarlabs.org">Sugar-devel@lists.sugarlabs.org</a><br>
<a href="http://lists.sugarlabs.org/listinfo/sugar-devel" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">Gonzalo Odiard<br><br><div>SugarLabs - Software for children learning <br></div></div>
</div></div>