Skip to content

fix(module): use normal import call for exposing module in default.nix#78

Merged
water-sucks merged 1 commit intonix-community:mainfrom
water-sucks:fix-infrec-for-legacy-systems-module
Jun 29, 2025
Merged

fix(module): use normal import call for exposing module in default.nix#78
water-sucks merged 1 commit intonix-community:mainfrom
water-sucks:fix-infrec-for-legacy-systems-module

Conversation

@water-sucks
Copy link
Copy Markdown
Collaborator

lib.importApply was causing an infinite recursion for some unknown reason in default.nix for the module export, but this function does not currently cause issues in the flake.

Tested with the following minimal example configuration.nix:

{pkgs, ...}: let
  # In pure evaluation mode, always use a full Git commit hash instead of a branch name.
  nixos-cli-url = "https://github.com/nix-community/nixos-cli/archive/ac75350abb4b09b7d761cfb5ad4ce3d0c514efb9.tar.gz";
  nixos-cli = import "${builtins.fetchTarball nixos-cli-url}" {inherit pkgs;};
in {
  imports = [nixos-cli.module];

  services.nixos-cli = {
    enable = true;
    package = nixos-cli.nixosLegacy;
    config = {
      # Other configuration for nixos-cli
    };
  };

  nix.settings = {
    substituters = ["https://watersucks.cachix.org"];
    trusted-public-keys = [
      "watersucks.cachix.org-1:6gadPC5R8iLWQ3EUtfu3GFrVY7X6I4Fwz/ihW25Jbv8="
    ];
  };

  fileSystems = {
    "/" = {
      device = "/dev/vda2";
      fsType = "ext4";
    };
    "/boot/efi" = {
      device = "/dev/vda1";
      fsType = "vfat";
    };
  };

  boot.loader = {
    grub = {
      enable = true;
      efiSupport = true;
      device = "nodev";
    };
  };

  system.stateVersion = "25.05";
}

Closes #77.

`lib.importApply` was causing an infinite recursion for some unknown reason in
`default.nix`, but does not currently cause issues in the flake.
@water-sucks water-sucks merged commit 48033e6 into nix-community:main Jun 29, 2025
1 check passed
@water-sucks water-sucks deleted the fix-infrec-for-legacy-systems-module branch June 29, 2025 21:34
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.

Fail to install due to "infinite recursion encountered"

1 participant