Oh, yes, you're obviously right! Somehow I missed that line...

Interdiff:

diff --git a/Makefile.am b/Makefile.am
index b8d2e26..c24e6ad 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -2146,9 +2146,8 @@ hs-coverage: $(haskell_tests) test/hs/hpc-htools
test/hs/hpc-mon-collector
        @mkdir_p@ $(COVERAGE_HS_DIR)
        hpc sum --union $(HPCEXCL) \
          htest.tix hpc-htools.tix hpc-mon-collector.tix > coverage-hs.tix
-       hpc report coverage-hs.tix > $(COVERAGE_HS_DIR)/report.txt
        hpc markup --destdir=$(COVERAGE_HS_DIR) coverage-hs.tix
-       hpc report coverage-hs.tix
+       hpc report coverage-hs.tix | tee $(COVERAGE_HS_DIR)/report.txt
        $(LN_S) -f hpc_index.html $(COVERAGE_HS_DIR)/index.html

 # Special "kind-of-QA" target for htools, needs special setup (all


On Wed, Aug 7, 2013 at 9:35 PM, Iustin Pop <[email protected]> wrote:

> On Wed, Aug 07, 2013 at 05:54:40PM +0200, Thomas Thrainer wrote:
> > In order to generate a coverage graph, we generate also the coverage
> > data as text format. This eases parsing on Buildbot.
> >
> > Signed-off-by: Thomas Thrainer <[email protected]>
> > ---
> >  Makefile.am | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/Makefile.am b/Makefile.am
> > index 7d3ca00..b8d2e26 100644
> > --- a/Makefile.am
> > +++ b/Makefile.am
> > @@ -2146,6 +2146,7 @@ hs-coverage: $(haskell_tests) test/hs/hpc-htools
> test/hs/hpc-mon-collector
> >       @mkdir_p@ $(COVERAGE_HS_DIR)
> >       hpc sum --union $(HPCEXCL) \
> >         htest.tix hpc-htools.tix hpc-mon-collector.tix > coverage-hs.tix
> > +     hpc report coverage-hs.tix > $(COVERAGE_HS_DIR)/report.txt
> >       hpc markup --destdir=$(COVERAGE_HS_DIR) coverage-hs.tix
> >       hpc report coverage-hs.tix
>
> As this is already run, would it make sense to collapse the two runs via
> tee?
>
> hpc report coverage-hs.tix | tee $(COVERAGE_HS_DIR)/report.txt
>
> iustin
>



-- 
Thomas Thrainer | Software Engineer | [email protected] |

Google Germany GmbH
Dienerstr. 12
80331 München

Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg
Geschäftsführer: Graham Law, Christine Elizabeth Flores

Reply via email to