Skip to content

travis-ci: fail if Coccinelle found something to transform

SZEDER Gábor git 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest2 patches 6 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/2travis-ci: run Coccinelle static analysis with two parallel jobsci/run-static-analysis.sh+1 -10 replies
2/2travis-ci: fail if Coccinelle static analysis found something to transformci/run-static-analysis.sh+19 -03 replies

Reviews trailers per patch

None of the 2 patches has a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchDerrick Stolee
SeriesR-b
1/2
2/2

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

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

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(-)
Signature · 1 line
2.18.0.408.g42635c01bc