Open Stack

On Thu, May 16, 2013 at 7:10 PM, Clark Boylan <clark.boylan at gmail.com> wrote:
> On Thu, May 16, 2013 at 7:04 PM, Henry Gessau <gessau at cisco.com> wrote:
>> When I run 'tox -e pep8' I get the following warning:
>>
>>   pep8 runtests: commands[0]
>>   WARNING:test command found but not installed in testenv
>>     cmd: /bin/bash
>>     env: /home/henry/Dev/q-cisco-ut/.tox/pep8
>>
> Tox is running bash as one of its commands. Bash is not installed into
> the pep8 virtualenv so you get this warning. You can see the command
> tox runs in the tox.ini file. In this case it is `bash
> ./tools/run_pep8.sh`. You can ignore this warning for now, but I
> believe Monty wants to fix it so that quantum does not use a run_pep8
> script. If he does that properly the warning should go away.
>
I have taken a second look at the patch that caused the warning to
appear and I think it should be reverted. tox and run_tests.sh should
each call flake8 directly. There is no reason to have a common script
when running just flake8 works. flake8 is also installed into the
virtualenv so reverting that change will make the warning go away.

Clark