← Back to team overview

kicad-developers team mailing list archive

Re: schematic.cpp includes sim/ngspice.h and consequently sharedspice.h

 

I think it's fine for macOS!

On Fri, Mar 19, 2021, 7:04 AM Wayne Stambaugh <stambaughw@xxxxxxxxx> wrote:

> It seems as though we are heading towards a consensus that making the
> simulator build mandatory unless there is some strenuous objection to
> doing so by one of our package developers.  I still want to give
> everyone a chance to reply before I make the change.  In the mean time,
> I think I'm  going to split out the ngspice settings because it will be
> a bit cleaner and it will fix the immediate build issues.
>
> Wayne
>
> On 3/19/21 4:06 AM, Nick Østergaard wrote:
> > I would be fine with permanently enabling the ngspice integration. It
> > has historically not been that complicated to support, I mean build
> > wise, and I don't think it makes a lot of difference for rebuilds, which
> > is where it could be nice to disable some things, like scripting
> sometimes.
> >
> > Speaking from experience on windows, linux and mac. At least I don't
> > remember any issues with it on newer releases of ngspice, and if a
> > problem crops of for some reason I am sure we can workaround it and
> > Holger is very responsive on requests from us.
> >
> > On Fri, 19 Mar 2021 at 04:14, Jon Evans <jon@xxxxxxxxxxxxx
> > <mailto:jon@xxxxxxxxxxxxx>> wrote:
> >
> >     I agree with Seth.  The longer a feature is part of KiCad, the more
> >     people will refer to it in forum posts, tutorials, etc.   This
> >     feature has now existed for plenty long enough for people to expect
> >     it.  If there are no packaging reasons to do otherwise, I support it
> >     being included in all builds.
> >
> >     -Jon
> >
> >     On Thu, Mar 18, 2021 at 10:49 PM Seth Hillbrand <seth@xxxxxxxxxxxxx
> >     <mailto:seth@xxxxxxxxxxxxx>> wrote:
> >
> >         I agree that this is one area where we can improve the user
> >         experience.
> >
> >         By first-class, I mean only that it is an integral part of KiCad
> >         that we develop and support.  We are no longer in the testing
> >         phase where it would make sense to have a conditional
> compilation.
> >
> >         -Seth
> >
> >         On Thu, Mar 18, 2021 at 4:51 PM Mark Roszko
> >         <mark.roszko@xxxxxxxxx <mailto:mark.roszko@xxxxxxxxx>> wrote:
> >
> >             You say it's first class but the spice gui needs alot of
> >             loving. The tool framework is also ducktaped poorly into it.
> >
> >             On Thu, Mar 18, 2021 at 7:14 PM Seth Hillbrand
> >             <seth@xxxxxxxxxxxxx <mailto:seth@xxxxxxxxxxxxx>> wrote:
> >
> >                 Do we still need spice as a build option?  It would be
> >                 nice to bring down the number of permutations out there
> >                 and the SPICE simulator is really a first-class KiCad
> >                 citizen nowadays.
> >
> >                 -S
> >
> >                 On Thu, Mar 18, 2021 at 4:04 PM Jon Evans
> >                 <jon@xxxxxxxxxxxxx <mailto:jon@xxxxxxxxxxxxx>> wrote:
> >
> >                     Saving in the project file is fine, I just mean if
> >                     you split out NGSPICE_SIMULATOR_SETTINGS to a
> >                     different file (not sim/ngspice.h) with no
> >                     dependencies on ngspice itself, the settings class
> >                     can be built without ngspice.
> >
> >                     On Thu, Mar 18, 2021 at 6:57 PM Wayne Stambaugh
> >                     <stambaughw@xxxxxxxxx <mailto:stambaughw@xxxxxxxxx>>
> >                     wrote:
> >
> >                         I thought about that but in what context does a
> >                         simulation make sense
> >                         out side of a schematic or a netlist generated
> >                         from a schematic?  I
> >                         saved the ngspice simulator settings in the
> >                         project file because that is
> >                         the proper scope of the setting.  I really don't
> >                         see a more logical
> >                         place to save the setting.   I suppose I could
> >                         split out the simulator
> >                         settings code but it would be awkward at best.
> >
> >                         On 3/18/21 6:42 PM, Jon Evans wrote:
> >                         > Wayne, I haven't checked this code carefully
> >                         but I'd recommend building
> >                         > the settings always, and moving the #ifdef to
> >                         a different level (i.e.
> >                         > make the settings not depend on ngspice)
> >                         >
> >                         > That way if the same settings files are shared
> >                         between a build with
> >                         > ngspice and a build without, they won't get
> >                         thrown away.
> >                         >
> >                         > On Thu, Mar 18, 2021 at 6:37 PM Wayne
> >                         Stambaugh <stambaughw@xxxxxxxxx
> >                         <mailto:stambaughw@xxxxxxxxx>
> >                         > <mailto:stambaughw@xxxxxxxxx
> >                         <mailto:stambaughw@xxxxxxxxx>>> wrote:
> >                         >
> >                         >     My bad.  I forgot the simulator was a
> >                         build option so I will have to
> >                         >     #ifdef the offending settings code.
> >                         >
> >                         >     - Wayne
> >                         >
> >                         >     On 3/18/21 6:02 PM, Eeli Kaikkonen wrote:
> >                         >     > I have disabled ngspice in cmake
> >                         settings but in a recent commit by
> >                         >     > Wayne #include <sim/ngspice.h>, which in
> >                         turn includes
> >                         >     > ngspice/sharedspice.h, was added to
> >                         schematic.cpp which makes
> >                         >     > compilation fail apparently because I
> >                         don't have ngspice at all.
> >                         >     >
> >                         >     > Eeli Kaikkonen
> >                         >     >
> >                         >     >
> >                         _______________________________________________
> >                         >     > Mailing list:
> >                         https://launchpad.net/~kicad-developers
> >                         <https://launchpad.net/~kicad-developers>
> >                         >     <https://launchpad.net/~kicad-developers
> >                         <https://launchpad.net/~kicad-developers>>
> >                         >     > Post to     :
> >                         kicad-developers@xxxxxxxxxxxxxxxxxxx
> >                         <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> >                         >
> >                          <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
> >                         <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>>
> >                         >     > Unsubscribe :
> >                         https://launchpad.net/~kicad-developers
> >                         <https://launchpad.net/~kicad-developers>
> >                         >     <https://launchpad.net/~kicad-developers
> >                         <https://launchpad.net/~kicad-developers>>
> >                         >     > More help   :
> >                         https://help.launchpad.net/ListHelp
> >                         <https://help.launchpad.net/ListHelp>
> >                         >     <https://help.launchpad.net/ListHelp
> >                         <https://help.launchpad.net/ListHelp>>
> >                         >     >
> >                         >
> >                         >
> >                          _______________________________________________
> >                         >     Mailing list:
> >                         https://launchpad.net/~kicad-developers
> >                         <https://launchpad.net/~kicad-developers>
> >                         >     <https://launchpad.net/~kicad-developers
> >                         <https://launchpad.net/~kicad-developers>>
> >                         >     Post to     :
> >                         kicad-developers@xxxxxxxxxxxxxxxxxxx
> >                         <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> >                         >
> >                          <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
> >                         <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>>
> >                         >     Unsubscribe :
> >                         https://launchpad.net/~kicad-developers
> >                         <https://launchpad.net/~kicad-developers>
> >                         >     <https://launchpad.net/~kicad-developers
> >                         <https://launchpad.net/~kicad-developers>>
> >                         >     More help   :
> >                         https://help.launchpad.net/ListHelp
> >                         <https://help.launchpad.net/ListHelp>
> >                         >     <https://help.launchpad.net/ListHelp
> >                         <https://help.launchpad.net/ListHelp>>
> >                         >
> >
> >                     _______________________________________________
> >                     Mailing list:
> >                     https://launchpad.net/~kicad-developers
> >                     <https://launchpad.net/~kicad-developers>
> >                     Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> >                     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> >                     Unsubscribe :
> >                     https://launchpad.net/~kicad-developers
> >                     <https://launchpad.net/~kicad-developers>
> >                     More help   : https://help.launchpad.net/ListHelp
> >                     <https://help.launchpad.net/ListHelp>
> >
> >
> >
> >                 --
> >                 KiCad Services Corporation Logo
> >                 Seth Hillbrand
> >                 *Lead Developer*
> >                 +1-530-302-5483‬
> >                 Long Beach, CA
> >                 www.kipro-pcb.com <https://www.kipro-pcb.com/>
> >                 info@xxxxxxxxxxxxx <mailto:info@xxxxxxxxxxxxx>
> >
> >                 _______________________________________________
> >                 Mailing list: https://launchpad.net/~kicad-developers
> >                 <https://launchpad.net/~kicad-developers>
> >                 Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> >                 <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> >                 Unsubscribe : https://launchpad.net/~kicad-developers
> >                 <https://launchpad.net/~kicad-developers>
> >                 More help   : https://help.launchpad.net/ListHelp
> >                 <https://help.launchpad.net/ListHelp>
> >
> >
> >
> >             --
> >             Mark
> >
> >
> >
> >         --
> >         KiCad Services Corporation Logo
> >         Seth Hillbrand
> >         *Lead Developer*
> >         +1-530-302-5483‬
> >         Long Beach, CA
> >         www.kipro-pcb.com <https://www.kipro-pcb.com/>
> >         info@xxxxxxxxxxxxx <mailto:info@xxxxxxxxxxxxx>
> >
> >     _______________________________________________
> >     Mailing list: https://launchpad.net/~kicad-developers
> >     <https://launchpad.net/~kicad-developers>
> >     Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> >     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> >     Unsubscribe : https://launchpad.net/~kicad-developers
> >     <https://launchpad.net/~kicad-developers>
> >     More help   : https://help.launchpad.net/ListHelp
> >     <https://help.launchpad.net/ListHelp>
> >
> >
> > _______________________________________________
> > 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
> >
>
> _______________________________________________
> 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
>

References