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

Re: [PATCH v6 00/18] reftable: stop using "git-compat-util.h"

From
Patrick Steinhardt <ps@pks.im>
Date
Mar 20, 2025, 15:17 UTC
Message-ID
<Z9wxki6ES7ZRN7Z9@pks.im>
In-Reply-To
<Z9Eu_fZbI8ndkzHw@pks.im>
On Wed, Mar 12, 2025 at 07:51:41AM +0100, Patrick Steinhardt wrote:
Show 42 quoted lines
> On Tue, Mar 11, 2025 at 04:29:27PM -0700, Junio C Hamano wrote:
> > Junio C Hamano <gitster@pobox.com> writes:
> > 
> > > Patrick Steinhardt <ps@pks.im> writes:
> > >
> > >> Changes in v6:
> > >>   - I have rebased the patch series on top of "master" at 03944513488
> > >>     (The eleventh batch, 2025-02-14). On the one hand this gets rid of
> > >>     the conflict with the zlib-ng changes, which have meanwhile been
> > >>     merged to that branch. And on the other hand it pulls in the new CI
> > >>     build for MSVC+Meson, which allows me to have better test coverage
> > >>     on Windows to better detect breakage there.
> > >>   - Stop moving around Win32 files so that we do the minimum viable
> > >>     change in this context. Hopefully, this makes it easier for Dscho to
> > >>     integrate into Git for Windows. If it doesn't I don't really have
> > >>     any other good ideas for how to do it, so the only alternative would
> > >>     be to just ignore Windows altogether.
> > >>   - Link to v5: https://lore.kernel.org/r/20250207-pks-reftable-drop-git-compat-util-v5-0-ba2adc79110f@pks.im
> > >
> > > Thanks.  I think the "unlink" thing on Windows I have was queued
> > > depended on the previous iteration of this series so let me make
> > > sure that I'd unentangle it while replacing the topic with this
> > > round.
> > >
> > > Will queue.  Thanks.
> > 
> > So, sorry I lost track.  I think I saw you ask Dscho about the
> > Windows specific bits once in a thread on What's cooking report,
> > but I do not recall the outcome.  Is this topic (and the unlink fix
> > that depends on this series) now OK, or would we see further updates
> > once 2.49 is tagged?
> 
> There was a class of test failures caused by mismatching allocators in
> Git for Windows for which I've sent a proposed fix for downstream [1].
> I've been talking to Johannes on Monday and he mentioned that he'll
> likely end up taking that patch as the pragmatic fix. Which to the best
> of my knowledge would mean that this series here is unblocked and can
> start to move forward again. Johannes, please correct me in case I'm
> misstating anything here.
> 
> So based on that I don't plan to reroll and hope that we can move
> forward with the series once Git v2.49.0 is out.

Quick status update: Johannes mentioned in [1] that he is fine with moving forward with this topic now. Thanks!

Patrick
[1]: https://github.com/gitgitgadget/git/pull/1874#issuecomment-2740342672
Previous: Patrick Steinhardt
Message 23 of 23 in “reftable: stop using "git-compat-util.h"”
  1. 00/18 reftable: stop using "git-compat-util.h"Patrick Steinhardt, Feb 18, 2025
  2. 01/18 reftable/stack: stop using `read_in_full()`Patrick Steinhardt, Feb 18, 2025
  3. 02/18 reftable/stack: stop using `write_in_full()`Patrick Steinhardt, Feb 18, 2025
  4. 03/18 reftable/blocksource: stop using `xmmap()`Patrick Steinhardt, Feb 18, 2025
  5. 04/18 reftable/record: stop using `COPY_ARRAY()`Patrick Steinhardt, Feb 18, 2025
  6. 05/18 reftable/record: stop using `BUG()` in `reftable_record_init()`Patrick Steinhardt, Feb 18, 2025
  7. 06/18 reftable/record: don't `BUG()` in `reftable_record_cmp()`Patrick Steinhardt, Feb 18, 2025
  8. 08/18 reftable/basics: stop using `st_mult()` in array allocatorsPatrick Steinhardt, Feb 18, 2025
  9. 07/18 reftable: stop using `BUG()` in trivial casesPatrick Steinhardt, Feb 18, 2025
  10. 10/18 reftable/reader: stop using `ARRAY_SIZE()` macroPatrick Steinhardt, Feb 18, 2025
  11. 09/18 reftable/basics: provide wrappers for big endian conversionPatrick Steinhardt, Feb 18, 2025
  12. 11/18 reftable/system: introduce `reftable_rand()`Patrick Steinhardt, Feb 18, 2025
  13. 12/18 reftable/stack: stop using `sleep_millisec()`Patrick Steinhardt, Feb 18, 2025
  14. 13/18 reftable/basics: stop using `SWAP()` macroPatrick Steinhardt, Feb 18, 2025
  15. 14/18 reftable/basics: introduce `REFTABLE_UNUSED` annotationPatrick Steinhardt, Feb 18, 2025
  16. 15/18 compat/mingw: split out POSIX-related bitsPatrick Steinhardt, Feb 18, 2025
  17. 16/18 git-compat-util.h: split out POSIX-emulating bitsPatrick Steinhardt, Feb 18, 2025
  18. 17/18 reftable: decouple from Git codebase by pulling in "compat/posix.h"Patrick Steinhardt, Feb 18, 2025
  19. 18/18 Makefile: skip reftable library for CoccinellePatrick Steinhardt, Feb 18, 2025
  20. Junio C HamanoFeb 18, 2025
  21. Junio C HamanoMar 11, 2025
  22. Patrick SteinhardtMar 12, 2025
  23. Patrick SteinhardtMar 20, 2025

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.