I&#39;m away from my machine, but quick response, I&#39;ll say more later.<br><br><br>On Sat, Jun 7, 2008 at 6:51 AM, Marco Pesenti Gritti &lt;<a href="mailto:mpgritti@gmail.com">mpgritti@gmail.com</a>&gt; wrote:<br><div class="gmail_quote">
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">I&#39;m still getting a few pylint errors, you don&#39;t?.</blockquote><div><br>I was just getting &quot;defined outside of __init__&quot; for ActivityBundle.manifest ... I don&#39;t know how I&#39;m supposed to fix that, since it is defined from a subroutine of __init__ that is also useful elsewhere.<br>
&nbsp;</div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"> Also can you please<br>
answer the question/notes I posted before I go ahead an do a full<br>
review?</blockquote><div><br>OK, here:<br><br>
* Please run pylint on the changed files. You can use<br>
sugar-jhbuild/scripts/data<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">/pylintrc. That will catch several problems<br>
and nitpicks that I&#39;d ask you to fix anyway.</blockquote><div><br>Done, I think, I will check again later. <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
* I dont understand all the<br>
warnings.warn(MalformedBundleException(...)). What&#39;s the purpose of<br>
those. Shouldn&#39;t they be real exceptions? What&#39;s the advantage of<br>
using the warnings module over logging.warning?</blockquote><div><br>
fixed in the patch I reposted<br>
&nbsp;</div><br>
* I&#39;m not sure what you are using dest_dir for, but it looks like it<br>
should be part of Config.<br><br>again, fixed.<br><br>
* Why are you moving the activity and env imports inline.<br><br>I answered that in my previous email: to make it easier to use these files without having the rest of Sugar installed.<br><br></div></div><br>