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

Build libdnf5 static library, re-enable unit tests that use hidden (private) libdnf5 symbols #1507

Merged

Conversation

jrohel
Copy link
Contributor

@jrohel jrohel commented May 23, 2024

Some unit tests use private symbols that are not exported from libdnf5.so shared library.
Therefore, a static version of the libdnf5 library is built in which all symbols are available. Unit tests that need private symbols use the static library.

Please merge after #1307 .

@jan-kolarik jan-kolarik self-assigned this May 27, 2024
@jrohel jrohel force-pushed the fix_disabled_unittests branch 2 times, most recently from e71d6d2 to 2ada044 Compare June 6, 2024 09:18
@jan-kolarik
Copy link
Member

I am merging this, although CI is broken now. It seems to be due to previous issues in Packit buildroot in combination with race during merging history undo PR and this one. Let's see and eventually fix the issues right in the subsequent PR.

@jan-kolarik jan-kolarik added this pull request to the merge queue Jun 10, 2024
Merged via the queue into rpm-software-management:main with commit 4a729aa Jun 10, 2024
2 of 15 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.

2 participants