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

test: Add library dependency for dlsym #919

Closed
wants to merge 1 commit into from

Conversation

sc108-lee
Copy link
Contributor

Fix build failure for test

[27/79] Linking target test/ioctl/libmock-ioctl.so FAILED: test/ioctl/libmock-ioctl.so
cc -o test/ioctl/libmock-ioctl.so test/ioctl/libmock-ioctl.so.p/mock.c.o test/ioctl/libmock-ioctl.so.p/util.c.o -Wl,--as-needed -Wl,--no-undefined -shared -fPIC -Wl,--start-group -Wl,-soname,libmock-ioctl.so -Wl,--end-group test/ioctl/libmock-ioctl.so.p/mock.c.o: In function ioctl': /root/Desktop/repo/libnvme_upstream/.build/../test/ioctl/mock.c:154: undefined reference to dlsym'

Fix build failure for test

[27/79] Linking target test/ioctl/libmock-ioctl.so
FAILED: test/ioctl/libmock-ioctl.so
cc  -o test/ioctl/libmock-ioctl.so test/ioctl/libmock-ioctl.so.p/mock.c.o test/ioctl/libmock-ioctl.so.p/util.c.o -Wl,--as-needed -Wl,--no-undefined -shared -fPIC -Wl,--start-group -Wl,-soname,libmock-ioctl.so -Wl,--end-group
test/ioctl/libmock-ioctl.so.p/mock.c.o: In function `ioctl':
/root/Desktop/repo/libnvme_upstream/.build/../test/ioctl/mock.c:154: undefined reference to `dlsym'

Signed-off-by: Steven Seungcheol Lee <[email protected]>
@igaw
Copy link
Collaborator

igaw commented Nov 29, 2024

Ooops, though have to look into why the CI build fails horrible.

@igaw
Copy link
Collaborator

igaw commented Nov 29, 2024

Which version of meson are you using? IIRC some libraries are added to the dependency list automatically and the dl lib might one of them. That might be the reason why it was failing in CI and on my machine.

@igaw
Copy link
Collaborator

igaw commented Dec 2, 2024

I've merged my fix from #923 which works around the problem when dl is not available. Thanks for the report!

@igaw igaw closed this Dec 2, 2024
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