Skip to content

Conversation

Kreijstal
Copy link
Contributor

I would like assistance updating ngspice to 44.2 as you can see the .dll is missing, there are some libtool warnings, not sure what I did wrong :)

@Kreijstal Kreijstal marked this pull request as ready for review February 4, 2025 18:53
@Kreijstal
Copy link
Contributor Author

hmm it seems on CI it compiles fine

"${_common_flags[@]}" \
--with-ngshared
make
make -j
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why? it's passed automatically

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

when testing it, it didn't seem it was getting passed, or maybe it was a placebo effect, but I thought -j made things faster.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've tested it on my laptop, and I'm pretty sure the makefiles that ngspice seems configuring "-j" is not passed automatically, i mean unless makefile also listen to an env var im Not aware of? This is outside the PKGBUILD

[LIBS="$LIBS -lreadline"],
[AC_MSG_ERROR([Couldn't find readline libraries.])])
+
+PKG_CHECK_MODULES_STATIC([NCURSES], [ncursesw], [
Copy link
Member

@lazka lazka Feb 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why is curses needed? It's not linking against it.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

only statically, it does get used.
https://github.com/imr/ngspice/blob/e011d14f6fd9595723b2acd6e21324826f25aef5/src/frontend/terminal.c#L26
because if you link dynamically we get another error, namely:
#10312

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ahhh you mean curses should not be runtime dependency, yeah,I think you're right, sorry about that.

cd $(DESTDIR)$(pkglibdir); \
rm -f ivlng*a libvvp* ; \
- mv ivlngvpi.* ivlng.vpi
+ if [ -f ivlngvpi.* ]; then \
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you know why this is needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We compile 3 times, correct, when doing the nonshared version (ngspice_con), ivlngvpi doesn't get generated, probably libtool shenanigans,
https://sourceforge.net/p/ngspice/bugs/735/

@Kreijstal
Copy link
Contributor Author

I notice that after simply updating the branch suddenly it cannot be built with clang, while obviously, it in the past could, is this a clang bug? or a ngspice regression or more likely clang doesnt accept this anymore?

@Kreijstal Kreijstal changed the title update to ngspice 44.2 update to ngspice 45.2 Sep 9, 2025
@Kreijstal
Copy link
Contributor Author

@lazka managed to make it buildable with 45.2 and clang again..

@lazka lazka merged commit f4d44a6 into msys2:master Sep 13, 2025
8 checks passed
MehdiChinoune pushed a commit to MehdiChinoune/MINGW-packages that referenced this pull request Sep 13, 2025
* update to ngspice 44.2

* curses not runtime dependent

* ngspice: enable osdi

Enabling osdi, allows for ngspice to simulate pdks like the IHP130pdk, https://github.com/IHP-GmbH/IHP-Open-PDK

* Update ngspice version to 45.2 in PKGBUILD

* Ngspice2osdi (#10)

* test

* ngspice 45.2

* Fix LDFLAGS syntax for clang in PKGBUILD

* Remove illegal specialization for compilation

* Update sha256sum for a package in PKGBUILD
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants