diff options
| author | Jon Schlueter <jschlueter@navigationsolutions.com> | 2015-02-13 08:28:05 -0500 |
|---|---|---|
| committer | Jon Schlueter <jschlueter@navigationsolutions.com> | 2015-02-13 08:37:24 -0500 |
| commit | 528f6ab639dc7ab2ba7439e3e766ee0df02cab4b (patch) | |
| tree | c03ce70a3474dbb748704d1b68a5f1828eb1fcdb /SConstruct | |
| parent | 11b0995b60abd7e38bc624845a648438550bb335 (diff) | |
| download | gpsd-528f6ab639dc7ab2ba7439e3e766ee0df02cab4b.tar.gz | |
Fixup scons audit for python
pylint was not checking python_progs correctly anymore due to name
mismatch. Converted to joining python_progs and a list of other scripts
to explicitly check
also added pep8 audit with error/warnings ignored for stylistic issues
that are not blockers.
Diffstat (limited to 'SConstruct')
| -rw-r--r-- | SConstruct | 11 |
1 files changed, 10 insertions, 1 deletions
@@ -1503,7 +1503,16 @@ Utility("scan-build", ["gpsd.h", "packet_names.h"], # Sanity-check Python code. pylint = Utility("pylint", ["jsongen.py", "maskaudit.py", python_built_extensions], - ['''pylint --rcfile=/dev/null --dummy-variables-rgx='^_' --msg-template="{path}:{line}: [{msg_id}({symbol}), {obj}] {msg}" --reports=n --disable=F0001,C0103,C0111,C1001,C0301,C0302,C0322,C0324,C0323,C0321,C0330,R0201,R0801,R0902,R0903,R0904,R0911,R0912,R0913,R0914,R0915,W0110,W0201,W0121,W0232,W0234,W0401,W0403,W0141,W0142,W0603,W0614,W0640,W0621,E1101,E1102,F0401 jsongen.py leapsecond.py maskaudit.py gpsprof.py gpscat.py gpsfake.py gegps.py gps/*.py xgps''']) + ['''pylint --rcfile=/dev/null --dummy-variables-rgx='^_' --msg-template="{path}:{line}: [{msg_id}({symbol}), {obj}] {msg}" --reports=n --disable=F0001,C0103,C0111,C1001,C0301,C0302,C0322,C0324,C0323,C0321,C0330,R0201,R0801,R0902,R0903,R0904,R0911,R0912,R0913,R0914,R0915,W0110,W0201,W0121,W0232,W0234,W0401,W0403,W0141,W0142,W0603,W0614,W0640,W0621,E1101,E1102,F0401 gps/*.py *.py ''' + " ".join(python_progs)]) + +# Additional Python readablity style checks +pep8 = Utility("pep8", ["jsongen.py", "maskaudit.py", python_built_extensions], + ['''pep8 --ignore=E501,W602,E122 {} gps/[a-zA-Z]*.py *.py'''.format(" ".join(python_progs))]) + +# Additional Python readablity style checks +flake8 = Utility("flake8", ["jsongen.py", "maskaudit.py", python_built_extensions], + ['''flake8 --ignore=E501,W602,E122 {} gps/[a-zA-Z]*.py *.py'''.format(" ".join(python_progs))]) + # Check the documentation for bogons, too Utility("xmllint", glob.glob("*.xml"), |