Skip to content

Commit

Permalink
Eliminate InstalledPackageInfo module name parameter.
Browse files Browse the repository at this point in the history
This parameter was a legacy when GHC depended directly on
Cabal; this is no longer the case so there is no need for
this parameter (which is unused inside Cabal.)

Signed-off-by: Edward Z. Yang <ezyang@cs.stanford.edu>
  • Loading branch information
ezyang committed Aug 16, 2015
1 parent 221ed31 commit 0642af1
Show file tree
Hide file tree
Showing 12 changed files with 22 additions and 24 deletions.
22 changes: 10 additions & 12 deletions Cabal/Distribution/InstalledPackageInfo.hs
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@
-- This module is meant to be local-only to Distribution...

module Distribution.InstalledPackageInfo (
InstalledPackageInfo_(..), InstalledPackageInfo,
InstalledPackageInfo(..),
libraryName,
OriginalModule(..), ExposedModule(..),
ParseResult(..), PError(..), PWarning,
Expand Down Expand Up @@ -71,7 +71,7 @@ import GHC.Generics (Generic)
-- The InstalledPackageInfo type


data InstalledPackageInfo_ m
data InstalledPackageInfo
= InstalledPackageInfo {
-- these parts are exactly the same as PackageDescription
installedPackageId :: InstalledPackageId,
Expand All @@ -90,8 +90,8 @@ data InstalledPackageInfo_ m
-- these parts are required by an installed package only:
exposed :: Bool,
exposedModules :: [ExposedModule],
instantiatedWith :: [(m, OriginalModule)],
hiddenModules :: [m],
instantiatedWith :: [(ModuleName, OriginalModule)],
hiddenModules :: [ModuleName],
trusted :: Bool,
importDirs :: [FilePath],
libraryDirs :: [FilePath],
Expand All @@ -112,23 +112,21 @@ data InstalledPackageInfo_ m
}
deriving (Generic, Read, Show)

libraryName :: InstalledPackageInfo_ a -> LibraryName
libraryName :: InstalledPackageInfo -> LibraryName
libraryName ipi = Package.packageKeyLibraryName (sourcePackageId ipi) (packageKey ipi)

instance Binary m => Binary (InstalledPackageInfo_ m)
instance Binary InstalledPackageInfo

instance Package.Package (InstalledPackageInfo_ str) where
instance Package.Package InstalledPackageInfo where
packageId = sourcePackageId

instance Package.HasInstalledPackageId (InstalledPackageInfo_ str) where
instance Package.HasInstalledPackageId InstalledPackageInfo where
installedPackageId = installedPackageId

instance Package.PackageInstalled (InstalledPackageInfo_ str) where
instance Package.PackageInstalled InstalledPackageInfo where
installedDepends = depends

type InstalledPackageInfo = InstalledPackageInfo_ ModuleName

emptyInstalledPackageInfo :: InstalledPackageInfo_ m
emptyInstalledPackageInfo :: InstalledPackageInfo
emptyInstalledPackageInfo
= InstalledPackageInfo {
installedPackageId = InstalledPackageId "",
Expand Down
2 changes: 1 addition & 1 deletion Cabal/Distribution/Simple/Build.hs
Original file line number Diff line number Diff line change
Expand Up @@ -388,7 +388,7 @@ testSuiteLibV09AsLibAndExe :: PackageDescription
-> (PackageDescription,
Library, ComponentLocalBuildInfo,
LocalBuildInfo,
IPI.InstalledPackageInfo_ ModuleName,
IPI.InstalledPackageInfo,
Executable, ComponentLocalBuildInfo)
testSuiteLibV09AsLibAndExe pkg_descr
test@TestSuite { testInterface = TestSuiteLibV09 _ m }
Expand Down
2 changes: 1 addition & 1 deletion Cabal/Distribution/Simple/GHC.hs
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ import Distribution.PackageDescription as PD
import Distribution.InstalledPackageInfo
( InstalledPackageInfo )
import qualified Distribution.InstalledPackageInfo as InstalledPackageInfo
( InstalledPackageInfo_(..) )
( InstalledPackageInfo(..) )
import Distribution.Simple.PackageIndex (InstalledPackageIndex)
import qualified Distribution.Simple.PackageIndex as PackageIndex
import Distribution.Simple.LocalBuildInfo
Expand Down
2 changes: 1 addition & 1 deletion Cabal/Distribution/Simple/GHC/Internal.hs
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ import Distribution.Package
import Distribution.InstalledPackageInfo
( InstalledPackageInfo )
import qualified Distribution.InstalledPackageInfo as InstalledPackageInfo
( InstalledPackageInfo_(..) )
( InstalledPackageInfo(..) )
import Distribution.PackageDescription as PD
( BuildInfo(..), Library(..), libModules
, hcOptions, usedExtensions, ModuleRenaming, lookupRenaming )
Expand Down
2 changes: 1 addition & 1 deletion Cabal/Distribution/Simple/GHCJS.hs
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ import Distribution.PackageDescription as PD
import Distribution.InstalledPackageInfo
( InstalledPackageInfo )
import qualified Distribution.InstalledPackageInfo as InstalledPackageInfo
( InstalledPackageInfo_(..) )
( InstalledPackageInfo(..) )
import Distribution.Package ( LibraryName(..), getHSLibraryName )
import Distribution.Simple.PackageIndex ( InstalledPackageIndex )
import qualified Distribution.Simple.PackageIndex as PackageIndex
Expand Down
2 changes: 1 addition & 1 deletion Cabal/Distribution/Simple/Haddock.hs
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ import Distribution.Simple.BuildPaths
import Distribution.Simple.PackageIndex (dependencyClosure)
import qualified Distribution.Simple.PackageIndex as PackageIndex
import qualified Distribution.InstalledPackageInfo as InstalledPackageInfo
( InstalledPackageInfo_(..) )
( InstalledPackageInfo(..) )
import Distribution.InstalledPackageInfo
( InstalledPackageInfo )
import Distribution.Simple.Utils
Expand Down
2 changes: 1 addition & 1 deletion Cabal/Distribution/Simple/LHC.hs
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ import Distribution.InstalledPackageInfo
( InstalledPackageInfo
, parseInstalledPackageInfo )
import qualified Distribution.InstalledPackageInfo as InstalledPackageInfo
( InstalledPackageInfo_(..) )
( InstalledPackageInfo(..) )
import Distribution.Simple.PackageIndex
import qualified Distribution.Simple.PackageIndex as PackageIndex
import Distribution.ParseUtils ( ParseResult(..) )
Expand Down
2 changes: 1 addition & 1 deletion Cabal/Distribution/Simple/PreProcess.hs
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ import Distribution.PackageDescription as PD
, TestSuiteInterface(..)
, Benchmark(..), benchmarkModules, BenchmarkInterface(..) )
import qualified Distribution.InstalledPackageInfo as Installed
( InstalledPackageInfo_(..) )
( InstalledPackageInfo(..) )
import qualified Distribution.Simple.PackageIndex as PackageIndex
import Distribution.Simple.CCompiler
( cSourceExtensions )
Expand Down
2 changes: 1 addition & 1 deletion Cabal/Distribution/Simple/Program/HcPkg.hs
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ import Prelude hiding (init)
import Distribution.Package
( PackageId, InstalledPackageId(..) )
import Distribution.InstalledPackageInfo
( InstalledPackageInfo, InstalledPackageInfo_(..)
( InstalledPackageInfo, InstalledPackageInfo(..)
, showInstalledPackageInfo
, emptyInstalledPackageInfo, fieldsInstalledPackageInfo )
import Distribution.ParseUtils
Expand Down
2 changes: 1 addition & 1 deletion Cabal/Distribution/Simple/Register.hs
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ import Distribution.Package
( Package(..), packageName, InstalledPackageId(..)
, getHSLibraryName )
import Distribution.InstalledPackageInfo
( InstalledPackageInfo, InstalledPackageInfo_(InstalledPackageInfo)
( InstalledPackageInfo, InstalledPackageInfo(InstalledPackageInfo)
, showInstalledPackageInfo )
import qualified Distribution.InstalledPackageInfo as IPI
import Distribution.Simple.Utils
Expand Down
2 changes: 1 addition & 1 deletion cabal-install/Distribution/Client/Haddock.hs
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ import Distribution.Simple.PackageIndex
import Distribution.Simple.Utils
( comparing, debug, installDirectoryContents, withTempDirectory )
import Distribution.InstalledPackageInfo as InstalledPackageInfo
( InstalledPackageInfo_(exposed) )
( InstalledPackageInfo(exposed) )

regenerateHaddockIndex :: Verbosity
-> InstalledPackageIndex -> ProgramConfiguration
Expand Down
4 changes: 2 additions & 2 deletions cabal-install/Distribution/Client/Types.hs
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ import Distribution.Package
, HasInstalledPackageId(..), PackageInstalled(..)
, LibraryName, packageKeyLibraryName )
import Distribution.InstalledPackageInfo
( InstalledPackageInfo, InstalledPackageInfo_ )
( InstalledPackageInfo )
import Distribution.PackageDescription
( Benchmark(..), GenericPackageDescription(..), FlagAssignment
, TestSuite(..) )
Expand Down Expand Up @@ -68,7 +68,7 @@ data SourcePackageDb = SourcePackageDb {
class Package pkg => PackageFixedDeps pkg where
depends :: pkg -> ComponentDeps [InstalledPackageId]

instance PackageFixedDeps (InstalledPackageInfo_ str) where
instance PackageFixedDeps InstalledPackageInfo where
depends = CD.fromInstalled . installedDepends


Expand Down

0 comments on commit 0642af1

Please sign in to comment.