Skip to content
This repository has been archived by the owner on Dec 2, 2019. It is now read-only.

update ngspice #257

Closed
adamwolf opened this issue Jan 5, 2019 · 15 comments · Fixed by #286
Closed

update ngspice #257

adamwolf opened this issue Jan 5, 2019 · 15 comments · Fixed by #286

Comments

@adamwolf
Copy link
Contributor

adamwolf commented Jan 5, 2019

@nickoe made a PR #256 for updating to the latest ngspice tag. It didn't work, and fails with:

03:08:47 default: [ 57%] Building CXX object eeschema/CMakeFiles/eeschema_kiface.dir/sim/ngspice.cpp.o 03:08:48 default: In file included from /vagrant/build/kicad/src/kicad/eeschema/sim/ngspice.cpp:28: 03:08:48 default: /vagrant/build/kicad/src/kicad/eeschema/sim/ngspice.h:31:10: fatal error: 'ngspice/sharedspice.h' file not found 03:08:48 default: #include <ngspice/sharedspice.h> 03:08:48 default: ^ 03:08:49 default: 1 error generated. 03:08:49 default: make[6]: *** [eeschema/CMakeFiles/eeschema_kiface.dir/sim/ngspice.cpp.o] Error 1

Let's fix this.

@nickoe
Copy link
Contributor

nickoe commented Jan 5, 2019

Where is the full log?

@adamwolf
Copy link
Contributor Author

adamwolf commented Jan 6, 2019

@adamwolf
Copy link
Contributor Author

@nickoe I tried updating ngspice again. Here's the log.

https://builder.wayneandlayne.com/job/kicad-mac-builder-10.14-upload/249/consoleFull

@nickoe
Copy link
Contributor

nickoe commented Jun 24, 2019

This is the error:

[ 53%] Building CXX object eeschema/CMakeFiles/eeschema_kiface_objects.dir/sim/ngspice.cpp.o
In file included from /vagrant/build/kicad/src/kicad/eeschema/sim/ngspice.cpp:28:
/vagrant/build/kicad/src/kicad/eeschema/sim/ngspice.h:31:10: fatal error: 'ngspice/sharedspice.h' file not found
#include <ngspice/sharedspice.h>
         ^~~~~~~~~~~~~~~~~~~~~~~
1 error generated.

It does look like --with-ngshared is enabled and it does build some sharedspice.c, but maybe the header is not installed to the correct location?

But then I see:
/usr/bin/install -c -m 644 noisedef.h numcards.h numconst.h numenum.h numgen.h numglobs.h onedev.h onemesh.h opdefs.h optdefs.h optndefs.h outpdefs.h plot.h pnode.h profile.h pssdefs.h pzdefs.h randnumb.h sen2defs.h sensdefs.h sensgen.h sharedspice.h sim.h smpdefs.h sperror.h spmatrix.h stringskip.h stringutil.h suffix.h swec.h tfdefs.h trandefs.h trcvdefs.h tskdefs.h twodev.h twomesh.h typedefs.h wordlist.h 1-f-code.h FastNorm3.h '/vagrant/build/ngspice-dest/include/ngspice'

Where it do look like it is installed to /vagrant/build/ngspice-dest/include/ngspice/.

But it looks like the issue may be that above install location and the location used for cmake of kicad is:
-DNGSPICE_INCLUDE_DIR=/vagrant/build/ngspice-dest/share/ngspice/include/ and does not match above.

But it does look like cmake is happy with it...:
-- Found ngspice: /vagrant/build/ngspice-dest/share/ngspice/include

Is ngspice ever installed to that location at any point? Or could this be some leftovers from previous versions? Is the /vagrant/build location completely virgin when we initiate a build?

@adamwolf
Copy link
Contributor Author

adamwolf commented Jun 24, 2019 via email

@adamwolf
Copy link
Contributor Author

@adamwolf
Copy link
Contributor Author

@adamwolf
Copy link
Contributor Author

nickoe, the build is on a completely brand new VM every single time. I'll try changing the include path. Thanks for the sleuthing.

@adamwolf
Copy link
Contributor Author

I got it working on my dev machine. Pushed to the branch, we 'll see what happens overnight.

nickoe added a commit that referenced this issue Jun 30, 2019
@nickoe
Copy link
Contributor

nickoe commented Jul 1, 2019

@adamwolf Could you check your build nodes? The builds have been running for 1 day and 15 hours by now. Maybe the build script is running too many things in parralel and using all the machine memory?

@adamwolf
Copy link
Contributor Author

adamwolf commented Jul 1, 2019 via email

@nickoe
Copy link
Contributor

nickoe commented Jul 1, 2019

@adamwolf ok, the build did became green!!!! woo

https://builder.wayneandlayne.com/job/kicad-mac-builder-10.14-ngspice-upload/10/

I guess we can merge #286

@adamwolf
Copy link
Contributor Author

adamwolf commented Jul 1, 2019 via email

@nickoe
Copy link
Contributor

nickoe commented Jul 1, 2019

@adamwolf Maybe you should delete the ngspice-upload build and re-enable https://builder.wayneandlayne.com/job/kicad-mac-builder-10.14-upload/ ?

@adamwolf
Copy link
Contributor Author

adamwolf commented Jul 2, 2019 via email

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants