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