Skip to content

Comments

lib/systems/default.nix: add efiArch suffixes#193175

Merged
alyssais merged 1 commit intoNixOS:masterfrom
unbel13ver:fix/efiarch-suffixes
Sep 29, 2022
Merged

lib/systems/default.nix: add efiArch suffixes#193175
alyssais merged 1 commit intoNixOS:masterfrom
unbel13ver:fix/efiarch-suffixes

Conversation

@unbel13ver
Copy link
Contributor

Move already implemented functionality to the upper level so it could be used in a more generic way.

Signed-off-by: Ivan Nikolaenko ivan.nikolaenko@unikie.com

Description of changes
Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 22.11 Release Notes (or backporting 22.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
    • (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes
  • Fits CONTRIBUTING.md.

@github-actions github-actions bot added 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 8.has: module (update) This PR changes an existing module in `nixos/` labels Sep 27, 2022
@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels Sep 27, 2022
@alyssais
Copy link
Member

This can go to master, btw, since it's not a mass-rebuild.

(Here's how to do that without accidentally pinging code owners.)

Move already implemented functionality to the upper level so
it could be used in a more generic way.

Signed-off-by: Ivan Nikolaenko <ivan.nikolaenko@unikie.com>
@github-actions github-actions bot added 6.topic: GNOME GNOME desktop environment and its underlying platform 6.topic: python Python is a high-level, general-purpose programming language. 8.has: documentation This PR adds or changes documentation labels Sep 28, 2022
@unbel13ver unbel13ver changed the base branch from staging to master September 28, 2022 10:27
@github-actions github-actions bot removed 6.topic: python Python is a high-level, general-purpose programming language. 8.has: documentation This PR adds or changes documentation labels Sep 28, 2022
@github-actions github-actions bot removed the 6.topic: GNOME GNOME desktop environment and its underlying platform label Sep 28, 2022
@unbel13ver
Copy link
Contributor Author

This can go to master, btw, since it's not a mass-rebuild.

(Here's how to do that without accidentally pinging code owners.)

And yet again, I've done that in a wrong way, sorry!

@ofborg ofborg bot added 2.status: merge conflict This PR has merge conflicts with the target branch and removed 2.status: merge conflict This PR has merge conflicts with the target branch labels Sep 28, 2022
@alyssais
Copy link
Member

And yet again, I've done that in a wrong way, sorry!

No worries — it's a horrible GitHub quirk. I heard the other day that we might have discovered an easier way of doing it, so hopefully that can end up in the documentation soon.

@alyssais alyssais merged commit f251840 into NixOS:master Sep 29, 2022
@roberth roberth added the 1.severity: channel blocker Blocks a channel label Sep 29, 2022
@roberth
Copy link
Member

roberth commented Sep 29, 2022

This broke eval, blocking the channels https://hydra.nixos.org/jobset/nixos/unstable-small#tabs-errors

{UNKNOWN}: aggregate job ‘tested’ failed with the error: nixos.tests.boot.biosCdrom.x86_64-linux: error: undefined variable 'targetArch'

       at /nix/store/k38k0x7adns7glmv4z6iw7gshrg1xxc4-source/nixos/modules/installer/cd-dvd/iso-image.nix:275:117:

          274|     # probe for devices, even with --skip-fs-probe.
          275|     grub-mkimage --directory=${grubPkgs.grub2_efi}/lib/grub/${grubPkgs.grub2_efi.grubTarget} -o $out/EFI/boot/boot${targetArch}.efi -p /EFI/boot -O ${grubPkgs.grub2_efi.grubTarget} \
             |                                                                                                                     ^
          276|       $MODULES
nixos.iso_minimal.x86_64-linux: error: undefined variable 'targetArch'

       at /nix/store/k38k0x7adns7glmv4z6iw7gshrg1xxc4-source/nixos/modules/installer/cd-dvd/iso-image.nix:275:117:

          274|     # probe for devices, even with --skip-fs-probe.
          275|     grub-mkimage --directory=${grubPkgs.grub2_efi}/lib/grub/${grubPkgs.grub2_efi.grubTarget} -o $out/EFI/boot/boot${targetArch}.efi -p /EFI/boot -O ${grubPkgs.grub2_efi.grubTarget} \
             |                                                                                                                     ^
          276|       $MODULES
 at /nix/store/yi625cadgqpikk21cys15minw5ppz9zq-hydra-perl-deps/lib/perl5/site_perl/5.34.1/Catalyst/Model/DBIC/Schema.pm line 526

@roberth
Copy link
Member

roberth commented Sep 29, 2022

And eval's back https://hydra.nixos.org/eval/1782600

@roberth roberth removed the 1.severity: channel blocker Blocks a channel label Sep 29, 2022
@unbel13ver unbel13ver deleted the fix/efiarch-suffixes branch May 31, 2023 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 8.has: module (update) This PR changes an existing module in `nixos/` 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants