kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #24347
Re: PATCH: 3D resolver
-
To:
kicad-developers@xxxxxxxxxxxxxxxxxxx
-
From:
Wayne Stambaugh <stambaughw@xxxxxxxxx>
-
Date:
Fri, 22 Apr 2016 10:52:47 -0400
-
In-reply-to:
<CAHBNN+N1tkqJGOpzbuGBab=Ba93rpVka29G+AxHBrsRrKAPxUQ@mail.gmail.com>
-
User-agent:
Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.0
Cirilo,
This patch no longer applies cleanly. Would you please fix it an repost it.
Thanks,
Wayne
On 4/20/2016 5:24 AM, Cirilo Bernardo wrote:
> The attached patch does the following:
>
> 1. Ensures ${KISYS3DMOD} is in the resolver's path list even if
> it is not defined in KiCad's internal path list (Configure Paths).
> This fixes a bug which was described on the dev list but not
> reported to the bug tracker.
>
> 2. ENV VARs paths defined by the shell take precedence over
> what is defined within KiCad; this is consistent with typical
> UNIX use: ENV_VAR=something pcbnew test.kicad_pcb
>
> 3. If a user defines a path via ${ENV_VAR}/som/file.wrl where
> ENV_VAR is an arbitrary environment variable (as opposed to
> KISYS3DMOD and others defined via KiCad's Configure Paths)
> then that ENV_VAR is automatically added to the list of
> base search paths. Previous behavior was to only include
> KISYS3DMOD and ENV_VARS defined via Configure Paths.
>
> 4. General code cleanup removing functions which were not
> necessary since wxFileName already performed the required
> tasks.
>
> 5. Fix a bug in the file modification time check where a path to
> a non-existent file fires a wxASSERT within wxWidgets.
>
> - Cirilo
>
>
>
> _______________________________________________
> 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
>
Follow ups
References