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

Enable unit tests in ADIOS2 static CI build #3751

Closed
vicentebolea opened this issue Aug 11, 2023 · 0 comments · Fixed by #3760
Closed

Enable unit tests in ADIOS2 static CI build #3751

vicentebolea opened this issue Aug 11, 2023 · 0 comments · Fixed by #3760
Labels
area: build Build issues area: ci Continuous Integration issues

Comments

@vicentebolea
Copy link
Collaborator

vicentebolea commented Aug 11, 2023

Extracted from #3749

This PR enables two static builds for ADIOS2 however they do not build the adios2 unittests, the reason is that each of our unit test is built in a single file, meaning that when enabling all the common options we build over 200 binaries. When we do a static build this is a problem since each of this binary will have the same size as the sum of all its dependencies. This result in Github actions returning a out-of-space error.

In order to enable tests in the static build we must group the tests in a single (or a few) binaries with the cmake function create_test_sourcelist https://cmake.org/cmake/help/book/mastering-cmake/chapter/Testing%20With%20CMake%20and%20CTest.html

Originally posted by @vicentebolea in #3749 (comment)

@vicentebolea vicentebolea added area: build Build issues area: ci Continuous Integration issues labels Aug 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: build Build issues area: ci Continuous Integration issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant