-
Notifications
You must be signed in to change notification settings - Fork 585
[GR] update Qt6Base_jll
#11052
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?
[GR] update Qt6Base_jll
#11052
Conversation
Crap, I'm clueless / stuck with this
EDIT: as pointed out in #11052 (comment), it looks like an ICE. |
I think this also needs a version bump on GR itself, due to the changing compat bounds on the dependencies. Best to do this when there is a new version of GR. |
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.
Version number must change.
Just tried locally, you need to switch to GCC 13 on Windows. |
Note that this still can't be merged without changing the JLL version, which is currently in sync with the upstream GR version. I think it's best to hold off merging this until the next upstream version is released, @jheinen ? |
Keeping upstream versions and jll versions in sync has mostly cosmetic value for the jll package maintainers. Waiting for a new upstream release is fine, but so is artificially increasing the version number of the jll package. Look e.g. at the recipe for |
In Plots CI, this was the reason for the
ColorXXX
andImageXXX
stack being left behind latest versions (see below).Cc @jheinen, since changing compat bounds are always risky in
Yggdrasil
.From
PlotsBase
test suite, usingPkg.status(; outdated = true, mode = Pkg.PKGMODE_MANIFEST)
:before PR
after PR
Tested locally (GR test suite + Plots test suite) on
linux
.Might be worth testing on macos & windows as well ...