Improve missing-home-modules warning formatting
authorMatthew Pickering <matthewtpickering@gmail.com>
Fri, 2 Mar 2018 21:12:22 +0000 (16:12 -0500)
committerBen Gamari <ben@smart-cactus.org>
Fri, 2 Mar 2018 21:12:43 +0000 (16:12 -0500)
Previously the modules were smashed together at the end of the line. As
the header is quite long, this meant for quite long lines which wrapped
on smaller terminals.

err msg: A
         B

Now they are nested underneath the long first line of the error message.

err msg:
  A
  B

Reviewers: bgamari

Reviewed By: bgamari

Subscribers: rwbarton, thomie, carter

Differential Revision: https://phabricator.haskell.org/D4454

compiler/main/GhcMake.hs

index 73a7157..d399d03 100644 (file)
@@ -201,11 +201,16 @@ warnMissingHomeModules hsc_env mod_graph =
 
     msg
       | gopt Opt_BuildingCabalPackage dflags
-      = text "These modules are needed for compilation but not listed in your .cabal file's other-modules: "
-        <> sep (map ppr missing)
+      = hang
+          (text "These modules are needed for compilation but not listed in your .cabal file's other-modules: ")
+          4
+          (sep (map ppr missing))
       | otherwise
-      = text "Modules are not listed in command line but needed for compilation: "
-        <> sep (map ppr missing)
+      =
+        hang
+          (text "Modules are not listed in command line but needed for compilation: ")
+          4
+          (sep (map ppr missing))
     warn = makeIntoWarning
       (Reason Opt_WarnMissingHomeModules)
       (mkPlainErrMsg dflags noSrcSpan msg)