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
undervolt: expose power limits as Nixopts #95011
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
/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. |
@glittershark please review. |
Both limit and window must be set. | ||
''; | ||
}; | ||
p2.window = mkOption { |
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.
just a thought, but we might be able to use a submodule type to deduplicate the window
and limit
type definitions. But that might be overly abstracted, so I'll leave it up to you / other reviewers
/status needs_merger |
Reminder: Please review! Reminder: This Pull Request is awaiting merger. If you are the assigned reviewer with commit permission, please have a look. If you can't, please say so. If the status is not accurate, please change it. If nothing happens, this PR will be put back in the |
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 @glittershark and @pbogdan for the reviews.
Thank you! Should we backport this to 20.09? It already has the undervolt update which allows for this feature. |
Yes, I think that would be fine, since its purely a feature-addition without risk of breakge. Its not my decision though. I think it would be best to open a backport PR and ping one of the release managers. |
Motivation for this change
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)