Skip to content

test: Add unit test verifying c application can dlopen library #898

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

Merged
merged 1 commit into from
May 23, 2025

Conversation

maxtmann
Copy link
Contributor

This test checks whether plugin library can be loaded with dlopen from c application . The reason is that plugin formerly didn't link against libstdc++ and relied on application to provide required linkage. This has been fixed by recent commit but test was not added.

This test should only be merged after PR #897, which fixes above mentioned issue on neuron platforms, has been merged.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@maxtmann maxtmann requested a review from a team as a code owner May 22, 2025 14:55
@bwbarrett bwbarrett force-pushed the test/unit_test_dlopen branch from 2832af0 to 316b419 Compare May 22, 2025 16:46
@maxtmann maxtmann force-pushed the test/unit_test_dlopen branch from 316b419 to ed6588a Compare May 22, 2025 19:22
This test checks whether plugin library can be loaded with dlopen from
c application . The reason is that plugin formerly didn't link against
libstdc++ and relied on application to provide required linkage. This
has been fixed by recent commit but test was not added.

Signed-off-by: Michael Axtmann <[email protected]>
@maxtmann maxtmann force-pushed the test/unit_test_dlopen branch from ed6588a to be538c5 Compare May 22, 2025 19:43
@maxtmann maxtmann merged commit 7a481d0 into aws:master May 23, 2025
24 checks passed
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.

4 participants