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
kwin: apply @tildearrow's low latency patch #101041
Conversation
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
plasma-framework, libcap, libdrm, mesa | ||
plasma-framework, libcap, libdrm, mesa, | ||
|
||
lowLatencyPatch ? true, fetchpatch ? null |
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.
lowLatencyPatch ? true, fetchpatch ? null | |
lowLatencyPatch ? true, fetchpatch |
Should we enable this on default? I am not sure. @ttuegel do you have an opinion about 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.
No, I think we should not add patches by default that aren't fixing some bug. But, we can add a top-level attribute for it, so it will be built with and without the patch on Hydra.
@@ -37,7 +39,13 @@ mkDerivation { | |||
patches = [ | |||
./0001-follow-symlinks.patch | |||
./0002-xwayland.patch | |||
]; | |||
]++ lib.optional lowLatencyPatch (fetchpatch { |
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.
]++ lib.optional lowLatencyPatch (fetchpatch { | |
] ++ lib.optional lowLatencyPatch (fetchpatch { |
Besides my other comment, please add a NixOS option to override the kwin package chosen by the Plasma module. |
@ttuegel I'm not the author, but I might give it a go (if the author doesn't mind, of course) since this PR seems to have stalled and I'm interested in learning more about NixOS. However, I'm having trouble figuring out the specifics (mostly naming). I looked around a bit and
In general I'm not super familiar with NixOS yet, and so while I've looked a bit at other modules I still don't have a good sense of what would be most consistent/predictable. Also, would |
@InternetUnexplorer is right, this is better as an overlay |
Motivation for this change
Adapting my previous work in #89234 to Plasma 5.18
Fixes #85541
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)