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@xxxxxxxxxx> wrote: > > On Thu, Oct 10, 2019 at 3:41 PM Yuval Lifshitz <ylifshit@xxxxxxxxxx> 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@xxxxxxxxxx> 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@xxxxxxx > >> To unsubscribe send an email to dev-leave@xxxxxxx > > > > -- > Cheers, > Brad
