[chirp_devel] [PATCH v2 0/7] logging, style, and drivers... oh, my!

Zach Welch
Thu Feb 26 00:42:13 PST 2015


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

-- 
1.9.1




More information about the chirp_devel mailing list