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
[20.09] undervolt: expose power limits as Nixopts #102215
Conversation
We no longer escape the flags because the power limit flags want two arguments If we escaped them, we'd only get one argument with an escaped space in it. Undervolt's flags don't have anything in them that would need to be escaped, so that shouldn't break anything (cherry picked from commit eb4e675)
/marvin opt-in |
Hi! I'm an experimental bot. My goal is to guide this PR through its stages, hopefully ending with a merge. You can read up on the usage here. |
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.
And this change is already in master.
/status needs_merger
What do you mean by that? |
I reviewed this before I realized it was a backport and it looked good to me. Conclusion: I'm good with this change and would merge it. |
Oh sorry, I confused this PR with my other one you reviewed haha. ^^ |
It is a convention (I'm not sure if its documented anywhere, but I do it and I have seen others do it too) to prefix backport PR titles with the release they are targeting. That helps to avoid that confusion. |
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.
Thanks! I don't see any reason against this backport, and our release managers are probably overworked. So let's just get this in :)
Motivation for this change
#95011 (comment)
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)