Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

package: Set LD_LIBRARY_PATH for tests to work against shared library #403

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Conan-Kudo
Copy link
Member

When a shared library is built, we need to have the test suite attempt to
use our newly built shared library. Without this, it fails with unresolved
symbols.

When a shared library is built, we need to have the test suite attempt to
use our newly built shared library. Without this, it fails with unresolved
symbols.
@mlschroe
Copy link
Member

mlschroe commented Oct 8, 2020

Actually the test suite does know how to use the newly built library, that's a cmake feature. So I'm kind of puzzled how you get errors.

@Conan-Kudo
Copy link
Member Author

@mlschroe It did not work until I made that change for my build here: https://build.opensuse.org/package/rdiff/home:Pharaoh_Atem:DNF_SUSE:DNF5/libsolv?linkrev=base&rev=2

@Conan-Kudo
Copy link
Member Author

@mlschroe Note this is only broken on openSUSE, as I don't have this problem on Fedora or Mageia.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants