-
-
Notifications
You must be signed in to change notification settings - Fork 15.1k
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
nixos/systemd-boot: Add support for an XBOOTLDR partition #285401
Conversation
7afa4bc
to
a04c9ba
Compare
@nikstur @RaitoBezarius ideally qemu-vm should provide an option to override the |
a04c9ba
to
1bcaddb
Compare
1bcaddb
to
db53fad
Compare
@nikstur @RaitoBezarius gentle ping |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
@nikstur @RaitoBezarius can you please take a look at the last blocker for this PR? Specifically these lines. |
Looks good from my side. |
Thanks. @ElvishJerricco @JulienMalka can we merge this then? |
@ofborg test systemd-boot |
db53fad
to
b470b44
Compare
Fixed a merge conflict. |
@ofborg test systemd-boot |
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. @RaitoBezarius shall we merge?
Let's go, I want to say that @sdht0 you have been incredibly patient with us, I would like to point out this is something I rarely see in nixpkgs contributions and a big big big thank you, this has been a very pleasant experience for me to review, even beyond the disagreements. |
Thanks for the kind words @RaitoBezarius! Nice to get this finally in. Thanks @JulienMalka @ElvishJerricco @nikstur for the reviews. And thanks again @colemickens for handing over your PR. I got to have quite a bit of fun. |
Very cool that (the original accidentally closed) PR was your first PR in nixpkgs, @sdht0! Quite a cool first contribution, and more already since then. 😎 🚀 |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/add-windows-to-boot-menu/40675/5 |
Description of changes
Continuation of PR #260241 which I closed inadvertently.
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.