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

Re: [PATCH] ci: run `make sparse` as a GitHub workflow

From
Junio C Hamano <gitster@pobox.com>
Date
Jul 14, 2021, 22:27 UTC
Message-ID
<xmqqsg0gttt5.fsf@gitster.g>
In-Reply-To
<nycvar.QRO.7.76.6.2107142350570.59@tvgsbejvaqbjf.bet>
Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
Show 16 quoted lines
>> It wasn't quite obvious why we justify spending 370 minutes one more
>> time only to rerun 30-second job, though.
>
> True.
>
> And this is not a new problem. Every time anything happens in those
> `osx-gcc` or `osx-clang` jobs (e.g. that transient problem with the broken
> pipes in t5516 [*1*], that's a fun one), a full re-run is necessary, or
> else the commit and/or PR will remain marked as broken.
>
> In other words, while it is totally appropriate for me to explain this to
> you in this here thread because it came up as a tangent, it would be
> inappropriate to stick that explanation into this patch's commit message.
> We do not make a habit of adding tangents that came up during patch
> reviews into commit messages, and I do not intend to start such a habit
> here, either.

I do not agree; a brief mention "even though piling more and more on the primary workflow would make it even less convenient to re-run, it is already so bad that another one would not hurt too much more" would be a clue good enough to motivate others to do something about it if they feel like it.

Previous: Johannes SchindelinNext: Johannes Schindelin
Message 14 of 22 in “ci: run `make sparse` as a GitHub workflow”
  1. ci: run `make sparse` as a GitHub workflowJohannes Schindelin via GitGitGadget, Jul 13, 2021
  2. Đoàn Trần Công DanhJul 13, 2021
  3. Johannes SchindelinJul 14, 2021
  4. Philippe BlainJul 13, 2021
  5. Johannes SchindelinJul 14, 2021
  6. Johannes SchindelinJul 14, 2021
  7. Ramsay JonesJul 16, 2021
  8. Junio C HamanoJul 13, 2021
  9. Johannes SchindelinJul 14, 2021
  10. Junio C HamanoJul 14, 2021
  11. Johannes SchindelinJul 14, 2021
  12. Junio C HamanoJul 14, 2021
  13. Johannes SchindelinJul 14, 2021
  14. Junio C HamanoJul 14, 2021
  15. Johannes SchindelinJul 16, 2021
  16. Junio C HamanoJul 16, 2021
  17. ci: run `make sparse` as part of the GitHub workflowJohannes Schindelin via GitGitGadget, Jul 14, 2021
  18. ci: run "apt-get update" before "apt-get install"Jeff King, Jul 26, 2021
  19. Jeff KingJul 26, 2021
  20. Junio C HamanoJul 26, 2021
  21. ci: run `make sparse` as part of the GitHub workflowJohannes Schindelin via GitGitGadget, Jul 26, 2021
  22. Junio C HamanoJul 26, 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.