-
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
qt/5: Use libglvnd on FreeBSD and Linux #23280
Conversation
The libglvnd package provides the necessary opengl/system and egl/system dependencies. The libglvnd package is a proper, non-system Conan package. Remove unnecessary check for wayland being enabled in xkbcommon. The Wayland option in xkbcommon only effects building an executable tool.
🤖 Beep Boop! This pull request is making changes to 'recipes/qt//'. 👋 @ericLemanissier @jwillikers @MartinDelille @paulharris you might be interested. 😉 |
Conan v1 pipeline ✔️All green in build 1 (
Conan v2 pipeline ✔️
All green in build 1 ( |
Hooks produced the following warnings for commit 40f8f27qt/5.15.10@#9e1da5fbe4b10a7daea511160d67e86e
qt/5.15.12@#3b517c330227a08300c01cbbeb6e5a62
qt/5.15.11@#27cce7f08eee8224906aa6aa8bdad289
qt/5.15.13@#69f550f02c3069f8f4d8cb2318d88c7e
qt/5.15.9@#ae62c17fe494a0eda39e3c6e2de21e79
|
Closing pending discussion at #23649 Also perhaps relevant to Qt only: back in the day, the generated *-config.cmake files would actually reference some of the OpenGL libraries with a full, absolute path. WHich can be okay on some Linux systems assuming they reference a system location, but certainly not if this were to be pointing to a binary in the Conan cache. On the other hand, we are not using thoe files in Conan, but it's worth noting. |
The libglvnd package provides the necessary
opengl/system
andegl/system
dependencies. The libglvnd package is a proper, non-system Conan package.Remove unnecessary check for wayland being enabled in xkbcommon. The Wayland option in xkbcommon only effects building an executable tool.