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

Re: [PATCH v1] travis-ci: override CFLAGS properly, add -Wdeclaration-after-statement

From
Junio C Hamano <gitster@pobox.com>
Date
Feb 9, 2016, 18:42 UTC
Message-ID
<xmqq4mdh4tel.fsf@gitster.mtv.corp.google.com>
In-Reply-To
<9FBB5AFB-D44C-43A1-A0C5-A5A45675A122@gmail.com>
Lars Schneider <larsxschneider@gmail.com> writes:
> Jeff Merkey made me aware of http://kernelnewbies.org/FirstKernelPatch [2]
> where I found checkpatch.pl [3]. Would it make sense to check all commits
> that are not in next/master/maint with this script on Travis-CI?

That does not help very much. These changes are already shown to people and dirtied their eyes, and most likely I've already have wasted time tweaking the glitches out locally. The damage has already been done.

It would make a lot of sense if the checkpatch is called inside Roberto Tyley's "pull-request-to-patch-submission" thing, though.

Previous: Ramsay JonesNext: Stefan Beller
Message 7 of 10 in “travis-ci: override CFLAGS properly, add -Wdeclaration-after-statement”
  1. travis-ci: override CFLAGS properly, add -Wdeclaration-after-statementlarsxschneider@gmail.com, Feb 8, 2016
  2. Jeff KingFeb 8, 2016
  3. Lars SchneiderFeb 9, 2016
  4. Jeff KingFeb 9, 2016
  5. Junio C HamanoFeb 9, 2016
  6. Ramsay JonesFeb 9, 2016
  7. Junio C HamanoFeb 9, 2016
  8. Stefan BellerFeb 9, 2016
  9. Roberto TyleyFeb 9, 2016
  10. Junio C HamanoFeb 9, 2016

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.