git/list[1] front-page[2] threads[3] people[4] search[5] about
 

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

From
Derrick Stolee <stolee@gmail.com>
Date
Jul 23, 2018, 13:25 UTC
Message-ID
<583d2499-c4f2-47a1-48b0-6beb2d4d9d9f@gmail.com>
In-Reply-To
<20180723130230.22491-1-szeder.dev@gmail.com>
On 7/23/2018 9:02 AM, SZEDER Gábor wrote:
Show 27 quoted lines
> 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@microsoft.com>
Previous: Derrick Stolee
Message 9 of 9 in “travis-ci: fail if Coccinelle found something to transform”
  1. 0/2 travis-ci: fail if Coccinelle found something to transformSZEDER Gábor, Jul 23, 2018
  2. 1/2 travis-ci: run Coccinelle static analysis with two parallel jobsSZEDER Gábor, Jul 23, 2018
  3. 2/2 travis-ci: fail if Coccinelle static analysis found something to transformSZEDER Gábor, Jul 23, 2018
  4. Derrick StoleeJul 23, 2018
  5. SZEDER GáborJul 26, 2018
  6. Junio C HamanoJul 26, 2018
  7. 3/2 squash! bisect--helper: `bisect_start` shell function partially in CSZEDER Gábor, Jul 23, 2018
  8. Derrick StoleeJul 23, 2018
  9. Derrick StoleeJul 23, 2018

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.