<font color="#222222" face="arial, sans-serif"><br></font>On Tue, Jun 19, 2012 at 4:22 PM, Manuel Kaufmann <span dir="ltr"><<a href="mailto:humitos@gmail.com" target="_blank">humitos@gmail.com</a>></span> wrote:<font color="#222222" face="arial, sans-serif"><br>
</font><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Tue, Jun 19, 2012 at 3:56 PM, Gonzalo Odiard <<a href="mailto:gonzalo@laptop.org">gonzalo@laptop.org</a>> wrote:<br>

> The short version is: please, don't add more changes to the needed by the<br>
> port.<br>
> If you find a bug, or anything wrong with the code, is very good, but is<br>
> better include it<br>
> in another patch. We can decide if have sense apply in the old branch too.<br>
<br>
</div>Ok, some of them have an explanation. They are not added just because.<br>
I will explain this below.<br>
<div class="im"><br>
> Also, pleeeease :) don't change spaces if not needed. In particular, the<br>
> second line,<br>
> when continue after the 80 chars, I prefer use 8 chars indentation. I am not<br>
> sure if there are a agreed way here, but if not, better keep the code as is.<br>
<br>
</div>About spaces. There is a convention[1] used that says we should use 4<br>
chars for indentation (in Python Code): PEP8<br>
<div class="im"><br></div></blockquote><div><br></div><div>Yeah, I know that, Mr Humitos :)</div><div>I am not talking about the indentation used to identify code structures (probably I should not use the word indentation) I am talking about the spaces used in a second line, when the code does not fit in 80 cols. Example:</div>
<div><br></div><div><blockquote class="gmail_quote" style><font size="1"><br><font face="courier new, monospace">        self.queryresults = opds.RemoteQueryResult(catalog_config,<br>-                '', query_language)<br>
+                                                   '', query_language)</font></font><br></blockquote></div><div><br></div><div>You are aligning the second row with the "(". Please use 8 chars in the second row.</div>
<div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
>> [1] <a href="http://wiki.sugarlabs.org/go/User:Humitos/PortingGetBooks" target="_blank">http://wiki.sugarlabs.org/go/User:Humitos/PortingGetBooks</a><br>
> The wiki page looks outdated in parts. Please update it.<br>
<br>
</div>What are the things that are outdated?<br>
<div class="im"><br></div></blockquote><div><br></div><div>There are a few notes about issues as pending, and probably were resolved later. </div><div> </div><div><br></div><div><span style> * [IMPORTANT] I'm not sure what is org.freedesktop.UDisks.Device for?</span><br style>
<span style>I don't understand it very well but I had to remove it because it was</span><br style><span style>failing</span><br style><br>UDisk is used to identify when a pendrive is connected to the xo. I am 99% sure is falling because you are using "import dbus" instead of "from gi.repository import DBus" (see below) <br>
<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
The fixs that you mentioned above are not present in the gtk2 version<br>
of the Activity. I mean, I didn't fix an old bug here, in fact, I<br>
fixed the bugs that appeared after the migration. I tried to get the<br>
same behaviour on both version without adding and removing anything. I<br>
think this is related on how are the signals emitted between gtk2 and<br>
Gtk3 and that is why I wrote that comment.<br>
<div class="im"><br></div></blockquote><div><br></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im"></div><div class="im">

> Probably the problem in devicemanager.py is you are using<br>
><br>
> import dbus<br>
> and not<br>
> from gi.repository import DBus<br>
<br>
</div>OK. I will take a look at this. Do you have an example about how to use this?<br>
<br>
from gi.repository import DBus<br>
<div class="im"><br></div></blockquote><div><br></div><div>It's like in all the other port :)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
<br>
> Testing the Gtk3 version, I have found the following error:<br>
> When I click in a book title in the listview, the information about the book<br>
> should be displayed in the textview at the botton of the screen. In the Gtk3<br>
> version, nothing is displayed. In the log I see:<br>
<br>
</div>Yes. This is working on the English version of the Activity. I will<br>
take a look a this, but I didn't change anything related with this.<br>
<div class="im"><br></div></blockquote><div><br></div><div>I know is not your fault. Probably gtk3 is returning differently the strings as gtk2. </div><div><br></div><div>Gonzalo</div><div> </div></div>