Re: PSA: Local static analysis builds on Linux and Mac OS X

2015-10-16 Thread Ehsan Akhgari
On 2015-10-16 2:09 AM, Nicholas Nethercote wrote: On Fri, Oct 16, 2015 at 5:13 AM, Ehsan Akhgari wrote: Is there an easy way to determine if that package has been installed? The easiest way is to turn it on in a build and see if the build succeeds. On my stock Ubuntu 15.04 box it failed...

Re: PSA: Local static analysis builds on Linux and Mac OS X

2015-10-15 Thread Nicholas Nethercote
On Fri, Oct 16, 2015 at 5:13 AM, Ehsan Akhgari wrote: >> >> Is there an easy way to determine if that package has been installed? > > The easiest way is to turn it on in a build and see if the build succeeds. On my stock Ubuntu 15.04 box it failed... >> And if it's not installed, is there an eas

Re: PSA: Local static analysis builds on Linux and Mac OS X

2015-10-15 Thread Andrew Halberstadt
On 15/10/15 01:59 PM, Ehsan Akhgari wrote: On 2015-10-14 9:18 PM, Mike Hommey wrote: BTW, since we're growing the set of checks the plugin does, what do we do to ensure that it doesn't add too much overhead to the compilation time? Currently nothing beyond taking care to not add slow checks.

Re: PSA: Local static analysis builds on Linux and Mac OS X

2015-10-15 Thread Ehsan Akhgari
On 2015-10-14 9:32 PM, Gregory Szorc wrote: On Wed, Oct 14, 2015 at 6:07 PM, Ehsan Akhgari mailto:ehsan.akhg...@gmail.com>> wrote: Last weekend, I finished upgrading our Linux and OS X debug and opt static analysis builds ("S" jobs on TreeHerder) to the latest revision of clang 3

Re: PSA: Local static analysis builds on Linux and Mac OS X

2015-10-15 Thread Ehsan Akhgari
On 2015-10-15 5:30 AM, Nicholas Nethercote wrote: On Wed, Oct 14, 2015 at 6:07 PM, Ehsan Akhgari wrote: On Linux, most other recent versions of clang should work too, such as the one you usually get from your distro, provided that the dev package for LLVM/Clang has been installed. Is there a

Re: PSA: Local static analysis builds on Linux and Mac OS X

2015-10-15 Thread Ehsan Akhgari
On 2015-10-14 9:18 PM, Mike Hommey wrote: BTW, since we're growing the set of checks the plugin does, what do we do to ensure that it doesn't add too much overhead to the compilation time? Currently nothing beyond taking care to not add slow checks. Since we're talking about the build speed he

Re: PSA: Local static analysis builds on Linux and Mac OS X

2015-10-15 Thread Nicholas Nethercote
On Wed, Oct 14, 2015 at 6:07 PM, Ehsan Akhgari wrote: > > On Linux, most other recent versions of clang should work too, such as the > one you usually get from your distro, provided that the dev package for > LLVM/Clang has been installed. Is there an easy way to determine if that package has bee

Re: PSA: Local static analysis builds on Linux and Mac OS X

2015-10-14 Thread Eric Rescorla
On Wed, Oct 14, 2015 at 6:37 PM, Martin Thomson wrote: > On Wed, Oct 14, 2015 at 6:32 PM, Gregory Szorc wrote: > > As you stated, this helps detect errors earlier during development, which > > is a huge win. Is there a good reason configure doesn't enable the clang > > plugin by default? > > > Y

Re: PSA: Local static analysis builds on Linux and Mac OS X

2015-10-14 Thread Martin Thomson
On Wed, Oct 14, 2015 at 6:32 PM, Gregory Szorc wrote: > As you stated, this helps detect errors earlier during development, which > is a huge win. Is there a good reason configure doesn't enable the clang > plugin by default? Yes please. If the checks are expensive, ac_add_options --disable-cla

Re: PSA: Local static analysis builds on Linux and Mac OS X

2015-10-14 Thread Gregory Szorc
On Wed, Oct 14, 2015 at 6:07 PM, Ehsan Akhgari wrote: > Last weekend, I finished upgrading our Linux and OS X debug and opt static > analysis builds ("S" jobs on TreeHerder) to the latest revision of clang > 3.7. > > Now, if you use the latest version of clang (3.7, which is available from < > ht

Re: PSA: Local static analysis builds on Linux and Mac OS X

2015-10-14 Thread Mike Hommey
On Wed, Oct 14, 2015 at 09:07:10PM -0400, Ehsan Akhgari wrote: > Last weekend, I finished upgrading our Linux and OS X debug and opt static > analysis builds ("S" jobs on TreeHerder) to the latest revision of clang > 3.7. > > Now, if you use the latest version of clang (3.7, which is available fro

PSA: Local static analysis builds on Linux and Mac OS X

2015-10-14 Thread Ehsan Akhgari
Last weekend, I finished upgrading our Linux and OS X debug and opt static analysis builds ("S" jobs on TreeHerder) to the latest revision of clang 3.7. Now, if you use the latest version of clang (3.7, which is available from < http://llvm.org/releases/download.html#3.7.0> in binary form), you ca