I'm away from my machine, but quick response, I'll say more later.<br><br><br>On Sat, Jun 7, 2008 at 6:51 AM, Marco Pesenti Gritti <<a href="mailto:mpgritti@gmail.com">mpgritti@gmail.com</a>> 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'm still getting a few pylint errors, you don't?.</blockquote><div><br>I was just getting "defined outside of __init__" for ActivityBundle.manifest ... I don't know how I'm supposed to fix that, since it is defined from a subroutine of __init__ that is also useful elsewhere.<br>
</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'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's the purpose of<br>
those. Shouldn't they be real exceptions? What's the advantage of<br>
using the warnings module over logging.warning?</blockquote><div><br>
fixed in the patch I reposted<br>
</div><br>
* I'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>