Ray Kinsella <m...@ashroe.eu> writes: > Scripts to count and track the lifecycle of DPDK symbols. > > The symbol-tool script reports on the growth of symbols over releases > and list expired symbols. The notify-symbol-maintainers script > consumes the input from symbol-tool and generates email notifications > of expired symbols. > > v2: reworked to fix pylint errors > v3: sent with the correct in-reply-to > v4: fix typos picked up by the CI > v5: fix terminal_size & directory args > v6: added list-expired, to list expired experimental symbols > v7: fix typo in comments > v8: added tool to notify maintainers of expired symbols > v9: removed hardcoded emails addressed and script names > v10: added ability to identify and notify the original contributors > > Ray Kinsella (3): > devtools: script to track symbols over releases > devtools: script to send notifications of expired symbols > maintainers: add new abi scripts > > MAINTAINERS | 2 + > devtools/notify-symbol-maintainers.py | 256 ++++++++++++++ > devtools/symbol-tool.py | 482 ++++++++++++++++++++++++++ > 3 files changed, 740 insertions(+) > create mode 100755 devtools/notify-symbol-maintainers.py > create mode 100755 devtools/symbol-tool.py
I get a whole mess of flake8 issues from this series (mostly 'backslash is redundant' and whitespace issues). I'm using flake8 because it pretty well enforces PEP8 style guide. I would like to see it addressed, but also I see that many of the python files in the DPDK tree don't actually pass. Example:: $ flake8 ./usertools/dpdk-devbind.py | wc -l 34 $ flake8 ./usertools/dpdk-devbind.py | sed 's@./usertools/dpdk-devbind.py[:0-9]* @@' | sort -u E128 continuation line under-indented for visual indent E302 expected 2 blank lines, found 1 E305 expected 2 blank lines after class or function definition, found 1 E501 line too long (105 > 79 characters) E501 line too long (80 > 79 characters) E501 line too long (82 > 79 characters) E501 line too long (83 > 79 characters) E501 line too long (84 > 79 characters) E501 line too long (85 > 79 characters) E501 line too long (86 > 79 characters) E501 line too long (91 > 79 characters) E502 the backslash is redundant between brackets E722 do not use bare 'except' Looks like we repeat the same kinds of errors everywhere (this is on multiple tools). Some of our in-tree python is better than others (like app/test/autotest.py which only has 1 flake). Maybe we can address this. Other comments inline on the patches.
