Hi Brad, Thanks for providing the script! I added 2 of the targets to the cmake system in this PR [1]. Still working on scan-build. As a side note, on Fedora30, the iwyu version is pretty old, and create many false positives, I had to build and install from source. Yuval [1] https://github.com/ceph/ceph/pull/31579 On Fri, Oct 11, 2019 at 8:45 AM Brad Hubbard <bhubbard@redhat.com> wrote:
On Thu, Oct 10, 2019 at 3:41 PM Yuval Lifshitz <ylifshit@redhat.com> wrote:
This is awesome!
First thing to note is these scans each take a long time to run.
How difficult would it be to add these as cmake targets?
With Coverity, currently impossible since the only version I can find that works is not publicly available.
As for the others I use the following script to run them so it wouldn't be that hard I guess. There's some changes in there at the moment to try and get them to only scan 'ceph code' (not submodule code) but that seems to be confusing scan-build as it currently produces zero results. I have some work to do there and there seems to be a lot of maintenance work around these scans. I'm not sure how much bang for our buck we would get by adding any of them as cmake targets.
On Thu, Oct 10, 2019 at 8:18 AM Brad Hubbard <bhubbard@redhat.com> wrote:
Latest static analyser results are up on http://people.redhat.com/bhubbard/
Weekly Fedora Copr builds are at https://copr.fedorainfracloud.org/coprs/badone/ceph-weeklies/
-- Cheers, Brad _______________________________________________ Dev mailing list -- dev@ceph.io To unsubscribe send an email to dev-leave@ceph.io
-- Cheers, Brad