Refactor builder path manipulation
[hadrian.git] / src / Settings / Builders / GhcCabal.hs
index cec876a..7a0669a 100644 (file)
@@ -1,51 +1,50 @@
 module Settings.Builders.GhcCabal (
-    cabalArgs, ghcCabalHsColourArgs, ghcIncludeDirs, bootPackageDbArgs,
-    customPackageArgs, ccArgs, cppArgs, ccWarnings, argStagedSettingList, needDll0
+    ghcCabalBuilderArgs, ghcCabalHsColourBuilderArgs, buildDll0
     ) where
 
-import Expression
-import Predicates hiding (stage)
-import Settings
+import Context
+import Flavour
+import Rules.Actions
+import Settings.Builders.Common
 
-cabalArgs :: Args
-cabalArgs = builder GhcCabal ? do
-    path <- getPackagePath
-    dir  <- getTargetDirectory
+ghcCabalBuilderArgs :: Args
+ghcCabalBuilderArgs = builder GhcCabal ? do
+    verbosity <- lift $ getVerbosity
     mconcat [ arg "configure"
-            , arg path
-            , arg dir
+            , arg =<< getPackagePath
+            , arg =<< getContextDirectory
             , dll0Args
-            , withStaged Ghc
+            , withStaged $ Ghc CompileHs
             , withStaged GhcPkg
-            , bootPackageDbArgs
+            , bootPackageDatabaseArgs
             , libraryArgs
             , with HsColour
             , configureArgs
             , packageConstraints
-            , withStaged Gcc
+            , withStaged $ Cc CompileC
             , notStage0 ? with Ld
             , with Ar
             , with Alex
-            , with Happy ]
+            , with Happy
+            , verbosity < Chatty ? append [ "-v0", "--configure-option=--quiet"
+                , "--configure-option=--disable-option-checking"  ] ]
 
-ghcCabalHsColourArgs :: Args
-ghcCabalHsColourArgs = builder GhcCabalHsColour ? do
+ghcCabalHsColourBuilderArgs :: Args
+ghcCabalHsColourBuilderArgs = builder GhcCabalHsColour ? do
     path <- getPackagePath
-    dir  <- getTargetDirectory
-    mconcat [ arg "hscolour"
-            , arg path
-            , arg dir ]
+    dir  <- getContextDirectory
+    append [ "hscolour", path, dir ]
 
 -- TODO: Isn't vanilla always built? If yes, some conditions are redundant.
 -- TODO: Need compiler_stage1_CONFIGURE_OPTS += --disable-library-for-ghci?
 libraryArgs :: Args
 libraryArgs = do
-    ways     <- getWays
+    ways     <- getLibraryWays
     withGhci <- lift ghcWithInterpreter
     append [ if vanilla `elem` ways
              then  "--enable-library-vanilla"
              else "--disable-library-vanilla"
-           , if vanilla `elem` ways && withGhci && not dynamicGhcPrograms
+           , if vanilla `elem` ways && withGhci && not (dynamicGhcPrograms flavour)
              then  "--enable-library-for-ghci"
              else "--disable-library-for-ghci"
            , if profiling `elem` ways
@@ -58,10 +57,12 @@ libraryArgs = do
 -- TODO: LD_OPTS?
 configureArgs :: Args
 configureArgs = do
+    top <- getTopDirectory
     let conf key = appendSubD $ "--configure-option=" ++ key
-        cFlags   = mconcat [ ccArgs
+        cFlags   = mconcat [ cArgs
                            , remove ["-Werror"]
-                           , argStagedSettingList ConfCcArgs ]
+                           , argStagedSettingList ConfCcArgs
+                           , arg $ "-I" ++ top -/- generatedPath ]
         ldFlags  = ldArgs  <> (argStagedSettingList ConfGccLinkerArgs)
         cppFlags = cppArgs <> (argStagedSettingList ConfCppArgs)
     mconcat
@@ -69,166 +70,56 @@ configureArgs = do
         , conf "LDFLAGS"  ldFlags
         , conf "CPPFLAGS" cppFlags
         , appendSubD "--gcc-options" $ cFlags <> ldFlags
-        , conf "--with-iconv-includes"    $ argSettingList IconvIncludeDirs
-        , conf "--with-iconv-libraries"   $ argSettingList IconvLibDirs
-        , conf "--with-gmp-includes"      $ argSettingList GmpIncludeDirs
-        , conf "--with-gmp-libraries"     $ argSettingList GmpLibDirs
+        , conf "--with-iconv-includes"    $ argSetting IconvIncludeDir
+        , conf "--with-iconv-libraries"   $ argSetting IconvLibDir
+        , conf "--with-gmp-includes"      $ argSetting GmpIncludeDir
+        , conf "--with-gmp-libraries"     $ argSetting GmpLibDir
         , crossCompiling ? (conf "--host" $ argSetting TargetPlatformFull)
-        , conf "--with-cc" $ argStagedBuilderPath Gcc ]
-
-bootPackageDbArgs :: Args
-bootPackageDbArgs = stage0 ? do
-    path <- getSetting GhcSourcePath
-    lift $ need [bootstrappingConfInitialised]
-    isGhc <- stagedBuilder Ghc ||^ stagedBuilder GhcM
-    let prefix = if isGhc then "-package-db " else "--package-db="
-    arg $ prefix ++ path -/- bootstrappingConf
+        , conf "--with-cc" $ argStagedBuilderPath (Cc CompileC) ]
 
 packageConstraints :: Args
 packageConstraints = stage0 ? do
     constraints <- lift . readFileLines $ bootPackageConstraints
     append $ concat [ ["--constraint", c] | c <- constraints ]
 
