Skip to content

Commit

Permalink
shellFor: Don't suck in src to compare to deps. [Fixes #51079]
Browse files Browse the repository at this point in the history
  • Loading branch information
ElvishJerricco committed Jan 29, 2019
1 parent 119a931 commit b40b39b
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions pkgs/development/haskell-modules/make-package-set.nix
Expand Up @@ -260,7 +260,10 @@ in package-set { inherit pkgs stdenv callPackage; } self // {
# bash$ nix-shell --run "cabal new-build all"
shellFor = { packages, withHoogle ? false, ... } @ args:
let
selected = packages self;
nullSrc = p: overrideCabal p (_: { src = null; });

# Make sure we *never* accidentally suck in src.
selected = map nullSrc (packages self);

packageInputs = map getBuildInputs selected;

Expand All @@ -272,7 +275,8 @@ in package-set { inherit pkgs stdenv callPackage; } self // {
# because cabal will end up ignoring that built version, assuming
# new-style commands.
haskellInputs = pkgs.lib.filter
(input: pkgs.lib.all (p: input.outPath != p.outPath) selected)
# nullSrc in case a dep is one of the selected packages.
(input: pkgs.lib.all (p: (nullSrc input).outPath != p.outPath) selected)
(pkgs.lib.concatMap (p: p.haskellBuildInputs) packageInputs);
systemInputs = pkgs.lib.concatMap (p: p.systemBuildInputs) packageInputs;

Expand Down

0 comments on commit b40b39b

Please sign in to comment.