| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | Álvaro Herrera <alvherre(at)kurilemu(dot)de> |
| Cc: | Narek Galstyan <narek(dot)galstyan(at)enterprisedb(dot)com>, pgsql-hackers(at)postgresql(dot)org, "narekg(at)berkeley(dot)edu" <narekg(at)berkeley(dot)edu>, ngalstyan4(at)gmail(dot)com |
| Subject: | Re: Coverage with make coverage-html is broken on latest Debian using lcov v2 |
| Date: | 2026-10-03 16:39:29 |
| Message-ID: | 77254.1791045569@sss.pgh.pa.us |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | 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(at)kurilemu(dot)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
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Greg Burd | 2026-10-03 16:52:56 | Re: Tepid: selective index updates for heap relations |
| Previous Message | Greg Burd | 2026-10-03 16:29:12 | Re: [PATCH] Corruption Issue: Fix missing tts_tid in ExecForceStoreHeapTuple |