Re: Coverage with make coverage-html is broken on latest Debian using lcov v2 - Mailing list pgsql-hackers

From Tom Lane
Subject Re: Coverage with make coverage-html is broken on latest Debian using lcov v2
Date
Msg-id 77254.1791045569@sss.pgh.pa.us
Whole thread
In response to Re: Coverage with make coverage-html is broken on latest Debian using lcov v2  (Álvaro Herrera <alvherre@kurilemu.de>)
Responses Re: Coverage with make coverage-html is broken on latest Debian using lcov v2
Re: Coverage with make coverage-html is broken on latest Debian using lcov v2
List pgsql-hackers
[ This thread went quiet, but you reminded me of it in answering
Pierre Forstmann's nearby question ]

=?utf-8?Q?=C3=81lvaro?= Herrera <alvherre@kurilemu.de> writes:
> Handling this part with your 0001 seems reasonable to me.  I think we
> should backpatch that one.

Agreed, and it doesn't look like that got done, so I'll go do it now.

> I'm not sure sure about the 0002 patch though.  It builds in the
> assumption that lcov is broken and that we're going to ignore these
> warnings by default [forever].  Do we really want to bake those flags
> into our build system?

That bothers me too, mainly because I foresee a risk of the switches
hiding genuine problems somewhere down the road.  Also the switches
Narek proposes don't match what I've found to be necessary on my
own installation (so maybe there is a gcov version dependency here
too?).

For the moment I'm content to insert the --ignore-errors flags
manually.  The 0001 patch should at least reduce the noise level
a bit.

            regards, tom lane



pgsql-hackers by date:

Previous
From: Greg Burd
Date:
Subject: Re: [PATCH] Corruption Issue: Fix missing tts_tid in ExecForceStoreHeapTuple
Next
From: Greg Burd
Date:
Subject: Re: Tepid: selective index updates for heap relations