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

Re: [PATCH v3] CI: select CC based on CC_PACKAGE (again)

From
Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Date
Apr 22, 2022, 11:43 UTC
Message-ID
<220422.86r15p4a34.gmgdl@evledraar.gmail.com>
In-Reply-To
<1654562506.42130.1650619559902@apps.talktalk.co.uk>
On Fri, Apr 22 2022, Phillip Wood wrote:
Show 36 quoted lines
> Hi Ævar
>
>> On 22 April 2022 at 10:20 Ævar Arnfjörð Bjarmason <avarab@gmail.com> wrote:
>> 
>> 
>> Fix a regression in 707d2f2fe86 (CI: use "$runs_on_pool", not
>> "$jobname" to select packages & config, 2021-11-23).
>> 
>> In that commit I changed CC=gcc from CC=gcc-9, but on OSX the "gcc" in
>> $PATH points to clang, we need to use gcc-9 instead. Likewise for the
>> linux-gcc job CC=gcc-8 was changed to the implicit CC=gcc, which would
>> select GCC 9.4.0 instead of GCC 8.4.0.
>> 
>> Furthermore in 25715419bf4 (CI: don't run "make test" twice in one
>> job, 2021-11-23) when the "linux-TEST-vars" job was split off from
>> "linux-gcc" the "cc_package: gcc-8" line was copied along with
>> it, so its "cc_package" line wasn't working as intended either.
>> 
>> As a table, this is what's changed by this commit, i.e. it only
>> affects the linux-gcc, linux-TEST-vars and osx-gcc jobs:
>> 
>> 	|-------------------+-----------+-------------------+-------+-------|
>> 	| jobname           | vector.cc | vector.cc_package | old   | new   |
>> 	|-------------------+-----------+-------------------+-------+-------|
>> 	| linux-clang       | clang     | -                 | clang | clang |
>> 	| linux-sha256      | clang     | -                 | clang | clang |
>> 	| linux-gcc         | gcc       | gcc-8             | gcc   | gcc-8 |
>> 	| osx-clang         | clang     | -                 | clang | clang |
>> 	| osx-gcc           | gcc       | gcc-9             | clang | gcc-9 |
>> 	| linux-gcc-default | gcc       | -                 | gcc   | gcc   |
>> 	| linux-TEST-vars   | gcc       | gcc-8             | gcc   | gcc-8 |
>> 	|-------------------+-----------+-------------------+-------+-------|
>
> Having this table is helpful. I do find it confusing that we're still setting 
> CC in the main.yaml and then overriding it in lib.sh. I think it would make 
> things clearer if we got find of the cc settings in the job matrix.
Yes, this stand-alone patch is just a band-aid.

Do you like the state at the tip of my larger CI series? It's only set in ci/lib.sh there, or in the case of the package we need the installation script is tasked with it now.

Previous: Phillip WoodNext: Junio C Hamano
Message 15 of 18 in “CI: select CC based on CC_PACKAGE (again)”
  1. CI: select CC based on CC_PACKAGE (again)Ævar Arnfjörð Bjarmason, Apr 21, 2022
  2. Phillip WoodApr 21, 2022
  3. CI: select CC based on CC_PACKAGE (again)Ævar Arnfjörð Bjarmason, Apr 21, 2022
  4. Carlo ArenasApr 21, 2022
  5. Junio C HamanoApr 21, 2022
  6. Carlo ArenasApr 21, 2022
  7. Junio C HamanoApr 21, 2022
  8. Junio C HamanoApr 21, 2022
  9. Ævar Arnfjörð BjarmasonApr 21, 2022
  10. Junio C HamanoApr 21, 2022
  11. Ævar Arnfjörð BjarmasonApr 21, 2022
  12. Junio C HamanoApr 21, 2022
  13. CI: select CC based on CC_PACKAGE (again)Ævar Arnfjörð Bjarmason, Apr 22, 2022
  14. Phillip WoodApr 22, 2022
  15. Ævar Arnfjörð BjarmasonApr 22, 2022
  16. Junio C HamanoApr 22, 2022
  17. Junio C HamanoApr 22, 2022
  18. Ævar Arnfjörð BjarmasonApr 22, 2022

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.