This series contains largely the same patches that I posted yesterday, revised to accomodate the feedback that they received. Here are the highlights of the changes since the first version:
In addition to fixing the reported bugs, the logger module now truncates the log file when it is opened. The version module has been folded into the logger module, so the version string can be written to the beginning of the log file.
The new pep8 helper script has been moved into the tools/ directory, and I integrated it with the run_all_tests script.
The chirp drivers are now located in chirp/drivers/. I verified that the patch correctly imports into the hg repo; 'hg log --follow' works. The trick here is to use 'git format-patch -C'.
Zach Welch (7): Add chirp.logger module (#2347) Use new logger module (#2347) Suppress startup debugging messages (#2347) Add pep8 checker script (#2355) fix style issues (#2355) Fix some style issues (#2355) Move drivers into their own directory (#2351)
chirp/bandplan.py | 3 +- chirp/detect.py | 4 +- chirp/directory.py | 9 +- chirp/drivers/__init__.py | 10 + chirp/{ => drivers}/alinco.py | 0 chirp/{ => drivers}/anytone.py | 13 +- chirp/{ => drivers}/ap510.py | 0 chirp/{ => drivers}/baofeng_uv3r.py | 7 +- chirp/{ => drivers}/bjuv55.py | 7 +- chirp/{ => drivers}/ft1802.py | 3 +- chirp/{ => drivers}/ft1d.py | 17 +- chirp/{ => drivers}/ft2800.py | 20 +- chirp/{ => drivers}/ft50.py | 3 +- chirp/{ => drivers}/ft50_ll.py | 0 chirp/{ => drivers}/ft60.py | 11 +- chirp/{ => drivers}/ft7800.py | 31 +-- chirp/{ => drivers}/ft817.py | 34 ++- chirp/{ => drivers}/ft857.py | 12 +- chirp/{ => drivers}/ft90.py | 56 ++-- chirp/{ => drivers}/ftm350.py | 11 +- chirp/{ => drivers}/h777.py | 16 +- chirp/{ => drivers}/ic208.py | 4 +- chirp/{ => drivers}/ic2100.py | 4 +- chirp/{ => drivers}/ic2200.py | 4 +- chirp/{ => drivers}/ic2720.py | 4 +- chirp/{ => drivers}/ic2820.py | 4 +- chirp/{ => drivers}/ic9x.py | 3 +- chirp/{ => drivers}/ic9x_icf.py | 3 +- chirp/{ => drivers}/ic9x_icf_ll.py | 0 chirp/{ => drivers}/ic9x_ll.py | 0 chirp/{ => drivers}/icf.py | 0 chirp/{ => drivers}/icomciv.py | 15 +- chirp/{ => drivers}/icq7.py | 4 +- chirp/{ => drivers}/ict70.py | 4 +- chirp/{ => drivers}/ict7h.py | 4 +- chirp/{ => drivers}/ict8.py | 3 +- chirp/{ => drivers}/icw32.py | 4 +- chirp/{ => drivers}/icx8x.py | 3 +- chirp/{ => drivers}/icx8x_ll.py | 0 chirp/{ => drivers}/id31.py | 3 +- chirp/{ => drivers}/id51.py | 3 +- chirp/{ => drivers}/id800.py | 4 +- chirp/{ => drivers}/id880.py | 4 +- chirp/{ => drivers}/idrp.py | 13 +- chirp/{ => drivers}/kenwood_hmk.py | 0 chirp/{ => drivers}/kenwood_itm.py | 0 chirp/{ => drivers}/kenwood_live.py | 11 +- chirp/{ => drivers}/kguv8d.py | 42 ++- chirp/{ => drivers}/kyd.py | 26 +- chirp/{ => drivers}/leixen.py | 5 - chirp/{ => drivers}/puxing.py | 7 +- chirp/{ => drivers}/th9800.py | 36 +-- chirp/{ => drivers}/th_uv3r.py | 7 +- chirp/{ => drivers}/th_uv3r25.py | 2 +- chirp/{ => drivers}/th_uvf8d.py | 0 chirp/{ => drivers}/thd72.py | 10 +- chirp/{ => drivers}/thuv1f.py | 0 chirp/{ => drivers}/tk8102.py | 8 +- chirp/{ => drivers}/tmv71.py | 2 +- chirp/{ => drivers}/tmv71_ll.py | 10 +- chirp/{ => drivers}/uv5r.py | 26 +- chirp/{ => drivers}/uvb5.py | 0 chirp/{ => drivers}/vx170.py | 10 +- chirp/{ => drivers}/vx2.py | 32 +-- chirp/{ => drivers}/vx3.py | 35 +-- chirp/{ => drivers}/vx5.py | 4 +- chirp/{ => drivers}/vx510.py | 4 +- chirp/{ => drivers}/vx6.py | 4 +- chirp/{ => drivers}/vx7.py | 4 +- chirp/{ => drivers}/vx8.py | 15 +- chirp/{ => drivers}/vxa700.py | 0 chirp/{ => drivers}/wouxun.py | 15 +- chirp/{ => drivers}/wouxun_common.py | 9 +- chirp/{ => drivers}/yaesu_clone.py | 13 +- chirp/logger.py | 155 +++++++++++ chirpc | 498 ++++++++++++++++++----------------- chirpui/mainapp.py | 9 +- chirpui/radiobrowser.py | 2 +- chirpui/reporting.py | 32 ++- chirpw | 46 ++-- csvdump/__init__.py | 1 - locale/check_parameters.py | 15 +- run_all_tests.sh | 10 +- share/make_supported.py | 1 + tests/run_tests | 2 +- tests/unit/base.py | 1 + tests/unit/test_bitwise.py | 15 +- tests/unit/test_chirp_common.py | 3 + tests/unit/test_mappingmodel.py | 11 +- tests/unit/test_platform.py | 1 + tests/unit/test_shiftdialog.py | 2 + tools/pep8.blacklist | 133 ++++++++++ tools/pep8.manifest | 4 + tools/pep8.py | 75 ++++++ 94 files changed, 1015 insertions(+), 695 deletions(-) create mode 100644 chirp/drivers/__init__.py rename chirp/{ => drivers}/alinco.py (100%) rename chirp/{ => drivers}/anytone.py (98%) rename chirp/{ => drivers}/ap510.py (100%) rename chirp/{ => drivers}/baofeng_uv3r.py (99%) rename chirp/{ => drivers}/bjuv55.py (99%) rename chirp/{ => drivers}/ft1802.py (98%) rename chirp/{ => drivers}/ft1d.py (99%) rename chirp/{ => drivers}/ft2800.py (94%) rename chirp/{ => drivers}/ft50.py (95%) rename chirp/{ => drivers}/ft50_ll.py (100%) rename chirp/{ => drivers}/ft60.py (99%) rename chirp/{ => drivers}/ft7800.py (97%) rename chirp/{ => drivers}/ft817.py (97%) rename chirp/{ => drivers}/ft857.py (99%) rename chirp/{ => drivers}/ft90.py (94%) rename chirp/{ => drivers}/ftm350.py (97%) rename chirp/{ => drivers}/h777.py (98%) rename chirp/{ => drivers}/ic208.py (98%) rename chirp/{ => drivers}/ic2100.py (98%) rename chirp/{ => drivers}/ic2200.py (98%) rename chirp/{ => drivers}/ic2720.py (98%) rename chirp/{ => drivers}/ic2820.py (99%) rename chirp/{ => drivers}/ic9x.py (99%) rename chirp/{ => drivers}/ic9x_icf.py (96%) rename chirp/{ => drivers}/ic9x_icf_ll.py (100%) rename chirp/{ => drivers}/ic9x_ll.py (100%) rename chirp/{ => drivers}/icf.py (100%) rename chirp/{ => drivers}/icomciv.py (96%) rename chirp/{ => drivers}/icq7.py (99%) rename chirp/{ => drivers}/ict70.py (98%) rename chirp/{ => drivers}/ict7h.py (97%) rename chirp/{ => drivers}/ict8.py (97%) rename chirp/{ => drivers}/icw32.py (98%) rename chirp/{ => drivers}/icx8x.py (98%) rename chirp/{ => drivers}/icx8x_ll.py (100%) rename chirp/{ => drivers}/id31.py (99%) rename chirp/{ => drivers}/id51.py (96%) rename chirp/{ => drivers}/id800.py (99%) rename chirp/{ => drivers}/id880.py (99%) rename chirp/{ => drivers}/idrp.py (94%) rename chirp/{ => drivers}/kenwood_hmk.py (100%) rename chirp/{ => drivers}/kenwood_itm.py (100%) rename chirp/{ => drivers}/kenwood_live.py (99%) rename chirp/{ => drivers}/kguv8d.py (96%) rename chirp/{ => drivers}/kyd.py (95%) rename chirp/{ => drivers}/leixen.py (99%) rename chirp/{ => drivers}/puxing.py (99%) rename chirp/{ => drivers}/th9800.py (96%) rename chirp/{ => drivers}/th_uv3r.py (98%) rename chirp/{ => drivers}/th_uv3r25.py (99%) rename chirp/{ => drivers}/th_uvf8d.py (100%) rename chirp/{ => drivers}/thd72.py (98%) rename chirp/{ => drivers}/thuv1f.py (100%) rename chirp/{ => drivers}/tk8102.py (98%) rename chirp/{ => drivers}/tmv71.py (98%) rename chirp/{ => drivers}/tmv71_ll.py (98%) rename chirp/{ => drivers}/uv5r.py (99%) rename chirp/{ => drivers}/uvb5.py (100%) rename chirp/{ => drivers}/vx170.py (94%) rename chirp/{ => drivers}/vx2.py (97%) rename chirp/{ => drivers}/vx3.py (97%) rename chirp/{ => drivers}/vx5.py (98%) rename chirp/{ => drivers}/vx510.py (98%) rename chirp/{ => drivers}/vx6.py (99%) rename chirp/{ => drivers}/vx7.py (99%) rename chirp/{ => drivers}/vx8.py (99%) rename chirp/{ => drivers}/vxa700.py (100%) rename chirp/{ => drivers}/wouxun.py (99%) rename chirp/{ => drivers}/wouxun_common.py (94%) rename chirp/{ => drivers}/yaesu_clone.py (95%) create mode 100644 chirp/logger.py create mode 100644 tools/pep8.blacklist create mode 100644 tools/pep8.manifest create mode 100755 tools/pep8.py