Skip to content
This repository was archived by the owner on Oct 15, 2022. It is now read-only.

Commit 45c692e

Browse files
committed
Do not install clippy, carnix, nixpkgs-fmt on macOS
1 parent 440e415 commit 45c692e

File tree

1 file changed

+13
-3
lines changed

1 file changed

+13
-3
lines changed

Diff for: shell.nix

+13-3
Original file line numberDiff line numberDiff line change
@@ -38,16 +38,13 @@ let
3838

3939
buildInputs = [
4040
pkgs.cargo
41-
pkgs.rustPackages.clippy
4241
pkgs.rustc
4342
pkgs.rustfmt
4443
pkgs.bashInteractive
4544
pkgs.git
4645
pkgs.direnv
4746
pkgs.shellcheck
48-
pkgs.carnix
4947
pkgs.nix-prefetch-git
50-
pkgs.nixpkgs-fmt
5148

5249
# To ensure we always have a compatible nix in our shells.
5350
# Travis doesn’t know `nix-env` otherwise.
@@ -56,6 +53,19 @@ let
5653
pkgs.darwin.Security
5754
pkgs.darwin.apple_sdk.frameworks.CoreServices
5855
pkgs.darwin.apple_sdk.frameworks.CoreFoundation
56+
] ++ pkgs.stdenv.lib.optionals (!pkgs.stdenv.isDarwin) [
57+
# Cachix is broken on macOS [1] and clippy is not built by Hydra [2].
58+
# Building clippy and carnix on macOS in CI takes about 25 minutes, so we
59+
# don't run lints (which require clippy) on macOS.
60+
#
61+
# [1] https://github.com/cachix/cachix/issues/228#issuecomment-533634704
62+
# [2] https://github.com/NixOS/nixpkgs/issues/77358
63+
pkgs.rustPackages.clippy
64+
pkgs.carnix
65+
66+
# These other tools are also used only for linting, and are thus not
67+
# required on macOS.
68+
pkgs.nixpkgs-fmt
5969
];
6070

6171
# we manually collect all build inputs,

0 commit comments

Comments
 (0)