summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEdward Z. Yang <ezyang@fb.com>2017-11-25 10:12:05 +0800
committerEdward Z. Yang <ezyang@fb.com>2017-11-30 19:25:29 -0500
commite1fb28384c44fcd29b0e60b9fd44767be22646af (patch)
tree7c25a031d26a6e3a53294f16d135e91d589c37e0
parent6998772043a7f0b0360116eb5ffcbaa5630b21fb (diff)
downloadhaskell-e1fb28384c44fcd29b0e60b9fd44767be22646af.tar.gz
Handle CPP properly in Backpack
Summary: Previously, we attempted to lookup 'hole' packages for include directories; this obviously is not going to work. Signed-off-by: Edward Z. Yang <ezyang@fb.com> Test Plan: validate Reviewers: ekmett, bgamari Subscribers: rwbarton, thomie GHC Trac Issues: #14525 Differential Revision: https://phabricator.haskell.org/D4234
-rw-r--r--compiler/main/Packages.hs10
-rw-r--r--testsuite/tests/backpack/cabal/bkpcabal01/p/P.hs.in11
2 files changed, 9 insertions, 2 deletions
diff --git a/compiler/main/Packages.hs b/compiler/main/Packages.hs
index c49581b0d9..14407be418 100644
--- a/compiler/main/Packages.hs
+++ b/compiler/main/Packages.hs
@@ -1892,8 +1892,14 @@ listVisibleModuleNames dflags =
getPreloadPackagesAnd :: DynFlags -> [PreloadUnitId] -> IO [PackageConfig]
getPreloadPackagesAnd dflags pkgids0 =
let
- pkgids = pkgids0 ++ map (toInstalledUnitId . moduleUnitId . snd)
- (thisUnitIdInsts dflags)
+ pkgids = pkgids0 ++
+ -- An indefinite package will have insts to HOLE,
+ -- which is not a real package. Don't look it up.
+ -- Fixes #14525
+ if isIndefinite dflags
+ then []
+ else map (toInstalledUnitId . moduleUnitId . snd)
+ (thisUnitIdInsts dflags)
state = pkgState dflags
pkg_map = pkgIdMap state
preload = preloadPackages state
diff --git a/testsuite/tests/backpack/cabal/bkpcabal01/p/P.hs.in1 b/testsuite/tests/backpack/cabal/bkpcabal01/p/P.hs.in1
index 327a032132..875c370915 100644
--- a/testsuite/tests/backpack/cabal/bkpcabal01/p/P.hs.in1
+++ b/testsuite/tests/backpack/cabal/bkpcabal01/p/P.hs.in1
@@ -1,3 +1,4 @@
+{-# LANGUAGE CPP #-}
module P where
import H
y = x