-
Notifications
You must be signed in to change notification settings - Fork 186
Add GTK4 to build matrix on GitHub actions #2767
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
base: master
Are you sure you want to change the base?
Conversation
1bddc88 to
ba8986a
Compare
ba8986a to
7ecd9b2
Compare
Add a gtk build matrix entry for GTK3 and GTK4. Included is a little bit of self-documenting of the dependencies to build on Linux. Part of eclipse-platform#2714
7ecd9b2 to
1d64c02
Compare
| -DskipNativeTests=false | ||
| -DfailIfNoTests=false | ||
| ${{ (inputs.runtodotests == false && contains(inputs.native, 'linux')) && '-DexcludedGroups=gtk3-todo' || '' }} | ||
| ${{ (inputs.runtodotests == false && inputs.gtk == 'gtk4') && '-DexcludedGroups=gtk4-todo' || '' }} |
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.
There are no tests currently tagged with gtk3-todos which is a good thing as GTK3 on x11 should be well covered.
There are some gtk3-wayland-todo tags, which are excluded in the next PR #2768
|
GTK4 on x11 tests seem stable enough (with the gtk4-todo exclusion tags). GTK4 on Wayland (and even GTK3 on Wayland) is more unstable testing. The wayland part of the matrix is a separate PR #2768 I think we are ready to submit this one, but I will wait until tomorrow in case anyone has any objections/concerns. |
|
I'm going to hold off on this until after we branch for this release. |
Included is a little bit of self-documenting of the dependencies to build on Linux.
Part of #2714
In draft until dependent PR #2765 is merged.