feat(cabal-install): more logging in buildAndRegisterUnpackedPackage

More logging in ProjectBuilding

Author: andreabedini

cabal-install/src/Distribution/Client/ProjectBuilding.hs

@@ -540,7 +540,8 @@ rebuildTarget
             void $ waitAsyncPackageDownload verbosity downloadMap pkg
           _ -> return ()
         return $ BuildResult DocsNotTried TestsNotTried Nothing
-    | otherwise =
+    | otherwise = do
+        info verbosity $ "[rebuildTarget] Rebuilding " ++ prettyShow (nodeKey pkg) ++ " with current status " ++ buildStatusToString pkgBuildStatus
         -- We rely on the 'BuildStatus' to decide which phase to start from:
         case pkgBuildStatus of
           BuildStatusDownload -> downloadPhase
@@ -583,7 +584,8 @@ rebuildTarget
       -- would only start from download or unpack phases.
       --
       rebuildPhase :: BuildStatusRebuild -> SymbolicPath CWD (Dir Pkg) -> IO BuildResult
-      rebuildPhase buildStatus srcdir =
+      rebuildPhase buildStatus srcdir = do
+        info verbosity $ "[rebuildPhase] Rebuilding " ++ prettyShow (nodeKey pkg) ++ " in " ++ prettyShow srcdir
         assert
           (isInplaceBuildStyle $ elabBuildStyle pkg)
           buildInplace
@@ -598,7 +600,8 @@ rebuildTarget
       -- TODO: [nice to have] ^^ do this relative stuff better
 
       buildAndInstall :: SymbolicPath CWD (Dir Pkg) -> SymbolicPath Pkg (Dir Dist) -> IO BuildResult
-      buildAndInstall srcdir builddir =
+      buildAndInstall srcdir builddir = do
+        info verbosity $ "[buildAndInstall] Building and installing " ++ prettyShow (nodeKey pkg) ++ " in " ++ prettyShow srcdir
         buildAndInstallUnpackedPackage
           verbosity
           distDirLayout
@@ -614,8 +617,9 @@ rebuildTarget
           builddir
 
       buildInplace :: BuildStatusRebuild -> SymbolicPath CWD (Dir Pkg) -> SymbolicPath Pkg (Dir Dist) -> IO BuildResult
-      buildInplace buildStatus srcdir builddir =
+      buildInplace buildStatus srcdir builddir = do
         -- TODO: [nice to have] use a relative build dir rather than absolute
+        info verbosity $ "[buildInplace] Building inplace " ++ prettyShow (nodeKey pkg) ++ " in " ++ prettyShow srcdir
         buildInplaceUnpackedPackage
           verbosity
           distDirLayout

cabal-install/src/Distribution/Client/ProjectBuilding/UnpackedPackage.hs

@@ -117,6 +117,7 @@ import Distribution.Compat.Directory (listDirectory)
 
 import Distribution.Client.ProjectBuilding.PackageFileMonitor
 import Distribution.System (Platform(..))
+import qualified Distribution.Compat.Graph as Graph
 
 -- | Each unpacked package is processed in the following phases:
 --
@@ -184,33 +185,39 @@ buildAndRegisterUnpackedPackage
   builddir
   mlogFile
   delegate = do
+    info verbosity $ "\n\nbuildAndRegisterUnpackedPackage: " ++ prettyShow (Graph.nodeKey pkg)
     -- Configure phase
     delegate $
       PBConfigurePhase $
-        annotateFailure mlogFile ConfigureFailed $
+        annotateFailure mlogFile ConfigureFailed $ do
+          info verbosity $ "--- Configure phase " ++ prettyShow (Graph.nodeKey pkg)
           setup configureCommand Cabal.configCommonFlags configureFlags configureArgs
 
     -- Build phase
     delegate $
       PBBuildPhase $
         annotateFailure mlogFile BuildFailed $ do
+          info verbosity $ "--- Build phase " ++ prettyShow (Graph.nodeKey pkg)
           setup buildCommand Cabal.buildCommonFlags (return . buildFlags) buildArgs
 
     -- Haddock phase
     whenHaddock $
       delegate $
         PBHaddockPhase $
           annotateFailure mlogFile HaddocksFailed $ do
+            info verbosity $ "--- Haddock phase " ++ prettyShow (Graph.nodeKey pkg)
             setup haddockCommand Cabal.haddockCommonFlags (return . haddockFlags) haddockArgs
 
     -- Install phase
     delegate $
       PBInstallPhase
         { runCopy = \destdir ->
-            annotateFailure mlogFile InstallFailed $
+            annotateFailure mlogFile InstallFailed $ do
+              info verbosity $ "--- Install phase, copy " ++ prettyShow (Graph.nodeKey pkg)
               setup Cabal.copyCommand Cabal.copyCommonFlags (return . copyFlags destdir) copyArgs
         , runRegister = \pkgDBStack registerOpts ->
             annotateFailure mlogFile InstallFailed $ do
+              info verbosity $ "--- Install phase, register " ++ prettyShow (Graph.nodeKey pkg)
               -- We register ourselves rather than via Setup.hs. We need to
               -- grab and modify the InstalledPackageInfo. We decide what
               -- the installed package id is, not the build system.
@@ -232,21 +239,24 @@ buildAndRegisterUnpackedPackage
     whenTest $
       delegate $
         PBTestPhase $
-          annotateFailure mlogFile TestsFailed $
+          annotateFailure mlogFile TestsFailed $ do
+            info verbosity $ "--- Test phase " ++ prettyShow (Graph.nodeKey pkg)
             setup testCommand Cabal.testCommonFlags (return . testFlags) testArgs
 
     -- Bench phase
     whenBench $
       delegate $
         PBBenchPhase $
-          annotateFailure mlogFile BenchFailed $
+          annotateFailure mlogFile BenchFailed $ do
+            info verbosity $ "--- Benchmark phase " ++ prettyShow (Graph.nodeKey pkg)
             setup benchCommand Cabal.benchmarkCommonFlags (return . benchFlags) benchArgs
 
     -- Repl phase
     whenRepl $
       delegate $
         PBReplPhase $
-          annotateFailure mlogFile ReplFailed $
+          annotateFailure mlogFile ReplFailed $ do
+            info verbosity $ "--- Repl phase " ++ prettyShow (Graph.nodeKey pkg)
             setupInteractive replCommand Cabal.replCommonFlags replFlags replArgs
 
     return ()