kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #44991
Re: schematic.cpp includes sim/ngspice.h and consequently sharedspice.h
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
>
Follow ups
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