Skip to content

Commit

Permalink
Merge pull request NixOS#327559 from philiptaron/issue-208242/amazon-…
Browse files Browse the repository at this point in the history
…image.nix

nixos/amazon-image: remove top-level `with lib`
  • Loading branch information
arianvp authored Jul 16, 2024
2 parents 7783833 + 2e17c4a commit c15e510
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 7 deletions.
8 changes: 4 additions & 4 deletions nixos/maintainers/scripts/ec2/amazon-image.nix
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
{ config, lib, pkgs, ... }:

with lib;

let
inherit (lib) mkOption optionalString types versionAtLeast;
inherit (lib.options) literalExpression;
cfg = config.amazonImage;
amiBootMode = if config.ec2.efi then "uefi" else "legacy-bios";

Expand All @@ -15,7 +15,7 @@ in {
# https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/nvme-ebs-volumes.html#timeout-nvme-ebs-volumes
config.boot.kernelParams =
let timeout =
if pkgs.lib.versionAtLeast config.boot.kernelPackages.kernel.version "4.15"
if versionAtLeast config.boot.kernelPackages.kernel.version "4.15"
then "4294967295"
else "255";
in [ "nvme_core.io_timeout=${timeout}" ];
Expand Down Expand Up @@ -156,5 +156,5 @@ in {
};
in if config.ec2.zfs.enable then zfsBuilder else extBuilder;

meta.maintainers = with maintainers; [ arianvp ];
meta.maintainers = with lib.maintainers; [ arianvp ];
}
5 changes: 2 additions & 3 deletions nixos/modules/virtualisation/amazon-image.nix
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,8 @@

{ config, lib, pkgs, ... }:

with lib;

let
inherit (lib) mkDefault mkIf;
cfg = config.ec2;
in

Expand Down Expand Up @@ -107,5 +106,5 @@ in
# (e.g. it depends on GTK).
services.udisks2.enable = false;
};
meta.maintainers = with maintainers; [ arianvp ];
meta.maintainers = with lib.maintainers; [ arianvp ];
}

0 comments on commit c15e510

Please sign in to comment.