-
-
Notifications
You must be signed in to change notification settings - Fork 14.2k
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
yuzu: update, switch to qt6, clean up, etc #228872
Conversation
8d0de7d
to
ae15eb3
Compare
staging-next merged, this should be a lot easier to test now. |
Thanks for picking this up! I just tested it on two machines and it worked quite well. With OpenGL, everything I tested worked, Vulkan crashed after a short time on Intel Arc (but worked on AMD Vega), though I don’t think that is something we can fix in nixpkgs. While yuzu should now support Wayland, that requires Also, nice that you could find the git metadata for the early-access branch, so we can reconstruct the source exactly. Because the source in pineapple’s repository is only tested on their build pipeline, we had problems using it in the past. |
It should pick up qtwayland ambiently if you're running it in a Wayland session, though it might be worth adding to the wrapper as well. |
I am using sway and also tried adding qtwayland to my user packages and system packages, but in both cases, it didn’t find the plugin. It’s possible that there is something wrong with my configuration, because I had the same problem with Anki. However, other people had the same problem which lead to the inclusion of qtwayland in the build inputs (reasoning, including references to discussions about similar problems). |
Let's just add it to the wrapper then. |
Updated to current release while I'm at it. |
Also, tested a bit more - on 680M (Rembrandt iGPU), 6700S (Navi 23 mobile) and 6800XT (Navi 21 desktop), seems fine for me in both modes, no weird crashing, no weird colors on Gamescope or Plasma. |
Hm, then this must have been an issue on my end. Thanks for testing it. |
Updated again since a bunch of perf fixes got merged to mainline. We should really set up an auto-updating flake for this... |
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.
Works great here (NixOS, x86_64-linux). Thanks for the update!
Trying to install here, if I don't manage, I'll test it once it's merged |
Tested as working over here, thanks for this! |
So, should I do another update and then merge this? |
Updated to latest versions, supposedly there's some more optimizations. |
Working beautifully for me, the updates brought some great optimisations to the game I want to play 😉. |
Description of changes
Testing appreciated. Vulkan updates are in staging-next.
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)