--- TODO: should be in a different file
--- TODO: put all validating options together in one file
-ccArgs :: Args
-ccArgs = validating ? ccWarnings
-
--- TODO: should be in a different file
-ccWarnings :: Args
-ccWarnings = do
-    let gccGe46 = notM $ (flag GccIsClang ||^ flag GccLt46)
-    mconcat [ turnWarningsIntoErrors ? arg "-Werror"
-            , arg "-Wall"
-            , flag GccIsClang ? arg "-Wno-unknown-pragmas"
-            , gccGe46 ? notM windowsHost ? arg "-Werror=unused-but-set-variable"
-            , gccGe46 ? arg "-Wno-error=inline" ]
-
-ldArgs :: Args
-ldArgs = mempty
-
-ghcIncludeDirs :: [FilePath]
-ghcIncludeDirs = [ "includes", "includes/dist"
-                 , "includes/dist-derivedconstants/header"
-                 , "includes/dist-ghcconstants/header" ]
-
 cppArgs :: Args
-cppArgs = append $ map ("-I" ++) ghcIncludeDirs
-
--- TODO: Is this needed?
--- ifeq "$(GMP_PREFER_FRAMEWORK)" "YES"
--- libraries/integer-gmp_CONFIGURE_OPTS += --with-gmp-framework-preferred
--- endif
-
--- TODO: move this somewhere
-customPackageArgs :: Args
-customPackageArgs = do
-    stage   <- getStage
-    rtsWays <- getRtsWays
-    mconcat
-        [ package integerGmp ?
-          mconcat [ windowsHost ? builder GhcCabal ?
-                    arg "--configure-option=--with-intree-gmp"
-                  , appendCcArgs ["-I" ++ pkgPath integerGmp -/- "gmp"] ]
-
-        , package base ?
-          builder GhcCabal ?
-          arg ("--flags=" ++ takeFileName (pkgPath integerLibrary))
-
-        , package ghcPrim ?
-          builder GhcCabal ? arg "--flag=include-ghc-prim"
-
-        , package compiler ?
-          builder GhcCabal ?
-          mconcat [ arg $ "--ghc-option=-DSTAGE=" ++ show (fromEnum stage + 1)
-                  , arg "--disable-library-for-ghci"
-                  , anyTargetOs ["openbsd"] ? arg "--ld-options=-E"
-                  , flag GhcUnregisterised ? arg "--ghc-option=-DNO_REGS"
-                  , notM ghcWithSMP ? arg "--ghc-option=-DNOSMP"
-                  , notM ghcWithSMP ? arg "--ghc-option=-optc-DNOSMP"
-                  , (threaded `elem` rtsWays) ?
-                    notStage0 ? arg "--ghc-option=-optc-DTHREADED_RTS"
-                  , ghcWithNativeCodeGen ? arg "--flags=ncg"
-                  , ghcWithInterpreter ?
-                    notStage0 ? arg "--flags=ghci"
-                  , ghcWithInterpreter ?
-                    ghcEnableTablesNextToCode ?
-                    notM (flag GhcUnregisterised) ?
-                    notStage0 ? arg "--ghc-option=-DGHCI_TABLES_NEXT_TO_CODE"
-                  , ghcWithInterpreter ?
-                    ghciWithDebugger ?
-                    notStage0 ? arg "--ghc-option=-DDEBUGGER"
-                  , ghcProfiled ?
-                    notStage0 ? arg "--ghc-pkg-option=--force" ]
-
-        , package ghc ?
-          builder GhcCabal ?
-          mconcat [ ghcWithInterpreter ?
-                    notStage0 ? arg "--flags=ghci" ]
-
-        , package haddock ?
-          builder GhcCabal ? append ["--flag", "in-ghc-tree"] ]
+cppArgs = arg $ "-I" ++ generatedPath
 
 withBuilderKey :: Builder -> String
 withBuilderKey b = case b of
     Ar       -> "--with-ar="
     Ld       -> "--with-ld="
