-
-
Notifications
You must be signed in to change notification settings - Fork 14k
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
qt6: 6.6.3 -> 6.7.0 #300933
qt6: 6.6.3 -> 6.7.0 #300933
Conversation
@ofborg build qt6.full |
We can reduce the churn on the patch files a lot by doing something like https://github.com/NixOS/nixpkgs/blob/nixos-unstable/pkgs/os-specific/linux/systemd/default.nix#L205C8-L205C102 |
|
Done. |
@wegank would you like to also take a look at this? (Sorry for the trouble but I really don't have the bandwidth/intention to fix these darwin failures) |
The aligned allocation failure is because clang is hardcoded to fail with aligned allocations if the deployment target is older than 10.13 because that’s the first release when Apple shipped support in the OS’s libc++. However, libc++ can implement aligned allocations using The error van be worked around by setting the deployment target to 10.13, but since the minimum version supported by Qt is 11.0, it’s better to use that. The easiest way is using Change that line to following (modulo typos sunce I’m replying on my phone). stdenv =
if stdenv.isDarwin then
overrideSDK stdenv {
darwinMinVersion = "11.0";
darwinSdkVersion = "11.0";
}
else
stdenv; |
@ofborg build qt6.qtdeclarative |
@ofborg build qt6.full telegram-desktop |
Result of 79 packages marked as broken and skipped:
69 packages failed to build:
1455 packages built:
|
Testing on an aarch64 system, will probably not be done rebuilding until tomorrow. |
pyqt6-webengine isn't happy, will look at it more tomorrow |
Still waiting for a pyqt6 and pyside6 release. |
Qt for Python release: 6.7 is now available! 🐍 |
Looks like also needs a darwin stdenv defined by |
Result of 79 packages marked as broken and skipped:
39 packages failed to build:
1496 packages built:
|
|
Now running nixpkgs-review, ready to merge if it comes back clean. |
@ofborg build qt6.qtdeclarative |
@ofborg test plasma6 |
Result of 79 packages marked as broken and skipped:
18 packages failed to build:
1517 packages built:
And darwin is also green on ofborg. |
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.
LGTM
Breaks Telegram and Prism Launcher for me, probably related to NixOS/nixpkgs#300933 This reverts commit d6761e1.
Hi, I've started having issues with Telegram and Prism Launcher in this commit range and this PR jumps out as a potential offender. My system is running GNOME + Wayland, on Telegram I can't see the media viewer window anymore when clicking on an image and in Prism Launcher I can't see the window for editing an instance. In both cases Alt + Tab shows a transparent window for those screens, but switching to it does not do anything. Downgrading to the older nixpkgs-unstable revision resolves the issue. Please let me know how can I help debug this further. |
This same issue is also reported by some Arch Linux users. Appears that all windows that's created maximized under gnome wayland are not visible. A temporary workaround is to set |
Thanks for the info! Is a more permanent fix being tracked somewhere that I can subscribe to? |
|
Description of changes
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.