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

Re: [PATCH] ci: add job for gcc-4.8 to GitHub Actions

From
Jeff King <peff@peff.net>
Date
Aug 16, 2021, 16:18 UTC
Message-ID
<YRqP7J7Gu0VmB6q2@coredump.intra.peff.net>
In-Reply-To
<444e7410-c9c9-1b90-da5f-d6862a35c1fc@gmail.com>
On Mon, Aug 16, 2021 at 12:06:46PM -0400, Derrick Stolee wrote:
Show 6 quoted lines
> Forgive me, I probably missed a discussion about this
> somewhere else on the list, but...
> 
> Could you describe why we want GCC 4.8 in our CI? Is that a
> compiler version that we officially support? What kind of
> syntax triggers a problem on 4.8 versus latest?

Try fb9d7431cf (travis-ci: build with GCC 4.8 as well, 2019-07-18). (found with "git log -Sgcc-4.8 ci"). The gist of it is to find variable declarations in for-loops.

IMHO it may be more trouble than it's worth. If we can't find a compiler that complains on this construct, then maybe it is not a construct worth worrying too much about.

-Peff
Previous: Derrick StoleeNext: SZEDER Gábor
Message 3 of 7 in “ci: add job for gcc-4.8 to GitHub Actions”
  1. ci: add job for gcc-4.8 to GitHub ActionsCarlo Marcelo Arenas Belón, Aug 16, 2021
  2. Derrick StoleeAug 16, 2021
  3. Jeff KingAug 16, 2021
  4. SZEDER GáborAug 17, 2021
  5. Jeff KingAug 17, 2021
  6. Junio C HamanoAug 17, 2021
  7. Carlo ArenasAug 16, 2021

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.