diff options
| author | Ben Gamari <ben@smart-cactus.org> | 2020-10-31 12:29:28 -0400 |
|---|---|---|
| committer | Ben Gamari <ben@smart-cactus.org> | 2020-10-31 12:33:02 -0400 |
| commit | b7c42ac6bf5c2409ca6a4f1b37d989a9610fc07f (patch) | |
| tree | cd77e3d1a638c3b8789190669659017aeb93d519 | |
| parent | eb368078204bceba51e3c80e387ce1d776762cfd (diff) | |
| download | haskell-wip/hadrian-runtest.tar.gz | |
hadrian: Don't capture RunTest outputwip/hadrian-runtest
There are a few reasons why capturing the output of the RunTest builder
is undesirable:
* there is a large amount of output which then gets unnecessarily
duplicated by Hadrian if the builder fails
* the output may contain codepoints which are unrepresentable in the
current codepage on Windows, causing Hadrian to crash
* capturing the output causes the testsuite driver to disable
its colorisation logic, making the output less legible.
| -rw-r--r-- | hadrian/src/Builder.hs | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/hadrian/src/Builder.hs b/hadrian/src/Builder.hs index 18a64b372b..3419dee422 100644 --- a/hadrian/src/Builder.hs +++ b/hadrian/src/Builder.hs @@ -304,6 +304,11 @@ instance H.Builder Builder where Makeindex -> unit $ cmd' [Cwd output] [path] (buildArgs ++ [input]) Tar _ -> cmd' buildOptions echo [path] buildArgs + + -- RunTest produces a very large amount of (colorised) output; + -- Don't attempt to capture it. + RunTest -> cmd echo [path] buildArgs + _ -> cmd' echo [path] buildArgs -- TODO: Some builders are required only on certain platforms. For example, |