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

Re: [PATCH 1/1] t7406: avoid failures solely due to timing issues

From
Junio C Hamano <gitster@pobox.com>
Date
Jul 23, 2018, 21:31 UTC
Message-ID
<xmqqlga14p16.fsf@gitster-ct.c.googlers.com>
In-Reply-To
<CAGZ79kag=YC6PxOnBbVAqY92dNd5yuqfvash-oPVrEktowNbVA@mail.gmail.com>
Stefan Beller <sbeller@google.com> writes:
Show 9 quoted lines
> This test was last touched in c66410ed32a (submodule init:
> redirect stdout to stderr, 2016-05-02), merged as f2c96ceb57a
> (Merge branch 'sb/submodule-init', 2016-05-17)
>
> The parallelizing effort was made before that
> (2335b870fa7 (submodule update: expose parallelism to the
> user, 2016-02-29) via bdebbeb3346 (Merge branch
> 'sb/submodule-parallel-update', 2016-04-06)), but did not
> review existing tests for the newly introduced raciness.

Ah, that explains what I was scratching my head about. I somehow thought that the parallel update broke an otherwise working test written back in the serial days, but it seems it was the other way around. The test was simply careless.

Nice to have the two-year old issue fixed.  Thanks, all.
Previous: Stefan BellerNext: Junio C Hamano
Message 5 of 7 in “t7406: avoid failures solely due to timing issues”
  1. 0/1 t7406: avoid failures solely due to timing issuesJohannes Schindelin via GitGitGadget, Jul 23, 2018
  2. 1/1 t7406: avoid failures solely due to timing issuesJohannes Schindelin via GitGitGadget, Jul 23, 2018
  3. Junio C HamanoJul 23, 2018
  4. Stefan BellerJul 23, 2018
  5. Junio C HamanoJul 23, 2018
  6. Junio C HamanoJul 23, 2018
  7. Torsten BögershausenJul 24, 2018

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.