<br><br><div class="gmail_quote">On Mon, Jul 9, 2012 at 3:01 PM, Dave Walker <span dir="ltr"><<a href="mailto:DaveWalker@ubuntu.com" target="_blank">DaveWalker@ubuntu.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im">On Mon, Jul 02, 2012 at 08:28:04AM -0400, Monty Taylor wrote:<br>
><br>
><br>
> On 07/02/2012 06:46 AM, John Garbutt wrote:<br>
> > Hi,<br>
> ><br>
> > I noticed I can now run the pep8 tests like this (taken from Jenkins job):<br>
> >     tox -v -epep8<br>
> >     ...<br>
> >     pep8: commands succeeded<br>
> >     congratulations :)<br>
> ><br>
> > But the old way to run tests seems to fail:<br>
> >     ./run-tests.sh -p<br>
> >     ...<br>
> >     File "/home/johngar/openstack/nova/.venv/local/lib/python2.7/site-packages/pep8.py", line 1220, in check_logical<br>
> >         for result in self.run_check(check, argument_names):<br>
> >     TypeError: 'NoneType' object is not iterable<br>
> ><br>
> > Is this expected?<br>
> > Did I just miss an email about this change?<br>
><br>
> I cannot reproduce this on my system. Can you run<br>
> "bash -x run_tests.sh -p" and pastebin the output? Also,<br>
> tools/with_venv.sh pep8 --version just to be sure.<br>
><br>
<br>
</div>Hi,<br>
<br>
The issue is that as of a recent change to upstream pep8 [1], the<br>
additional pep8 rules in tools/hacking.py need to be changed from<br>
returns to yields.. :(<br></blockquote><div><br></div><div>Proof of Concept:  <a href="https://review.openstack.org/#/c/9569">https://review.openstack.org/#/c/9569</a></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<br>
[1] <a href="https://github.com/jcrocholl/pep8/commit/b9f72b16011aac981ce9e3a47fd0ffb1d3d2e085" target="_blank">https://github.com/jcrocholl/pep8/commit/b9f72b16011aac981ce9e3a47fd0ffb1d3d2e085</a><br>
<br>
Kind Regards,<br>
<br>
Dave Walker <<a href="mailto:Dave.Walker@canonical.com">Dave.Walker@canonical.com</a>><br>
Engineering Manager,<br>
Ubuntu Server Infrastructure<br>
<br>_______________________________________________<br>
Mailing list: <a href="https://launchpad.net/~openstack" target="_blank">https://launchpad.net/~openstack</a><br>
Post to     : <a href="mailto:openstack@lists.launchpad.net">openstack@lists.launchpad.net</a><br>
Unsubscribe : <a href="https://launchpad.net/~openstack" target="_blank">https://launchpad.net/~openstack</a><br>
More help   : <a href="https://help.launchpad.net/ListHelp" target="_blank">https://help.launchpad.net/ListHelp</a><br>
<br></blockquote></div><br>