Skip to content
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

NetBSD: missing link against libexecinfo #1111

Closed
0-wiz-0 opened this issue Jun 17, 2023 · 3 comments
Closed

NetBSD: missing link against libexecinfo #1111

0-wiz-0 opened this issue Jun 17, 2023 · 3 comments

Comments

@0-wiz-0
Copy link
Contributor

0-wiz-0 commented Jun 17, 2023

When building geeqie-2.1 on NetBSD, linking fails with:

ld: src/geeqie.p/main.cc.o: undefined reference to symbol 'backtrace'
ld: /usr/lib/libexecinfo.so.0: error adding symbols: DSO missing from command line

On NetBSD, backtrace() is provided in libexecinfo, and since main.cc uses this (when the execinfo.h header is found, which NetBSD provides), it needs to link against -lexecinfo on NetBSD.

I don't speak meson well, can you please add this?

@caclark
Copy link
Collaborator

caclark commented Jun 18, 2023

Unfortunately I do not use NetBSD and my understanding of meson is not too good. Perhaps you could try the attached .diif and make the necessary changes?
1111.diff.gz

@0-wiz-0
Copy link
Contributor Author

0-wiz-0 commented Jun 18, 2023

Thanks for the patch!
Sadly, it doesn't work, since meson tries to find libexecinfo via pkg-config and NetBSD's base system libraries do not provide pkg-config files.
libexecinfo is part of NetBSD since 2012, so I think you could unconditionally depend on it on NetBSD.
However, it pointed me in the right direction. What do you think of #1114?

@0-wiz-0
Copy link
Contributor Author

0-wiz-0 commented Jun 19, 2023

#1114 was merged so this is fixed, thanks!

@0-wiz-0 0-wiz-0 closed this as completed Jun 19, 2023
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 a pull request may close this issue.

2 participants