Hi Jim,
Thank you for the feedback. If you're able to test this against your own KG-UV920P-A, that would be great.
Unless I hear otherwise, I'll create a new patch with those two corrections when I have a chance this evening. Would the convention be to share it in this same email thread, or start a new one?
Thanks, Matt KF7TAL
On Mon, Feb 14, 2022 at 7:59 AM Jim Unroe rock.unroe@gmail.com wrote:
Matthew and Chirp-Devel,
Nice! Thank you for developing this driver. I am going to try to find time today to dig my Wouxun KG-UV920P-A out of mothballs and give this a go. It looks great!
Below are a couple of things that you might have to address before it gets accepted (unless Dan takes care of it for you):
- I think it was requested that the commit message be broken up into
lines of less than 80 characters. I could be wrong but something got me into the habit of doing it. ;-)
- I don't see where the driver was added to the cpep8.manifest. It
needs to be listed in the manifest so that the build system will know to perform a pep8 style check on it during the build process. The good news is that your driver passes the style check with flying colors.
Thank you again for your work developing this driver.
Jim KC9HI
On Sun, Feb 13, 2022 at 9:46 PM Matthew Handley via chirp_devel chirp_devel@intrepid.danplanet.com wrote:
chirp_devel mailing list chirp_devel@intrepid.danplanet.com http://intrepid.danplanet.com/mailman/listinfo/chirp_devel Developer docs: