[chirp_devel] Build test results: Failure
Tested changes:
[Attilio Pannella ppan70@gmail.com] Minor update to support RTX Radioddity QB25 (a QYT KT7900D clone) - fix for Issue Number #5135
[Patrik Nilsson asavartzeth@gmail.com] Fix Linux/Unix desktop entry file Fixes #5075 #3491
- Update "Version" to 1.0 - Fix "GenericName" not being generic - Fix "Comment" being redundant with "GenericName" and wrong format - Fix hard-coded icon extension - Fix "Categories" using incorrect non-existing main category "Application", using "Utility" instead. - Add ability to open files with chirpw (based on Debian patch) - Add several "Keywords" for discoverability
Fixing the hard-coded icon enables theming and is a pre-requsite for adding a scalable icon and/or icons of different sizes. It should also resolve bug #3491.
The latest version of the Freedesktop specification can be found at: https://standards.freedesktop.org/desktop-entry-spec/desktop-entry-spec-late...
A table of acceptable main categories can be found at: https://standards.freedesktop.org/menu-spec/menu-spec-1.0.html#category-regi...
[Tom Hayward tom@tomh.us] [id880] Fix typo in charset definition. #281
[Tom Hayward tom@tomh.us] [thf6a] Support full charset (ASCII). Fixes #141
[Tom Hayward tom@tomh.us] [id880] Support full charset. Fixes #281
[Tom Hayward tom@tomh.us] [vx5] Support full charset (ASCII). Fixes #292
[Tom Hayward tom@tomh.us] [id31a] set used bit when creating new memory, clear when deleting. Fixes #269
[Tom Hayward tom@tomh.us] Support PyGTK < 2.22 in bank edit. Fixes #231
[Tom Hayward tom@tomh.us] [d710] [v71] [d72] Fix tone list (not all tones are supported). Fixes #212
[Dan Smith dsmith@danplanet.com] [vx7] Fix setting memory power levels on 220MHz band Fixes #214
[Dan Smith dsmith@danplanet.com] fips: Pennsylvania FIPS code was wrong. #117
[Marco Filippi iz3gme.marco@gmail.com] Consider lower bound frequency of each valid_band as valid Fix bug #181
[Tom Hayward tom@tomh.us] tmd700: allow 8-char names. Fixes #176
[Dan Smith dsmith@danplanet.com] Fix the "blind deletion" problem, as well as properly direct copy/paste Fixes #172
[David Griffith dave@661.org] Bug #155 fix: VX-7 1.25m power levels
[David Griffith dave@661.org] New INSTALL and README files Fixes #122
[Tom Hayward tom@tomh.us] thd72: only use hardware flow on OS X. Fixes #166
[Marco Filippi iz3gme.marco@gmail.com] [FT817] Tone freq not set correctly Same as #88 for FT857, to avoid code duplication fix code have been moved from ft857 to its ancestor class Fix bug #163
[Tom Hayward tom@tomh.us] Fix Mac .app so paths with spaces work. Fixes Bug #145
Full log:
Started by an SCM change Building in workspace /var/lib/jenkins/jobs/chirp-test/workspace [workspace] $ hg showconfig paths.default [workspace] $ hg pull --rev default [workspace] $ hg update --clean --rev default 2 files updated, 0 files merged, 0 files removed, 0 files unresolved [workspace] $ hg log --rev . --template {node} [workspace] $ hg log --rev . --template {rev} [workspace] $ hg log --rev c56244c58fe85688ef0da20a3f7bfebc0f447e6c [workspace] $ hg log --template "<changeset node='{node}' author='{author|xmlescape}' rev='{rev}' date='{date}'><msg>{desc|xmlescape}</msg><added>{file_adds|stringify|xmlescape}</added><deleted>{file_dels|stringify|xmlescape}</deleted><files>{files|stringify|xmlescape}</files><parents>{parents}</parents></changeset>\n" --rev default:0 --follow --prune c56244c58fe85688ef0da20a3f7bfebc0f447e6c No emails were triggered. [workspace] $ /bin/sh -xe /tmp/hudson8569872365646041100.sh [workspace] $ /bin/sh -xe /tmp/hudson3133213269494249771.sh + PATH=/usr/bin:/bin:/usr/local/bin ./run_all_tests.sh NOTE: nosetests required for unit tests! FAIL: Please keep commit message lines to <80 columns Checking for PEP8 regressions... ./chirp/platform.py:255:80: E501 line too long (82 > 79 characters) ./chirp/ui/mainapp.py:1909:80: E501 line too long (82 > 79 characters) ./chirp/ui/mainapp.py:1965:80: E501 line too long (82 > 79 characters)
real 0m9.278s user 0m9.032s sys 0m0.036s ================================================ Tests FAILED: style tests, unit tests Build step 'Execute shell' marked build as failure Email was triggered for: Failure Sending email for trigger: Failure
Tested changes:
[Michael Lippold michael.lippold@smartacus.net] [id880] Adjust multiplier table implementation to prevent CL ERR when updating radio. Fixes #4871
[Tom Hayward tom@tomh.us] [id880] Fix typo in charset definition. #281
[Tom Hayward tom@tomh.us] [thf6a] Support full charset (ASCII). Fixes #141
[Tom Hayward tom@tomh.us] [id880] Support full charset. Fixes #281
[Tom Hayward tom@tomh.us] [vx5] Support full charset (ASCII). Fixes #292
[Tom Hayward tom@tomh.us] [id31a] set used bit when creating new memory, clear when deleting. Fixes #269
[Tom Hayward tom@tomh.us] Support PyGTK < 2.22 in bank edit. Fixes #231
[Tom Hayward tom@tomh.us] [d710] [v71] [d72] Fix tone list (not all tones are supported). Fixes #212
[Dan Smith dsmith@danplanet.com] [vx7] Fix setting memory power levels on 220MHz band Fixes #214
[Dan Smith dsmith@danplanet.com] fips: Pennsylvania FIPS code was wrong. #117
[Marco Filippi iz3gme.marco@gmail.com] Consider lower bound frequency of each valid_band as valid Fix bug #181
[Tom Hayward tom@tomh.us] tmd700: allow 8-char names. Fixes #176
[Dan Smith dsmith@danplanet.com] Fix the "blind deletion" problem, as well as properly direct copy/paste Fixes #172
[David Griffith dave@661.org] Bug #155 fix: VX-7 1.25m power levels
[David Griffith dave@661.org] New INSTALL and README files Fixes #122
[Tom Hayward tom@tomh.us] thd72: only use hardware flow on OS X. Fixes #166
[Marco Filippi iz3gme.marco@gmail.com] [FT817] Tone freq not set correctly Same as #88 for FT857, to avoid code duplication fix code have been moved from ft857 to its ancestor class Fix bug #163
[Tom Hayward tom@tomh.us] Fix Mac .app so paths with spaces work. Fixes Bug #145
Full log:
Started by an SCM change Building in workspace /var/lib/jenkins/jobs/chirp-test/workspace [workspace] $ hg showconfig paths.default [workspace] $ hg pull --rev default [workspace] $ hg update --clean --rev default 1 files updated, 0 files merged, 0 files removed, 0 files unresolved [workspace] $ hg log --rev . --template {node} [workspace] $ hg log --rev . --template {rev} [workspace] $ hg log --rev b4000bed33bd10e80c07ffad79e3271ceae41be4 [workspace] $ hg log --template "<changeset node='{node}' author='{author|xmlescape}' rev='{rev}' date='{date}'><msg>{desc|xmlescape}</msg><added>{file_adds|stringify|xmlescape}</added><deleted>{file_dels|stringify|xmlescape}</deleted><files>{files|stringify|xmlescape}</files><parents>{parents}</parents></changeset>\n" --rev default:0 --follow --prune b4000bed33bd10e80c07ffad79e3271ceae41be4 No emails were triggered. [workspace] $ /bin/sh -xe /tmp/hudson2544381166348191403.sh [workspace] $ /bin/sh -xe /tmp/hudson5165683195967400118.sh + PATH=/usr/bin:/bin:/usr/local/bin ./run_all_tests.sh NOTE: nosetests required for unit tests! FAIL: Please keep commit message lines to <80 columns Checking for PEP8 regressions... ./chirp/platform.py:255:80: E501 line too long (82 > 79 characters) ./chirp/ui/mainapp.py:1909:80: E501 line too long (82 > 79 characters) ./chirp/ui/mainapp.py:1965:80: E501 line too long (82 > 79 characters)
real 0m9.108s user 0m8.924s sys 0m0.032s ================================================ Tests FAILED: style tests, unit tests Build step 'Execute shell' marked build as failure Email was triggered for: Failure Sending email for trigger: Failure
Tested changes:
[Imuli i@imu.li] [uv5r] UV-82 use ASCII charset Partially fixes #5227
The UV-82 supports all of ASCII (with two replacements).
I imagine other Baofeng radios do also, so this provides an easy means to specify supported characters for different models.
[Tom Hayward tom@tomh.us] [id880] Fix typo in charset definition. #281
[Tom Hayward tom@tomh.us] [thf6a] Support full charset (ASCII). Fixes #141
[Tom Hayward tom@tomh.us] [id880] Support full charset. Fixes #281
[Tom Hayward tom@tomh.us] [vx5] Support full charset (ASCII). Fixes #292
[Tom Hayward tom@tomh.us] [id31a] set used bit when creating new memory, clear when deleting. Fixes #269
[Tom Hayward tom@tomh.us] Support PyGTK < 2.22 in bank edit. Fixes #231
[Tom Hayward tom@tomh.us] [d710] [v71] [d72] Fix tone list (not all tones are supported). Fixes #212
[Dan Smith dsmith@danplanet.com] [vx7] Fix setting memory power levels on 220MHz band Fixes #214
[Dan Smith dsmith@danplanet.com] fips: Pennsylvania FIPS code was wrong. #117
[Marco Filippi iz3gme.marco@gmail.com] Consider lower bound frequency of each valid_band as valid Fix bug #181
[Tom Hayward tom@tomh.us] tmd700: allow 8-char names. Fixes #176
[Dan Smith dsmith@danplanet.com] Fix the "blind deletion" problem, as well as properly direct copy/paste Fixes #172
[David Griffith dave@661.org] Bug #155 fix: VX-7 1.25m power levels
[David Griffith dave@661.org] New INSTALL and README files Fixes #122
[Tom Hayward tom@tomh.us] thd72: only use hardware flow on OS X. Fixes #166
[Marco Filippi iz3gme.marco@gmail.com] [FT817] Tone freq not set correctly Same as #88 for FT857, to avoid code duplication fix code have been moved from ft857 to its ancestor class Fix bug #163
[Tom Hayward tom@tomh.us] Fix Mac .app so paths with spaces work. Fixes Bug #145
Full log:
Started by an SCM change Building in workspace /var/lib/jenkins/jobs/chirp-test/workspace [workspace] $ hg showconfig paths.default [workspace] $ hg pull --rev default [workspace] $ hg update --clean --rev default 1 files updated, 0 files merged, 0 files removed, 0 files unresolved [workspace] $ hg log --rev . --template {node} [workspace] $ hg log --rev . --template {rev} [workspace] $ hg log --rev 22db381cf682befc8409d756170a6762e05229db [workspace] $ hg log --template "<changeset node='{node}' author='{author|xmlescape}' rev='{rev}' date='{date}'><msg>{desc|xmlescape}</msg><added>{file_adds|stringify|xmlescape}</added><deleted>{file_dels|stringify|xmlescape}</deleted><files>{files|stringify|xmlescape}</files><parents>{parents}</parents></changeset>\n" --rev default:0 --follow --prune 22db381cf682befc8409d756170a6762e05229db No emails were triggered. [workspace] $ /bin/sh -xe /tmp/hudson5727640608277937245.sh [workspace] $ /bin/sh -xe /tmp/hudson4487680953742221178.sh + PATH=/usr/bin:/bin:/usr/local/bin ./run_all_tests.sh NOTE: nosetests required for unit tests! Patch 'tip' is OK Checking for PEP8 regressions... ./chirp/platform.py:255:80: E501 line too long (82 > 79 characters) ./chirp/ui/mainapp.py:1909:80: E501 line too long (82 > 79 characters) ./chirp/ui/mainapp.py:1965:80: E501 line too long (82 > 79 characters)
real 0m9.575s user 0m9.356s sys 0m0.048s ================================================ Tests FAILED: unit tests Build step 'Execute shell' marked build as failure Email was triggered for: Failure Sending email for trigger: Failure
Tested changes:
[Dan Smith dsmith@danplanet.com] Remove unnecessary test tools
After #5237 these are no longer necessary and duplicative.
[Dan Smith dsmith@danplanet.com] Add tox.ini for running tests
Tox handles venv creation and management of setup install for running tests. Since we have a venv required already, this just makes sense.
Now, instead of:
./run_all_tests.sh
You can just run:
tox
Assuming you have tox installed (pip install tox). You can run individual test groups like this:
tox -e unit tox -e driver
and still pass arguments to the driver tests per usual with:
tox -e driver -- -d Icom_IC-2200H
I'll be working to make the build system use tox for running tests with better isolation and hopefully more repeatability.
#5237
[Dan Smith dsmith@danplanet.com] Some fixes to run_tests
First, in r2787 we made the BruteForce test do RadioFeatures validation on a memory, avoiding a failure if the memory did not validate. For some reason we still required that radio.validate_memory() pass for any of those tests, which doesn't allow for the radio to ban quirks in the tests. This makes us not fail if the radio rightly fails to validate.
Second, we were modifying the memory in do_mode() so that once we passed FM in the mode sequence we were always trying freq+100Mhz. Depending on how we ran the tests (i.e. which order we hit the modes) this would sometimes fail. This makes us copy the original memory so we're stable throughout the modes.
Related to #5237
[Dan Smith dsmith@danplanet.com] [icq7] Add validate_memory() for sub-30MHz AM requirement
This driver has a requirement that anything under 30MHz be AM. Validate that in validate_memory() so we can properly handle cases where that won't work.
Found in the pursuit of #5237
[Dan Smith dsmith@danplanet.com] Remove dedicated silly rptbuild setup handler
This just makes setup.py more complicated and there's no reason to build this as a separate package. Just build it as an additional script on the main default build.
Needed for #5237
[Dan Smith dsmith@danplanet.com] Let make_supported.py take an output filename as the first argument
Needed for #5237
[Dan Smith dsmith@danplanet.com] More pep8 compliance fixes
#2355
[Tom Hayward tom@tomh.us] [id880] Fix typo in charset definition. #281
[Tom Hayward tom@tomh.us] [thf6a] Support full charset (ASCII). Fixes #141
[Tom Hayward tom@tomh.us] [id880] Support full charset. Fixes #281
[Tom Hayward tom@tomh.us] [vx5] Support full charset (ASCII). Fixes #292
[Tom Hayward tom@tomh.us] [id31a] set used bit when creating new memory, clear when deleting. Fixes #269
[Tom Hayward tom@tomh.us] Support PyGTK < 2.22 in bank edit. Fixes #231
[Tom Hayward tom@tomh.us] [d710] [v71] [d72] Fix tone list (not all tones are supported). Fixes #212
[Dan Smith dsmith@danplanet.com] [vx7] Fix setting memory power levels on 220MHz band Fixes #214
[Dan Smith dsmith@danplanet.com] fips: Pennsylvania FIPS code was wrong. #117
[Marco Filippi iz3gme.marco@gmail.com] Consider lower bound frequency of each valid_band as valid Fix bug #181
[Tom Hayward tom@tomh.us] tmd700: allow 8-char names. Fixes #176
[Dan Smith dsmith@danplanet.com] Fix the "blind deletion" problem, as well as properly direct copy/paste Fixes #172
[David Griffith dave@661.org] Bug #155 fix: VX-7 1.25m power levels
[David Griffith dave@661.org] New INSTALL and README files Fixes #122
[Tom Hayward tom@tomh.us] thd72: only use hardware flow on OS X. Fixes #166
[Marco Filippi iz3gme.marco@gmail.com] [FT817] Tone freq not set correctly Same as #88 for FT857, to avoid code duplication fix code have been moved from ft857 to its ancestor class Fix bug #163
[Tom Hayward tom@tomh.us] Fix Mac .app so paths with spaces work. Fixes Bug #145
Full log:
Started by an SCM change Building in workspace /var/lib/jenkins/jobs/chirp-test/workspace [workspace] $ hg showconfig paths.default [workspace] $ hg pull --rev default [workspace] $ hg update --clean --rev default 7 files updated, 0 files merged, 2 files removed, 0 files unresolved [workspace] $ hg log --rev . --template {node} [workspace] $ hg log --rev . --template {rev} [workspace] $ hg log --rev 9b7e86dfaf1bb7fc697b99e94a744fb136ad536d [workspace] $ hg log --template "<changeset node='{node}' author='{author|xmlescape}' rev='{rev}' date='{date}'><msg>{desc|xmlescape}</msg><added>{file_adds|stringify|xmlescape}</added><deleted>{file_dels|stringify|xmlescape}</deleted><files>{files|stringify|xmlescape}</files><parents>{parents}</parents></changeset>\n" --rev default:0 --follow --prune 9b7e86dfaf1bb7fc697b99e94a744fb136ad536d No emails were triggered. [workspace] $ /bin/sh -xe /tmp/hudson2210702522770226416.sh [workspace] $ /bin/sh -xe /tmp/hudson5692053982839956029.sh + PATH=/usr/bin:/bin:/usr/local/bin ./run_all_tests.sh /tmp/hudson5692053982839956029.sh: 2: /tmp/hudson5692053982839956029.sh: ./run_all_tests.sh: not found Build step 'Execute shell' marked build as failure Email was triggered for: Failure Sending email for trigger: Failure
participants (1)
-
Build System