[chirp_devel] Adding Settings support to the VX8-R

Keith Williamson
Wed Jun 7 08:00:41 PDT 2017


Hi,

Attached is patch "vx8-4881-4.patch" which corrects the line > 79 chars
violation. Also, I discovered that I left out the pad value of one of the
new #seekto format strings I had added (had 0x%0X instead of 0x%04X).
That's also corrected with this patch.

Cheers,

Keith


On Tue, Jun 6, 2017 at 7:32 PM, Keith Williamson <hkwilliamson at gmail.com>
wrote:

> Ah, I see. I figured if a patch "failed" the build it was yanked.
>
> No problem. I'll send a patch that just fixes the PEP-8 error.
>
> On Tue, Jun 6, 2017 at 6:14 PM, Dan Smith via chirp_devel <
> chirp_devel at intrepid.danplanet.com> wrote:
>
>> > Resubmitting "vx8-4881-2.patch" as "vx8-4881-3.patch" because of PEP-8
>> > error due to line length of 80. My bad..thought max line length was 80.
>> > Ensured that all lines are strictly less than 80 chars.
>>
>> It's already in tree, so I can't apply this unless I revert the other
>> one. Can you just send a teensy incremental change rebased on the tip?
>>
>> Thanks!
>>
>> --Dan
>> _______________________________________________
>> chirp_devel mailing list
>> chirp_devel at intrepid.danplanet.com
>> http://intrepid.danplanet.com/mailman/listinfo/chirp_devel
>> Developer docs: http://chirp.danplanet.com/projects/chirp/wiki/Developers
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://intrepid.danplanet.com/pipermail/chirp_devel/attachments/20170607/af3cdba3/attachment-0001.html 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: vx8-4881-4.patch
Type: text/x-patch
Size: 1094 bytes
Desc: not available
Url : http://intrepid.danplanet.com/pipermail/chirp_devel/attachments/20170607/af3cdba3/attachment-0001.bin 


More information about the chirp_devel mailing list