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

Re: [PATCH 2/3] ci: update Perforce version to r23.2

From
Patrick Steinhardt <ps@pks.im>
Date
Jul 24, 2024, 09:01 UTC
Message-ID
<ZqDC7O985AUUg3Pd@tanuki>
In-Reply-To
<da9c4d1e-0fce-3bf3-c35b-32704a8a2129@gmx.de>
On Wed, Jul 24, 2024 at 10:39:54AM +0200, Johannes Schindelin wrote:
Show 32 quoted lines
> Hi Patrick,
> 
> On Tue, 23 Jul 2024, Patrick Steinhardt wrote:
> 
> > Update our Perforce version from r21.2 to r23.2. Note that the updated
> > version is not the newest version. Instead, it is the last version where
> > the way that Perforce is being distributed remains the same as in r21.2.
> > Newer releases stopped distributing p4 and p4d executablesas well as the
> > macOS archives directly and would thus require more work.
> 
> An alternative would be to simply stop installing `p4` in CI. I would
> actually be in favor of that, for multiple reasons:
> 
> - The pace of reviews and integration of `git-p4` patches has slowed down
>   over the couple of years. For example,
>   https://lore.kernel.org/git/20210510183638.156a6b1d@ado-tr/ has not seen
>   any traction in over three years (most likely because we no longer have
>   any active contributor with a vested interest in `git-p4`), and
>   https://github.com/gitgitgadget/git/pull/1028 and
>   https://github.com/gitgitgadget/git/pull/1070 have not even been
>   submitted to the Git mailing list (most likely because of the hurdles to
>   contribute).
> 
> - Over the years, it has been made harder and harder to install Perforce
>   in CI. I spent a good deal of time trying to keep the Homebrew taps up
>   to date (which was hard because Perforce kept replacing the archive
>   behind that URL with newer versions, which always broke Homebrew's SHA
>   check until it was adjusted accordingly).
> 
> - The `git-p4` tests use quite a bit of time and electricity in all those
>   CI builds. Therefore, it seems desirable to me to stop running these
>   tests as part of the CI builds.

I don't think that is a good idea. If we stop installing p4, the result is that _nobody_ will ever run the tests at all. The tests, and by extension git-p4 itself, would start to bitrot and we wouldn't notice any kind of regressions at all anymore.

If we want to consider going down that route, I'd rather say we should do it all or nothing: either we rip out git-p4 and the tests, or we leave both of them in. I couldn't care less about git-p4 itself, so I would not mind ripping it out altogether. But there may be users of this script out there that do care, so I don't want to make that decision unilaterally.

Patrick
Previous: Johannes SchindelinNext: Junio C Hamano
Message 7 of 19 in “Improvements for Perforce tests”
  1. 0/3 Improvements for Perforce testsPatrick Steinhardt, Jul 23, 2024
  2. 1/3 t98xx: fix Perforce tests with p4d r23 and newerPatrick Steinhardt, Jul 23, 2024
  3. Justin ToblerJul 30, 2024
  4. Patrick SteinhardtJul 31, 2024
  5. 2/3 ci: update Perforce version to r23.2Patrick Steinhardt, Jul 23, 2024
  6. Johannes SchindelinJul 24, 2024
  7. Patrick SteinhardtJul 24, 2024
  8. Junio C HamanoJul 24, 2024
  9. Patrick SteinhardtJul 30, 2024
  10. Justin ToblerJul 30, 2024
  11. Patrick SteinhardtJul 31, 2024
  12. 3/3 t98xx: mark Perforce tests as memory-leak freePatrick Steinhardt, Jul 23, 2024
  13. Justin ToblerJul 30, 2024
  14. 0/3 Improvements for Perforce testsPatrick Steinhardt, Jul 31, 2024
  15. 1/3 t98xx: fix Perforce tests with p4d r23 and newerPatrick Steinhardt, Jul 31, 2024
  16. 2/3 ci: update Perforce version to r23.2Patrick Steinhardt, Jul 31, 2024
  17. 3/3 t98xx: mark Perforce tests as memory-leak freePatrick Steinhardt, Jul 31, 2024
  18. Justin ToblerJul 31, 2024
  19. Junio C HamanoJul 31, 2024

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.