kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #44992
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
-
schematic.cpp includes sim/ngspice.h and consequently sharedspice.h
From: Eeli Kaikkonen, 2021-03-18
-
Re: schematic.cpp includes sim/ngspice.h and consequently sharedspice.h
From: Wayne Stambaugh, 2021-03-18
-
Re: schematic.cpp includes sim/ngspice.h and consequently sharedspice.h
From: Jon Evans, 2021-03-18
-
Re: schematic.cpp includes sim/ngspice.h and consequently sharedspice.h
From: Wayne Stambaugh, 2021-03-18
-
Re: schematic.cpp includes sim/ngspice.h and consequently sharedspice.h
From: Jon Evans, 2021-03-18
-
Re: schematic.cpp includes sim/ngspice.h and consequently sharedspice.h
From: Seth Hillbrand, 2021-03-18
-
Re: schematic.cpp includes sim/ngspice.h and consequently sharedspice.h
From: Mark Roszko, 2021-03-18
-
Re: schematic.cpp includes sim/ngspice.h and consequently sharedspice.h
From: Seth Hillbrand, 2021-03-19
-
Re: schematic.cpp includes sim/ngspice.h and consequently sharedspice.h
From: Jon Evans, 2021-03-19
-
Re: schematic.cpp includes sim/ngspice.h and consequently sharedspice.h
From: Nick Østergaard, 2021-03-19
-
Re: schematic.cpp includes sim/ngspice.h and consequently sharedspice.h
From: Wayne Stambaugh, 2021-03-19