kicad-developers team mailing list archive

[Wayne Stambaugh <stambaughw@xxxxxxxxx>]

Oliver,

Please be patient.  I realize that I am falling way behind with the
patch review (let alone my own work) but I am doing my best.

I'm OK with this behavior.  If no one else objects and someone can find
time to review the patch, I'm fine with it being committed.

Cheers,

Wayne

On 2/9/2017 8:30 AM, Oliver Walters wrote:
> Has anyone had a change to look at this one? :)
> 
> On Fri, Feb 3, 2017 at 12:37 PM, Chris Pavlina <pavlina.chris@xxxxxxxxx
> <mailto:pavlina.chris@xxxxxxxxx>> wrote:
> 
>     I don't have a chance to test this right now, but I REALLY like the
>     idea. As a (non-official) library maintainer I'd use the hell out of
>     this.
> 
>     On Fri, Feb 03, 2017 at 10:40:30AM +1100, Oliver Walters wrote:
>     > bump :)
>     >
>     > On Thu, Feb 2, 2017 at 1:28 AM, Oliver Walters <
>     > oliver.henry.walters@xxxxxxxxx
>     <mailto:oliver.henry.walters@xxxxxxxxx>> wrote:
>     >
>     > > Hi all,
>     > >
>     > > Currently a lot of the .pretty libs have the library name (e.g.
>     > > "Socket_Strips.pretty") duplicated (or partly duplicated) in the
>     footprint
>     > > name.
>     > >
>     > > This is (mainly) required to allow the footprint filtering in
>     CvPCB to
>     > > match the filters defined for components.
>     > >
>     > > One case where this is very annoying is for the various
>     > > Connectors_xxx.pretty libraries (JST / Hirose / Harwin / etc).
>     To reduce
>     > > the length of the filenames, a while ago the decision was made
>     to remove
>     > > the Connector_ prefix from the individual filenames.
>     > >
>     > > However, this now means that generic connector symbols have to have
>     > > multiple footprint filters to match the various connector
>     footprints.
>     > >
>     > > Looking through the code I saw that originally there was the
>     option to
>     > > include the LibraryName: in the search string (which is what I
>     wanted to
>     > > implement). However it was commented out as at some time a
>     developer has
>     > > decided that this behaviour was "messy".
>     > >
>     > > I have attached a patch that does the following:
>     > >
>     > > 1. If a footprint filter contains a : (colon) character, then
>     the filter
>     > > is matched against the pattern <LibName>:<FootprintName>
>     > > 2. If there is *no* : (colon) character present, then it is matched
>     > > against <FootprintName> (this is the current behaviour, and so
>     will not
>     > > break any current footprint filters.
>     > > 3. The same behaviour applies for the manual filter string in CvPCB
>     > >
>     > > If merged, this will allow the .kicad_mod files to have
>     significantly
>     > > shorter names and we can use the footprint association filters more
>     > > intelligently.
>     > >
>     > > Cheers,
>     > > Oliver
>     > >
> 
> 
> 
> 
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp
>