Opened on 08/17/2016 at 11:39:41 AM
Closed on 08/17/2016 at 03:12:58 PM
#4342 closed defect (fixed)
tox fails due to incompatibility of flake8-putty with flake8 3.0
| Reported by: | sebastian | Assignee: | sebastian |
|---|---|---|---|
| Priority: | P2 | Milestone: | |
| Module: | Sitescripts | Keywords: | |
| Cc: | kvas, jsonesen | Blocked By: | |
| Blocking: | Platform: | Unknown / Cross platform | |
| Ready: | yes | Confidential: | no |
| Tester: | Unknown | Verified working: | no |
| Review URL(s): |
https://codereview.adblockplus.org/29349917/ |
||
Description
How to reproduce
- Clone sitescripts, cms or buildtools.
- Run tox from the repository root.
Observed behaviour
When tox runs flake8 following exception is raised:
Traceback (most recent call last):
File ".tox/py27/bin/flake8", line 11, in <module>
sys.exit(main())
File "/home/sebastian/src/sitescripts/.tox/py27/local/lib/python2.7/site-packages/flake8/main/cli.py", line 16, in main
app.run(argv)
File "/home/sebastian/src/sitescripts/.tox/py27/local/lib/python2.7/site-packages/flake8/main/application.py", line 316, in run
self._run(argv)
File "/home/sebastian/src/sitescripts/.tox/py27/local/lib/python2.7/site-packages/flake8/main/application.py", line 299, in _run
self.initialize(argv)
File "/home/sebastian/src/sitescripts/.tox/py27/local/lib/python2.7/site-packages/flake8/main/application.py", line 290, in initialize
self.register_plugin_options()
File "/home/sebastian/src/sitescripts/.tox/py27/local/lib/python2.7/site-packages/flake8/main/application.py", line 150, in register_plugin_options
self.check_plugins.register_options(self.option_manager)
File "/home/sebastian/src/sitescripts/.tox/py27/local/lib/python2.7/site-packages/flake8/plugins/manager.py", line 481, in register_options
list(self.manager.map(register_and_enable))
File "/home/sebastian/src/sitescripts/.tox/py27/local/lib/python2.7/site-packages/flake8/plugins/manager.py", line 277, in map
yield func(self.plugins[name], *args, **kwargs)
File "/home/sebastian/src/sitescripts/.tox/py27/local/lib/python2.7/site-packages/flake8/plugins/manager.py", line 477, in register_and_enable
call_register_options(plugin)
File "/home/sebastian/src/sitescripts/.tox/py27/local/lib/python2.7/site-packages/flake8/plugins/manager.py", line 373, in generated_function
return method(optmanager, *args, **kwargs)
File "/home/sebastian/src/sitescripts/.tox/py27/local/lib/python2.7/site-packages/flake8/plugins/manager.py", line 223, in register_options
add_options(optmanager)
File "/home/sebastian/src/sitescripts/.tox/py27/local/lib/python2.7/site-packages/flake8_putty/extension.py", line 132, in add_options
parser.config_options.append('putty-select')
AttributeError: 'OptionManager' object has no attribute 'config_options'
ERROR: InvocationError: '/home/sebastian/src/sitescripts/.tox/py27/bin/flake8 sitescripts multiplexer.py multiplexer.fcgi'
This didn't happen before flake8 3.0 has been released. This is an incompatibility of flake8-putty, a third-party flake8 extension, which we use for more granular ignores in order to deal with legacy code.
Expected behaviour
tox should complete successfully, running flake8 without any errors.
What to change
As long as flake8-putty is incompatible with flake8 3.0, we have to explicitly require flake8==2.*.
Attachments (0)
Change History (5)
comment:1 Changed on 08/17/2016 at 11:54:49 AM by sebastian
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:2 Changed on 08/17/2016 at 03:11:18 PM by abpbot
comment:3 Changed on 08/17/2016 at 03:12:14 PM by abpbot
A commit referencing this issue has landed:
Issue 4342 - Pin flake8 2.x for compatibility with flake8-putty
comment:4 Changed on 08/17/2016 at 03:12:20 PM by abpbot
A commit referencing this issue has landed:
Issue 4342 - Pin flake8 2.x for compatibility with flake8-putty
comment:5 Changed on 08/17/2016 at 03:12:58 PM by sebastian
- Resolution set to fixed
- Status changed from reviewing to closed

A commit referencing this issue has landed:
Issue 4342 - Pin flake8 2.x for compatibility with flake8-putty