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 |
ElvishJerricco
left a comment
There was a problem hiding this comment.
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 |
|
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
Description of changes
Continuation of PR #260241 which I closed inadvertently.
Things done
nix.conf? (See Nix manual)sandbox = relaxedsandbox = truenix-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.