-    Gcc _    -> "--with-gcc="
-    Ghc _    -> "--with-ghc="
+    Cc  _ _  -> "--with-gcc="
+    Ghc _ _  -> "--with-ghc="
     Alex     -> "--with-alex="
     Happy    -> "--with-happy="
     GhcPkg _ -> "--with-ghc-pkg="
     HsColour -> "--with-hscolour="
-    _        -> error "withBuilderKey: not supported builder"
+    _        -> error $ "withBuilderKey: not supported builder " ++ show b
 
--- Expression 'with Gcc' appends "--with-gcc=/path/to/gcc" and needs Gcc.
+-- Expression 'with Alex' appends "--with-alex=/path/to/alex" and needs Alex.
 with :: Builder -> Args
-with b = specified b ? do
-    top  <- getSetting GhcSourcePath
+with b = isSpecified b ? do
+    top  <- getTopDirectory
     path <- getBuilderPath b
-    lift $ needBuilder laxDependencies b
-    append [withBuilderKey b ++ top -/- path]
+    lift $ needBuilder b
+    arg $ withBuilderKey b ++ unifyPath (top </> path)
 
 withStaged :: (Stage -> Builder) -> Args
-withStaged sb = (with . sb) =<< getStage
-
-argM :: Action String -> Args
-argM = (arg =<<) . lift
-
-argSetting :: Setting -> Args
-argSetting = argM . setting
-
-argSettingList :: SettingList -> Args
-argSettingList = (append =<<) . lift . settingList
-
-argStagedSettingList :: (Stage -> SettingList) -> Args
-argStagedSettingList ss = (argSettingList . ss) =<< getStage
-
-argStagedBuilderPath :: (Stage -> Builder) -> Args
-argStagedBuilderPath sb = (argM . builderPath . sb) =<< getStage
-
--- Pass arguments to Gcc and corresponding lists of sub-arguments of GhcCabal
--- TODO: simplify
-appendCcArgs :: [String] -> Args
-appendCcArgs xs = do
-    mconcat [ stagedBuilder Gcc  ? append xs
-            , stagedBuilder GccM ? append xs
-            , builder GhcCabal   ? appendSub "--configure-option=CFLAGS" xs
-            , builder GhcCabal   ? appendSub "--gcc-options" xs ]
+withStaged sb = with . sb =<< getStage
 
-needDll0 :: Stage -> Package -> Action Bool
-needDll0 stage pkg = do
+buildDll0 :: Context -> Action Bool
+buildDll0 Context {..} = do
     windows <- windowsHost
-    return $ windows && pkg == compiler && stage == Stage1
+    return $ windows && stage == Stage1 && package == compiler
 
 -- This is a positional argument, hence:
 -- * if it is empty, we need to emit one empty string argument;
 -- * otherwise, we must collapse it into one space-separated string.
 dll0Args :: Args
 dll0Args = do
-    stage    <- getStage
-    pkg      <- getPackage
-    dll0     <- lift $ needDll0 stage pkg
+    context  <- getContext
+    dll0     <- lift $ buildDll0 context
     withGhci <- lift ghcWithInterpreter
     arg . unwords . concat $ [ modules     | dll0             ]
                           ++ [ ghciModules | dll0 && withGhci ] -- see #9552