-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
gm2calc: migrate to Conan v2 #18689
gm2calc: migrate to Conan v2 #18689
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Closing temporarily to avoid unnecessary load on the CI. Will reopen when I'm actively working on the PR again. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
b94b1b1
to
a2e0386
Compare
This comment has been minimized.
This comment has been minimized.
0cf2879
to
1673427
Compare
This comment has been minimized.
This comment has been minimized.
1673427
to
016d04d
Compare
Conan v1 pipeline ✔️All green in build 8 (
Conan v2 pipeline ✔️
All green in build 9 ( |
|
||
def generate(self): | ||
tc = CMakeToolchain(self) | ||
tc.variables["CMAKE_WINDOWS_EXPORT_ALL_SYMBOLS"] = True |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tc.variables["CMAKE_WINDOWS_EXPORT_ALL_SYMBOLS"] = True | |
tc.variables["CMAKE_WINDOWS_EXPORT_ALL_SYMBOLS"] = True | |
tc.variables["ENABLE_TESTS"] = True |
Since 2.2 you can actually disable tests:
https://github.com/GM2Calc/GM2Calc/blob/v2.2.0/CMakeLists.txt#L9
TODO
src/slhaea.h:25:10: fatal error: boost/algorithm/string/classification.hpp: No such file or directory
.