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
netboot: Support cmdline variable from netboot.xyz #150360
Conversation
The idea looks good to me. I'm getting confused by that |
|
A quick test with the repl shows me that you're correct (I find that syntax confusing all the same...):
|
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.
Thank you!
@GrahamcOfBorg test boot.uefiNetboot boot.biosNetboot |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
c28f8bb
to
7e7510d
Compare
For clarity, it looks like currently Maybe @antonym can quickly weigh in with a suggestion? I don't have a strong preference either way. |
After thinking about it, I think just having |
I agree. |
Successfully created backport PR #155562 for |
Motivation for this change
Allows the kernel params to be specified from
netboot.xyz
's utilities menu.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/
)nixos/doc/manual/md-to-db.sh
to update generated release notes