Skip to content

Commit

Permalink
travis: Fix flake8 failures from flake8 3.0.
Browse files Browse the repository at this point in the history
The "hacking" plugin for flake8 is not currently compatible with flake8
3.0.  Ensure that we install flake8 2.x on travis-ci.  Also update the
docs to indicate this incompatibility.

Signed-off-by: Russell Bryant <[email protected]>
Acked-by: Andy Zhou <[email protected]>
  • Loading branch information
russellb committed Jul 29, 2016
1 parent d5eec8d commit ac1432a
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 6 deletions.
6 changes: 5 additions & 1 deletion .travis/linux-prepare.sh
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,8 @@

git clone git://git.kernel.org/pub/scm/devel/sparse/chrisl/sparse.git
cd sparse && make && make install && cd ..
pip install --disable-pip-version-check --user six flake8 hacking

# Incompatibility between flake8 3.0.x and the hacking plugin:
# https://gitlab.com/pycqa/flake8/issues/153
# https://bugs.launchpad.net/hacking/+bug/1607942
pip install --disable-pip-version-check --user six "flake8<3.0" hacking
12 changes: 7 additions & 5 deletions INSTALL.md
Original file line number Diff line number Diff line change
Expand Up @@ -127,11 +127,13 @@ installing the following to obtain better warnings:

- clang, version 3.4 or later

- “flake8”, along with the “hacking” flake8 plugin (for Python code).
The automatic flake8 check that runs against Python code has some
warnings enabled that come from the "hacking" flake8 plugin. If it's
not installed, the warnings just won't occur until it's run on a system
with "hacking" installed.
- “flake8”, version 2.X, along with the “hacking” flake8 plugin (for Python
code). The automatic flake8 check that runs against Python code has some
warnings enabled that come from the "hacking" flake8 plugin. If it's not
installed, the warnings just won't occur until it's run on a system with
"hacking" installed. Note that there are problems with flake8 3.0 and the
“hacking” plugin. To ensure you get flake8 2.X, you can use
“pip install ‘flake8<3.0’”.

Also, you may find the ovs-dev script found in utilities/ovs-dev.py useful.

Expand Down

0 comments on commit ac1432a

Please sign in to comment.