Skip to content

fix(install): bootloader installation fixes#160

Merged
water-sucks merged 1 commit intonix-community:mainfrom
Sporif:install-fix-bootloader-installation
Jan 27, 2026
Merged

fix(install): bootloader installation fixes#160
water-sucks merged 1 commit intonix-community:mainfrom
Sporif:install-fix-bootloader-installation

Conversation

@Sporif
Copy link
Copy Markdown
Collaborator

@Sporif Sporif commented Jan 26, 2026

Add the following fixes from nixos-install:

Also use an explicit argument index to avoid repeating root in the Sprintf call.

Add the following fixes from nixos-install:
 - NixOS/nixpkgs@d61cc625: fix removal of non-empty /mnt
 - NixOS/nixpkgs@fa39e3d6: fail if we can't set up bootloader
 - NixOS/nixpkgs@5d470b88: clear executable cache after entering chroot

Also use an explicit argument index to avoid repeating `root` in the `Sprintf` call.
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Jan 26, 2026

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@water-sucks water-sucks merged commit b3beec6 into nix-community:main Jan 27, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants