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

Re: ab/mark-leak-free-tests-even-more (was: What's cooking in git.git (Nov 2021, #04; Sun, 14))

From
Elijah Newren <newren@gmail.com>
Date
Nov 15, 2021, 22:16 UTC
Message-ID
<CABPp-BES7SX06i3+AZS2gxkGdbS6nHy5r00E_WhMaSpZ6PdjrA@mail.gmail.com>
In-Reply-To
<211115.861r3hrz61.gmgdl@evledraar.gmail.com>

On Mon, Nov 15, 2021 at 12:35 PM Ævar Arnfjörð Bjarmason <avarab@gmail.com> wrote:

Show 32 quoted lines
>
> On Mon, Nov 15 2021, Junio C Hamano wrote:
>
> > * ab/mark-leak-free-tests-even-more (2021-11-01) 15 commits
> >  - leak tests: mark some fast-import tests as passing with SANITIZE=leak
> >  - leak tests: mark some config tests as passing with SANITIZE=leak
> >  - leak tests: mark some status tests as passing with SANITIZE=leak
> >  - leak tests: mark some clone tests as passing with SANITIZE=leak
> >  - leak tests: mark some add tests as passing with SANITIZE=leak
> >  - leak tests: mark some diff tests as passing with SANITIZE=leak
> >  - leak tests: mark some apply tests as passing with SANITIZE=leak
> >  - leak tests: mark some notes tests as passing with SANITIZE=leak
> >  - leak tests: mark some update-index tests as passing with SANITIZE=leak
> >  - leak tests: mark some rev-parse tests as passing with SANITIZE=leak
> >  - leak tests: mark some rev-list tests as passing with SANITIZE=leak
> >  - leak tests: mark some misc tests as passing with SANITIZE=leak
> >  - leak tests: mark most gettext tests as passing with SANITIZE=leak
> >  - leak tests: mark "sort" test as passing SANITIZE=leak
> >  - leak tests: mark a read-tree test as passing SANITIZE=leak
> >
> >  More tests are marked as leak-free.
> >
> >  Will merge to 'next'?
>
> That would be great. I've got a rather large queue of leak fixes
> pending, and due to marking up the tests incrementally they've pretty
> much got to go in sequence.
>
> For this sort of topic that's just marking existing tests as running
> under the linux-leaks job I'd think it could proceed relatively quickly
> after it's not been causing test smoke on master/next/seen for a bit,
> which with the release cycle has been a couple of weeks in this case.

I just gave it my belated review; I think it's helpful and also pretty simple. Merging to next seems like a good plan to me.

Previous: Ævar Arnfjörð BjarmasonNext: Ævar Arnfjörð Bjarmason
Message 3 of 6 in “What's cooking in git.git (Nov 2021, #04; Sun, 14)”
  1. Junio C HamanoNov 15, 2021
  2. ab/mark-leak-free-tests-even-more (was: What's cooking in git.git (Nov 2021, #04; Sun, 14))Ævar Arnfjörð Bjarmason, Nov 15, 2021
  3. Elijah NewrenNov 15, 2021
  4. ab/only-single-progress-at-once (was: What's cooking in git.git (Nov 2021, #04; Sun, 14))Ævar Arnfjörð Bjarmason, Nov 15, 2021
  5. Fabian StelzerNov 15, 2021
  6. ab/refs-errno-cleanup + hn/reftable (was: What's cooking in git.git (Nov 2021, #04; Sun, 14))Ævar Arnfjörð Bjarmason, Nov 15, 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.