On Wed, 2017-04-05 at 06:53 -0400, John Ferlan wrote:
> I thought that John still had a nightly coverity job running
that
> would trigger the -DSTATIC_ANALYSIS codepaths. If that's not the case, then
we'd wnat to look
> at enabling that in one of the centos CI jobs.
I still have a run of Coverity every night although I have been less
diligent about checking errors lately. Mainly because generally things
that are considered a false positive were being rejected when I posted
patches. I keep about 20 patches in a local branch.
There was a point quite a few months ago where my nightly build started
failing because either I changed the Coverity version or the compiler
version on my laptop - cannot recall exactly. That resulted in a bunch
more local patches until I finally had too many and posted that pile
late last month. Enabling static analysis in CI builds was something I
suggested in my cover - although now I've done it for my every day work
environment so I will see the problems much sooner.
Did we ever get in touch with the Coverity folks about
enabling Coverity Scan[1] for libvirt? QEMU is already
part of the program.
[1]
https://scan.coverity.com/
--
Andrea Bolognani / Red Hat / Virtualization