1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
511
512
513
514
515
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
532
533
534
535
536
537
538
539
540
541
542
543
544
545
546
547
548
549
550
551
552
553
554
555
556
557
558
559
560
561
562
563
564
565
566
567
568
569
570
571
572
573
574
575
576
577
578
579
580
581
582
583
584
585
586
587
588
589
590
591
592
593
594
595
596
597
598
599
600
601
602
603
604
605
606
607
608
|
{-# LANGUAGE InstanceSigs, TypeOperators #-}
module Builder (
-- * Data types
ArMode (..), CcMode (..), ConfigurationInfo (..), DependencyType (..),
GhcMode (..), GhcPkgMode (..), HaddockMode (..), TestMode(..), SphinxMode (..),
TarMode (..), GitMode (..), Builder (..), Win32TarballsMode(..),
-- * Builder properties
builderProvenance, systemBuilderPath, builderPath, isSpecified, needBuilders,
runBuilder, runBuilderWith, runBuilderWithCmdOptions, getBuilderPath,
builderEnvironment,
-- * Ad hoc builder invocation
applyPatch
) where
import Control.Exception.Extra (Partial)
import qualified Data.ByteString.Lazy.Char8 as BSL
import Development.Shake.Classes
import Development.Shake.Command
import Development.Shake.FilePath
import GHC.Generics
import qualified Hadrian.Builder as H
import Hadrian.Builder hiding (Builder)
import Hadrian.Builder.Ar
import Hadrian.Builder.Sphinx
import Hadrian.Builder.Tar
import Hadrian.Oracles.Path
import Hadrian.Oracles.TextFile
import Hadrian.Utilities
import Oracles.Setting (bashPath)
import System.Exit
import System.IO (stderr)
import Base
import Context
import Oracles.Flag
import Oracles.Setting (setting, Setting(..))
import Packages
import GHC.IO.Encoding (getFileSystemEncoding)
import qualified Data.ByteString as BS
import qualified GHC.Foreign as GHC
-- | C compiler can be used in two different modes:
-- * Compile or preprocess a source file.
-- * Extract source dependencies by passing @-MM@ command line argument.
data CcMode = CompileC | FindCDependencies DependencyType deriving (Eq, Generic, Show)
data DependencyType = CDep | CxxDep deriving (Eq, Generic, Show)
instance Binary CcMode
instance Hashable CcMode
instance NFData CcMode
instance Binary DependencyType
instance Hashable DependencyType
instance NFData DependencyType
-- | GHC can be used in four different modes:
-- * Compile a Haskell source file.
-- * Compile a C source file.
-- * Extract source dependencies by passing @-M@ command line argument.
-- * Link object files & static libraries into an executable.
data GhcMode = CompileHs
| CompileCWithGhc
| CompileCppWithGhc
| FindHsDependencies
| LinkHs
| ToolArgs
deriving (Eq, Generic, Show)
instance Binary GhcMode
instance Hashable GhcMode
instance NFData GhcMode
-- | To configure a package we need two pieces of information, which we choose
-- to record separately for convenience.
--
-- * Command line arguments to be passed to the setup script.
--
-- * Package configuration flags that enable/disable certain package features.
-- Here is an example from "Settings.Packages":
--
-- > package rts
-- > ? builder (Cabal Flags)
-- > ? any (wayUnit Profiling) rtsWays
-- > ? arg "profiling"
--
-- This instructs package configuration functions (such as 'configurePackage')
-- to enable the @profiling@ Cabal flag when processing @rts.cabal@ and
-- building RTS with profiling information.
data ConfigurationInfo
= Setup
| Flags
| Install
deriving (Eq, Generic, Show)
instance Binary ConfigurationInfo
instance Hashable ConfigurationInfo
instance NFData ConfigurationInfo
-- TODO: Do we really need all these modes? Why do we need 'Dependencies'? We
-- can extract dependencies using the Cabal library.
-- | 'GhcPkg' can initialise a package database and register packages in it.
data GhcPkgMode = Recache -- ^ Recache a package database
| Copy -- ^ Copy a package from one database to another.
| Dependencies -- ^ Compute package dependencies.
| Unregister -- ^ Unregister a package.
| Update -- ^ Update a package.
deriving (Eq, Generic, Show)
instance Binary GhcPkgMode
instance Hashable GhcPkgMode
instance NFData GhcPkgMode
-- | Haddock can be used in two different modes:
-- * Generate documentation for a single package
-- * Generate an index page for a collection of packages
data HaddockMode = BuildPackage | BuildIndex deriving (Eq, Generic, Show)
instance Binary HaddockMode
instance Hashable HaddockMode
instance NFData HaddockMode
-- | The testsuite driver can be called in two different modes:
-- * Actually run the tests
-- * Get all the hadrian targets needed for the current test configuration
data TestMode = RunTest | GetExtraDeps deriving (Eq, Generic, Show)
instance Binary TestMode
instance Hashable TestMode
instance NFData TestMode
-- | Git is used to create source distributions
data GitMode = ListFiles deriving (Eq, Generic, Show)
instance Binary GitMode
instance Hashable GitMode
instance NFData GitMode
data Win32TarballsMode = ListTarballs | VerifyTarballs | DownloadTarballs deriving (Eq, Generic, Show)
instance Binary Win32TarballsMode
instance Hashable Win32TarballsMode
instance NFData Win32TarballsMode
-- | A 'Builder' is a (usually external) command invoked in a separate process
-- via 'cmd'. Here are some examples:
-- * 'Alex' is a lexical analyser generator that builds @Lexer.hs@ from @Lexer.x@.
-- * 'Ghc' 'Stage0' is the bootstrapping Haskell compiler used in 'Stage0'.
-- * 'Ghc' @StageN@ (N > 0) is the GHC built in stage (N - 1) and used in @StageN@.
--
-- The 'Cabal' builder is unusual in that it does not correspond to an external
-- program but instead relies on the Cabal library for package configuration.
data Builder = Alex
| Ar ArMode Stage
| Autoreconf FilePath
| Cabal ConfigurationInfo Stage
| Cc CcMode Stage
| Configure FilePath
| DeriveConstants
| GenApply
| GenPrimopCode
| Ghc GhcMode Stage
| GhcPkg GhcPkgMode Stage
| Haddock HaddockMode
| Happy
| Hp2Ps
| Hpc
| HsCpp
| Hsc2Hs Stage
| Ld Stage --- ^ linker
| Make FilePath
| Makeinfo
| MergeObjects Stage -- ^ linker to be used to merge object files.
| Nm
| Objdump
| Patch
| Python
| Ranlib
| Testsuite TestMode
| Sphinx SphinxMode
| Tar TarMode
| Unlit
| Xelatex
| Makeindex -- ^ from xelatex
| Git GitMode
| Win32Tarballs Win32TarballsMode
deriving (Eq, Generic, Show)
instance Binary Builder
instance Hashable Builder
instance NFData Builder
-- | Some builders are built by this very build system, in which case
-- 'builderProvenance' returns the corresponding build 'Context' (which includes
-- 'Stage' and GHC 'Package').
builderProvenance :: Builder -> Maybe Context
builderProvenance = \case
DeriveConstants -> context stage0Boot deriveConstants
GenApply -> context stage0Boot genapply
GenPrimopCode -> context stage0Boot genprimopcode
Ghc _ (Stage0 {})-> Nothing
Ghc _ stage -> context (predStage stage) ghc
GhcPkg _ (Stage0 {}) -> Nothing
GhcPkg _ s -> context (predStage s) ghcPkg
Haddock _ -> context Stage1 haddock
Hsc2Hs _ -> context stage0Boot hsc2hs
Unlit -> context stage0Boot unlit
-- Never used
Hpc -> context Stage1 hpcBin
Hp2Ps -> context stage0Boot hp2ps
_ -> Nothing
where
context s p = Just $ vanillaContext s p
instance H.Builder Builder where
builderPath :: Builder -> Action FilePath
builderPath builder = case builderProvenance builder of
Nothing -> systemBuilderPath builder
Just context -> programPath context
runtimeDependencies :: Builder -> Action [FilePath]
runtimeDependencies = \case
Autoreconf dir -> return [dir -/- "configure.ac"]
Configure dir -> return [dir -/- "configure"]
Ghc _ (Stage0 {}) -> do
-- Read the boot GHC version here to make sure we rebuild when it
-- changes (#18001).
_bootGhcVersion <- setting GhcVersion
pure []
Ghc _ stage -> do
root <- buildRoot
touchyPath <- programPath (vanillaContext (Stage0 InTreeLibs) touchy)
unlitPath <- builderPath Unlit
-- GHC from the previous stage is used to build artifacts in the
-- current stage. Need the previous stage's GHC deps.
ghcdeps <- ghcBinDeps (predStage stage)
return $ [ unlitPath ]
++ ghcdeps
++ [ touchyPath | windowsHost ]
++ [ root -/- mingwStamp | windowsHost ]
-- proxy for the entire mingw toolchain that
-- we have in inplace/mingw initially, and then at
-- root -/- mingw.
Hsc2Hs stage -> (\p -> [p]) <$> templateHscPath stage
Make dir -> return [dir -/- "Makefile"]
Haddock _ -> haddockDeps Stage1 -- Haddock currently runs in Stage1
_ -> return []
-- query the builder for some information.
-- contrast this with runBuilderWith, which returns @Action ()@
-- this returns the @stdout@ from running the builder.
-- For now this only implements asking @ghc-pkg@ about package
-- dependencies and asking the testsuite driver about hadrian
-- dependencies for tests.
askBuilderWith :: Builder -> BuildInfo -> Action String
askBuilderWith builder BuildInfo {..} = case builder of
GhcPkg Dependencies _ -> do
let input = fromSingleton msgIn buildInputs
msgIn = "[askBuilder] Exactly one input file expected."
needBuilders [builder]
path <- H.builderPath builder
-- we do not depend on bare builders. E.g. we won't depend on `clang`
-- or `ld` or `ar`. Unless they are provided with fully qualified paths
-- this is the job of the person invoking ./configure to pass e.g.
-- CC=$(which clang) if they want the fully qualified clang path!
when (path /= takeFileName path) $
need [path]
Stdout stdout <- cmd' [path] ["--no-user-package-db", "field", input, "depends"]
return stdout
Testsuite GetExtraDeps -> do
path <- builderPath builder
withResources buildResources $
withTempFile $ \temp -> do
() <- cmd' [path] (buildArgs ++ ["--only-report-hadrian-deps", temp])
readFile' temp
Git ListFiles -> do
path <- builderPath builder
withResources buildResources $ do
-- NUL separated list of files
-- We need to read this in the filesystem encoding
enc <- liftIO getFileSystemEncoding
Stdout stdout <- cmd' BinaryPipes [path] buildArgs
liftIO $ BS.useAsCStringLen stdout $ \fp -> GHC.peekCStringLen enc fp
Win32Tarballs ListTarballs -> do
path <- builderPath builder
withResources buildResources $ do
Stdout stdout <- cmd' [path] buildArgs
pure stdout
_ -> error $ "Builder " ++ show builder ++ " can not be asked!"
runBuilderWith :: Builder -> BuildInfo -> Action ()
runBuilderWith builder BuildInfo {..} = do
path <- builderPath builder
withResources buildResources $ do
verbosity <- getVerbosity
let input = fromSingleton msgIn buildInputs
msgIn = "[runBuilderWith] Exactly one input file expected."
output = fromSingleton msgOut buildOutputs
msgOut = "[runBuilderWith] Exactly one output file expected."
-- Capture stdout and write it to the output file.
captureStdout = do
Stdout stdout <- cmd' [path] buildArgs
-- see Note [Capture stdout as a ByteString]
writeFileChangedBS output stdout
case builder of
Ar Pack stg -> do
useTempFile <- arSupportsAtFile stg
if useTempFile then runAr path buildArgs buildInputs
else runArWithoutTempFile path buildArgs buildInputs
Ar Unpack _ -> cmd' [Cwd output] [path] buildArgs
Autoreconf dir -> do
bash <- bashPath
cmd' [Cwd dir] [bash, path] buildArgs
Configure dir -> do
-- Inject /bin/bash into `libtool`, instead of /bin/sh,
-- otherwise Windows breaks. TODO: Figure out why.
bash <- bashPath
let env = AddEnv "CONFIG_SHELL" bash
cmd' env [Cwd dir] ["sh", path] buildOptions buildArgs
GenApply -> captureStdout
GenPrimopCode -> do
stdin <- readFile' input
Stdout stdout <- cmd' (Stdin stdin) [path] buildArgs
-- see Note [Capture stdout as a ByteString]
writeFileChangedBS output stdout
GhcPkg Copy _ -> do
Stdout pkgDesc <- cmd' [path]
[ "--expand-pkgroot"
, "--no-user-package-db"
, "describe"
, input -- the package name
]
cmd' (Stdin pkgDesc) [path] (buildArgs ++ ["-"])
GhcPkg Unregister _ -> do
-- unregistering is allowed to fail (e.g. when a package
-- isn't already present)
Exit _ <- cmd' [path] (buildArgs ++ [input])
return ()
HsCpp -> captureStdout
Make dir -> cmd' path ["-C", dir] buildArgs
Makeinfo -> do
cmd' [path] "--no-split" [ "-o", output] [input]
Xelatex ->
-- xelatex produces an incredible amount of output, almost
-- all of which is useless. Suppress it unless user
-- requests a loud build.
if verbosity >= Diagnostic
then cmd' [Cwd output] [path] buildArgs
else do (Stdouterr out, Exit code) <- cmd' [Cwd output] [path] buildArgs
when (code /= ExitSuccess) $ do
liftIO $ BSL.hPutStrLn stderr out
putFailure "xelatex failed!"
fail "xelatex failed"
Makeindex -> unit $ cmd' [Cwd output] [path] (buildArgs ++ [input])
Tar _ -> cmd' buildOptions [path] buildArgs
-- RunTest produces a very large amount of (colorised) output;
-- Don't attempt to capture it.
Testsuite RunTest -> do
Exit code <- cmd [path] buildArgs
when (code /= ExitSuccess) $ do
fail "tests failed"
_ -> cmd' [path] buildArgs
-- TODO: Some builders are required only on certain platforms. For example,
-- 'Objdump' is only required on OpenBSD and AIX. Add support for platform
-- specific optional builders as soon as we can reliably test this feature.
-- See https://github.com/snowleopard/hadrian/issues/211.
isOptional :: Builder -> Bool
isOptional = \case
Objdump -> True
-- alex and happy are not required when building source distributions
-- and ./configure will complain if they are not available when building in-tree
Happy -> True
Alex -> True
_ -> False
-- | Determine the location of a system 'Builder'.
systemBuilderPath :: Builder -> Action FilePath
systemBuilderPath builder = case builder of
Alex -> fromKey "alex"
Ar _ (Stage0 {})-> fromKey "system-ar"
Ar _ _ -> fromKey "ar"
Autoreconf _ -> stripExe =<< fromKey "autoreconf"
Cc _ (Stage0 {}) -> fromKey "system-cc"
Cc _ _ -> fromKey "cc"
-- We can't ask configure for the path to configure!
Configure _ -> return "configure"
Ghc _ (Stage0 {}) -> fromKey "system-ghc"
GhcPkg _ (Stage0 {}) -> fromKey "system-ghc-pkg"
Happy -> fromKey "happy"
HsCpp -> fromKey "hs-cpp"
Ld _ -> fromKey "ld"
-- MergeObjects Stage0 is a special case in case of
-- cross-compiling. We're building stage1, e.g. code which will be
-- executed on the host and hence we need to use host's merge
-- objects tool and not the target merge object tool.
-- Note, merge object tool is usually platform linker with some
-- parameters. E.g. building a cross-compiler on and for x86_64
-- which will target ppc64 means that MergeObjects Stage0 will use
-- x86_64 linker and MergeObject _ will use ppc64 linker.
MergeObjects (Stage0 {}) -> fromKey "system-merge-objects"
MergeObjects _ -> fromKey "merge-objects"
Make _ -> fromKey "make"
Makeinfo -> fromKey "makeinfo"
Nm -> fromKey "nm"
Objdump -> fromKey "objdump"
Patch -> fromKey "patch"
Python -> fromKey "python"
Ranlib -> fromKey "ranlib"
Testsuite _ -> fromKey "python"
Sphinx _ -> fromKey "sphinx-build"
Tar _ -> fromKey "tar"
Git _ -> fromKey "git"
Xelatex -> fromKey "xelatex"
Makeindex -> fromKey "makeindex"
Win32Tarballs _ -> fromKey "python"
Cabal _ _ -> fromKey "cabal"
_ -> error $ "No entry for " ++ show builder ++ inCfg
where
inCfg = " in " ++ quote configFile ++ " file."
fromKey key = do
let unpack = fromMaybe . error $ "Cannot find path to builder "
++ quote key ++ inCfg ++ " Did you skip configure?"
path <- unpack <$> lookupValue configFile key
if null path
then do
unless (isOptional builder) . error $ "Non optional builder "
++ quote key ++ " is not specified" ++ inCfg
return "" -- TODO: Use a safe interface.
else do
-- angerman: I find this lookupInPath rather questionable.
-- if we specify CC, LD, ... *without* a path, that is intentional
-- lookupInPath should be done by the person invoking the configure
-- script iif they want to have that full path, if they just want
-- some generic tool name (on purpose!) the build system should not
-- go behind their backs to add a path they likely never wanted.
fullPath <- lookupInPath path
case (windowsHost, hasExtension fullPath) of
(False, _ ) -> return path
(True , True ) -> fixAbsolutePathOnWindows fullPath
(True , False) -> fixAbsolutePathOnWindows fullPath <&> (<.> exe)
-- Without this function, on Windows we can observe a bad builder path
-- for 'autoreconf'. If the relevant system.config field is set to
-- /usr/bin/autoreconf in the file, the path that we read
-- is C:/msys64/usr/bin/autoreconf.exe. A standard msys2 set up happens
-- to have an executable named 'autoreconf' there, without the 'exe'
-- extension. Hence this function.
stripExe s = do
let sNoExt = dropExtension s
exists <- doesFileExist s
if exists then return s else return sNoExt
-- | Was the path to a given system 'Builder' specified in configuration files?
isSpecified :: Builder -> Action Bool
isSpecified = fmap (not . null) . systemBuilderPath
-- | Apply a patch by executing the 'Patch' builder in a given directory.
applyPatch :: FilePath -> FilePath -> Action ()
applyPatch dir patch = do
let file = dir -/- patch
needBuilders [Patch]
path <- builderPath Patch
putBuild $ "| Apply patch " ++ file
quietly $ cmd' [Cwd dir, FileStdin file] [path, "-p0"]
-- Note [cmd wrapper]
-- ~~~~~~~~~~~~~~~~~~
-- `cmd'` is a wrapper for Shake's `cmd` that allows us to customize what is
-- output in the terminal in case of failure.
--
-- However `cmd` is quite a complex function because:
--
-- 1) it relies on a CmdArguments type class to be variadic -- it can be called
-- with any number of arguments, as long as they are valid arguments -- and to
-- return either "Action r" or "IO r".
--
-- 2) its behavior depends on the returned "r" type! In particular, if it has
-- to return a value of type Exit or ExitCode, then it doesn't raise an
-- exception if the exit code isn't 0! It also doesn't echo the command
-- stdout/stderr if it is requested in a Stdout/Stderr/Stdouterr result. Result
-- types are handled via CmdResult type class.
--
-- To wrap `cmd` while keeping its behavior, we need to replicate some of these
-- type classes.
--
-- 1) CmdWrap corresponds to CmdArguments except that we do our own stuff in
-- the base case (i.e. in the instance for `Action r`).
--
-- 2) Sadly CmdResult internals aren't exposed by Shake, so when we get a
-- `CmdResult r => r` we can't tell anything about `r`. In particular, we can't
-- tell if an Exit or ExitCode value is returned in `r`. So we use our own
-- HasExit type class to provide the `hasExit` predicate that tells us if we
-- should throw an exception as `cmd` would do in case of failure or not.
--
-- Note [Capture stdout as a ByteString]
-- ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
-- As of shake-0.19.6, capturing a process stdout as a `String` using `Stdout`
-- mangles the encoding if it some other use of `Stdout` also captures it as a
-- `ByteString`; see <https://github.com/ndmitchell/shake/issues/828>. This
-- can cause us real problems, since `cmd'` (see Note [cmd wrapper]) *always*
-- captures stdout as a `ByteString`.
--
-- Fortunately, a simple workaround is to avoid capturing stdout as a `String`
-- in the first place. It’s usually unnecessary (and is in fact pointless work),
-- as most of the time the captured output is immediately written to a file, so
-- we can just treat it as an opaque binary stream.
-- | Wrapper for Shake's 'cmd'
--
-- See Note [cmd wrapper]
cmd' :: (Partial, CmdWrap args) => args :-> Action r
cmd' = cmdArgs mempty
-- See Note [cmd wrapper]
class HasExit a where
-- | Indicate if `a` is Exit or ExitCode
-- See Note [cmd wrapper]
hasExit :: a -> Bool
instance HasExit ExitCode where hasExit = const True
instance HasExit Exit where hasExit = const True
instance HasExit () where hasExit = const False
instance HasExit (Stdouterr a) where hasExit = const False
instance HasExit (Stdout a) where hasExit = const False
instance (HasExit a, HasExit b) => HasExit (a,b) where
hasExit (a,b) = hasExit a || hasExit b
instance (HasExit a, HasExit b, HasExit c) => HasExit (a,b,c) where
hasExit (a,b,c) = hasExit a || hasExit b || hasExit c
class CmdWrap t where
cmdArgs :: Partial => CmdArgument -> t
instance (IsCmdArgument a, CmdWrap r) => CmdWrap (a -> r) where
cmdArgs xs x = cmdArgs $ xs `mappend` toCmdArgument x
instance CmdWrap CmdArgument where
cmdArgs = id
instance (HasExit r, CmdResult r) => CmdWrap (Action r) where
cmdArgs (CmdArgument x) = do
verbosity <- getVerbosity
let real_args = mconcat
[ -- don't print stderr and stdout in command failure exception
toCmdArgument (WithStderr False)
, toCmdArgument (WithStdout False)
-- caller specified arguments come last to allow them to overload
-- the previous ones.
, CmdArgument x
]
(Stdout out, Stderr err, cmdline :: CmdLine, Exit code, r :: r) <- cmd real_args
if hasExit r
-- if the caller queries the exit code of the command, we don't do
-- anything here. In particular we don't throw an exception.
-- (this is used e.g. to allow ghc-pkg to fail to unregister)
-- See Note [cmd wrapper]
then pure r
else do
-- In every case, we only print both command outputs (stdout/stderr)
-- onto Hadrian's stderr because Hadrian's stdout may be piped into
-- another process and we don't want random command output to break
-- this.
--
-- For example, the result of "hadrian tool:ghc/Main.hs --flavour=ghc-in-ghci"
-- is directly passed as arguments for ghc in "hadrian/ghci-cabal" script.
let dump x = liftIO (BSL.hPutStr stderr x)
case code of
ExitSuccess -> do
-- Suppress stdout/stderr depending on Shake's verbosity setting
when (verbosity > Silent) (dump err)
when (verbosity >= Verbose) (dump out)
pure r
ExitFailure i -> do
putError ("Command line: " ++ fromCmdLine cmdline)
putError ("===> Command failed with error code: " ++ show i)
dump err
dump out
error "Command failed"
|