-
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
fltk: Use the mesa-glu Conan package #22958
base: master
Are you sure you want to change the base?
Conversation
This comment has been minimized.
This comment has been minimized.
Since fltk is CMake-based, I'll likely need to upstream a fix for locating GLU. |
9b527af
to
81955e8
Compare
This comment has been minimized.
This comment has been minimized.
This is done the same way as for the FreeGLUT package in conan-io#22428. Fixes conan-io#22944.
81955e8
to
9b1c871
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This reverts commit 98b6c07.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@jcar87 I don't know if you meant to close this one as it has to do with |
Apologies! Didn't mean to close this one - this one needs a review in the similar vein, as the pattern is very similar - but indeed this is not the same case as the others |
GLU is way simpler in terms of complexity. |
This comment has been minimized.
This comment has been minimized.
Looks like a timeout error in CI. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Conan v1 pipeline ✔️All green in build 10 (
Conan v2 pipeline ✔️
All green in build 10 ( |
This is done the same way as for the FreeGLUT package in #22428. Fixes #22944.