Re: [PATCH v2 00/21] builtin/config: remove global state
- From
Patrick Steinhardt <ps@pks.im>
- Date
- May 14, 2024, 14:52 UTC
- Message-ID
- <ZkN6s1OYfJOpy7x3@tanuki>
- In-Reply-To
- <xmqqzfss78kw.fsf@gitster.g>
On Tue, May 14, 2024 at 07:48:47AM -0700, Junio C Hamano wrote:
Show 18 quoted lines
> Patrick Steinhardt <ps@pks.im> writes: > > > this is the second version of my patch series that removes global state > > from "builtin/config.c". Changes compared to v1: > > > > - Reinstated a comment in patch 5. > > > > - Fixed a memory leak in patch 9. > > > > - A couple of commit message fixes. > > > > The series continues to build on top of ps/config-subcommands. > > I do not offhand know if this iteration has already been seen by me, > but a few recent CI runs of 'seen' did break with *-leaks jobs in > t13XX series around "config". Hopefully with the fix in "patch 9" > listed above the problem has gone away? We'll know soon enough when > I push out the integration result.
I think so, yes. v1 also broke pipelines at GitLab -- I didn't notice though because pipelines had already been broken due to the Python 2 deprecation in Ubuntu 24.04, so I missed that there were in fact multiple issues. The GitLab pipeline now passes with v2, so I assume that it would also pass in GitHub now.
Patrick