-
Notifications
You must be signed in to change notification settings - Fork 47
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Potential rpath problem with libplanarity #592
Comments
Wait I replied too fast, let me properly read your comment |
so my analogy was that Digraphs+libplanarity is roughly comparable to Semigroups + libsemigroups, in terms of build system requirements and issues. Thus, the issue of missing I would offer to submit a patch but unfortunately cannot currently reproduce the issue :-/. Perhaps @AndreasEnge can and is willing to help.... Or perhaps we can find a Linux distro where it is easy to repro and then setup a virtualized one to make it reproducible in CI tests? |
Thanks @fingolfin, I understood this after reading your issue properly, I did not pay enough attention the first time :( I'll be happy to accept a PR if one is forthcoming, I also can't reproduce this at present, so don't know exactly what will fix it. |
Hello, and sorry for the delay! When I compile gap with digraphs in Guix (currently it is disabled), then execute Indeed ldd on digraphs.so shows that libplanarity.so.0 is not found. But my impression is that this will be the same for the semigroups package (which depends on digraphs, so that I cannot meaningfully test it): ldd on semigroups.so also does not find libsemigroups.so. This may be an idosyncrasy of Guix, or maybe a more general problem. Andreas |
I was sitting with @AndreasEnge to update the guix package for GAP, and we tested which packages work. We run into an error that looks a lot like the libsemigroups problem related to rpath handling.
Specifically, it built the bundled libplanarity but that then later was not found. Fixing this may be a matter of adding
KEXT_LDFLAGS += -Wl,-rpath,$(PWD)/bin/lib
to the build system. But there is of course a second part, setting rpath for an installed libplanarity, which I guess could also be adapted from Semigroups....The text was updated successfully, but these errors were encountered: