<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Looks good.  <div><br></div><div>I would suggest you change the uses of 'args +=' to args.extend, just to be consistent with how it is done in the rest of the function.</div><div><br></div><div>for example:</div><div> args += ['--submit', submitURL]</div><div><br></div><div>to:</div><div>args.extend(['--submit', submitURL])</div><div><br></div><div>A buildbot reload/restart will be needed for your changes to take effect.</div><div><br><div><div>On Mar 20, 2014, at 11:29 AM, Galina Kistanova <<a href="mailto:gkistanova@gmail.com">gkistanova@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div dir="ltr"><div><div>Hi Renato,<br><br></div>Looks fine for me. Please feel free to commit it.<br><br></div>Thanks<br><br>Galina</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Mar 20, 2014 at 9:01 AM, Tobias Grosser <span dir="ltr"><<a href="mailto:tobias@grosser.es" target="_blank">tobias@grosser.es</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On 03/20/2014 04:59 PM, Renato Golin wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Tobi,<br>
<br>
You don't need to change PollyBuilder because it already passes<br>
submitURL to LNTBuilder.<br>
<br>
New patch attached.<br>
</blockquote>
<br></div>
OK, it looks good to me then (assuming it is tested).<br>
<br>
Thanks Renato!<br>
<br>
Cheers<br>
<br>
</blockquote></div><br><br clear="all"><br>-- <br><div dir="ltr"><div>Thanks<br><br></div>Galina<br></div>
</div>
</blockquote></div><br></div></body></html>