[chirp_devel] Build test results: Failure

Jens J.
Fri Oct 18 19:51:54 PDT 2013


ok try this patch attached...

have you looked into pre-commit test triggers?



On Friday, October 18, 2013 8:43 PM, Dan Smith <dsmith at danplanet.com> wrote:
 
> ok, looked at vx-8 code, and think the fix is simply returning 
> name.rstrip() !!

Yep.

> repatched. both patches attached.

These already went into the tree, which is how the test bot got them.
Can you send another small patch on top of the current tip of the tree
that just fixes the problem?

Ideally, I would run the tests before I apply/push, or push proposed
changesets to the bot and let it apply them only if the tests run, but
I'm not quite there yet :)


> (I'm starting to see why mq is useful - with ability to pop, edit,
> refresh, and repush patches, edit refresh, etc).

Yeah :)

-- 
Dan Smith
www.danplanet.com
KK7DS


_______________________________________________
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/20131018/c8082a11/attachment-0001.html 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: vx3-banks-names-fix.patch
Type: application/octet-stream
Size: 1312 bytes
Desc: not available
Url : http://intrepid.danplanet.com/pipermail/chirp_devel/attachments/20131018/c8082a11/attachment-0001.obj 


More information about the chirp_devel mailing list