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

little fix cmake + little fix github pipelines #636

Merged
merged 8 commits into from
Jul 24, 2024

Conversation

Arniiiii
Copy link
Contributor

@Arniiiii Arniiiii commented Jul 24, 2024

Problem:
Install, enscriptem and macos pipelines were failing.

Install because there was find_package(ut ) with specific old version if BOOST_UT_TEST_INSTALLED_VERSION at test/CMakeLists.txt.

enscriptem because there was ask specific ask for set(CMAKE_CXX_STANDARD 20) at test/CMakeLists.txt and there was an example that could not be compiled. I've fixed cmake and wrapped with macro the failing part of the example

macos was failing because it couldn't install pyyaml. After my friend at live through discord tried to reproduce and made mistake, it was found out that pipx install pyaml works, but pipx install pyyaml doesn't work. IDK why. Though, there's test fail at macos pipeline, and a guy said it's because source_location is broken on macos.

Also there was some bad docstring for BOOST_UT_ENABLE_SANITIZERS .

Also added two .sh scripts at tests for easy checking of installed version.

Also all cmake commands at pipelines now produce more verbose output and some commands parallelized at -j4 .

maybe review, please
@krzysztof-jusiak

@krzysztof-jusiak krzysztof-jusiak merged commit 11c2bff into boost-ext:master Jul 24, 2024
12 checks passed
@krzysztof-jusiak
Copy link
Contributor

Thank you @Arniiiii very much! 🙇

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