I think this is good.  I also prefer the radio source under the chirp tree, fwiw I would prefer also source (chirpui) under this tree as well so as so to make indexed source better.

Either drivers or radios is fine.  I would lean towards radio only because the classes in this folder would mostly inherit from radio. Either way good change imo.

Date: Wed, 25 Feb 2015 08:55:50 -0800
From: dsmith@danplanet.com
To: chirp_devel@intrepid.danplanet.com
Subject: Re: [chirp_devel] [PATCH 12/9] Move radios into their own directory (#2351)

> diff --git a/radio/__init__.py b/radio/__init__.py
> new file mode 100644
> index 0000000..726217b
> --- /dev/null
> +++ b/radio/__init__.py
 
This isn't okay, as it will generate a top-level module called 'radio'
if the package is installed on the system. I.e. it would look like this:
 
 >>> from radio import ic2820
 
This needs to be under chirp/, and I'd prefer "drivers".
 
Note that your chirpc rename patch did not work, as mercurial didn't
properly generate a follow-able rename change and instead just
deleted/added the old/new file(s). So, that means someone who uses
mercurial will have to generate that patch (jab).
 
Before we do, I'd like to ask the group whether they think this is
really something we need to change. I agree that chirp/ is very
cluttered, but I'm also not sure that it really matters. This massive
rename is kindof a hassle (and will break my muscle memory).
 
So: should we do this?
 
--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