
<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Jul 30, 2010, at 6:19 PM, David García Garzón wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div>On Tuesday 27 July 2010 18:55:43 Xavier Serra Román wrote:<br><blockquote type="cite">Hi guys,<br></blockquote><blockquote type="cite">I finished the integration of clamrefactor with NetworkEditor. The code is<br></blockquote><blockquote type="cite">in the patch. I'm sorry for all the warning it throws, I don't know how to<br></blockquote><blockquote type="cite">silence them. The code has some limitations:<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">1. clamrefactor.py and migrationScript need to be in the same directory<br></blockquote><blockquote type="cite">from where you launch NetworkEditor 2. NetworkEditor has to be launched<br></blockquote><blockquote type="cite">using: PYTHONPATH=`pwd` ./NetworkEditor This is needed for the Python<br></blockquote><blockquote type="cite">Interpreter embedded to find the module.<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">The SConstruct file is changed too because it's needed to link with the<br></blockquote><blockquote type="cite">Python libraries<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">Bye!<br></blockquote><br>We should meet to decide what to do with the scripts and paths. Embedding <br>would be the perfect solution, at least for the migration script but also for <br>the clamrefactor module. Is that possible?<br><br></div></blockquote><div><br></div><div>I don't really know. The safest assumption would be that it's possible. I'll search around</div><div>to see If I find something. We should decide too how to treat errors (tell the user the </div><div>specific error; just tell that something happened; do nothing and load the network file </div><div>without any change...)</div><div><br></div><br><blockquote type="cite"><div>It also looks like the loadFromString and loadFrom file have some code <br>duplication, and that the python bit looks like an alien we should move apart <br>to some class.<br><br></div></blockquote><div><br></div><div>I though so too, but I didn't know how to solve the problem. Maybe we can come up with</div><div>a solution together.</div><br><blockquote type="cite"><div>If you are still by Barcelona, drop yourself by my office and let's do some <br>pair programming.<br><font class="Apple-style-span" color="#000000"><font class="Apple-style-span" color="#144FAE"><br></font></font></div></blockquote><blockquote type="cite"><div>David.<br>_______________________________________________<br>clam-devel mailing list<br><a href="mailto:clam-devel@lists.clam-project.org">clam-devel@lists.clam-project.org</a><br>http://lists.clam-project.org/listinfo.cgi/clam-devel-clam-project.org<br></div></blockquote></div><br></body></html>