Re: [PATCH 0/2] travis-ci: fail if Coccinelle found something to transform

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 7/23/2018 9:02 AM, SZEDER Gábor wrote:
The first patch makes the static analysis build job on Travis CI
faster by running it with 'make -j2'.

The second patch makes it more more useful by failing the build job if
Coccinelle finds something to transform, thereby drawing our attention
to undesired code patterns trying to enter the codebase.


With these patches applied, the static analysis build job would fail
on current 'pu' because of two small issues on two branches:

   - js/range-diff: Dscho has sent out v4 of this series over the
     weekend, which already incorporates Coccinelle's suggestion, so
     it's basically done.
- pb/bisect-helper-2: this topic has not seen an update in about 9
     months, so I'll send a followup patch 3/2 to be applied on top or
     squashed in, whichever is deemed better.


SZEDER Gábor (2):
   travis-ci: run Coccinelle static analysis with two parallel jobs
   travis-ci: fail if Coccinelle static analysis found something to
     transform

  ci/run-static-analysis.sh | 21 ++++++++++++++++++++-
  1 file changed, 20 insertions(+), 1 deletion(-)

Thanks, Szeder.

I agree that we should use all possible automation as part of CI to catch things early. I like the direction here.

Reviewed-by: Derrick Stolee <dstolee@xxxxxxxxxxxxx>




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux