On Apr 6, 2015 3:43 PM, "Dan Smith" <dsmith@danplanet.com> wrote:

Hi Charles,

> Wrapped up the PEP-8 cleanups as much as possible.  There was one where
> it complained about under indenting, but that's nit picking to the
> extreme.  Caught a few minor issues as well in the process, too.

Was this supposed to be a patch against your last patch? If so, I can't
get this to apply cleanly against that.

Either way, until your patch makes it into the tree, you should be
sending patches rebased on current tip each time. A patch against a
patch requires me to manually merge them to pass the checks.

Could you maybe send a clean patch against current tip that adds the
driver with everything passing the checks?

Really looking forward to having this in the tree, as I know it's
desired by a lot of folks. Thanks!

--Dan


_______________________________________________
chirp_devel mailing list
chirp_devel@intrepid.danplanet.com
http://intrepid.danplanet.com/mailman/listinfo/chirp_devel
Developer docs: http://chirp.danplanet.com/projects/chirp/wiki/Developers