Re: "What's cooking" interim report

From: Patrick Steinhardt <ps@pks.im>
Date: Apr 11, 2025, 09:32 UTC
Message-ID: <Z_jhsVJi5U0qinbp@pks.im>
In-Reply-To: <xmqqy0w9orsn.fsf@gitster.g>

On Wed, Apr 09, 2025 at 04:10:00PM -0700, Junio C Hamano wrote:

Show 20 quoted lines

> [New Topics]
>  * ps/object-file-cleanup (2025-04-08) 10 commits
>   - object-store: merge "object-store-ll.h" and "object-store.h"
>   - object-store: remove global array of cached objects
>   - object: split out functions relating to object store subsystem
>   - object-file: split out functions relating to index subsystem
>   - object-file: split up concerns of `HASH_*` flags
>   - object-file: split out functions relating to object store subsystem
>   - object-file: move `xmmap()` into "wrapper.c"
>   - object-file: move `git_open_cloexec()` to "compat/open.c"
>   - object-file: move `safe_create_leading_directories()` into "dir.c"
>   - Merge branch 'ps/object-wo-the-repository' into ps/object-file-cleanup
>   (this branch uses ps/object-wo-the-repository.)
> 
>   Code clean-up.
> 
>   Will merge to 'next'?
>   cf. <xmqqsemiteot.fsf@gitster.g>
>   cf. <CABPp-BFv_WGVUexyQBHkCcyO0N+UQCcJ3eM9Oh6bP_ZCUkaxLw@mail.gmail.com>
>   source: <20250408-pks-split-object-file-v1-0-f1fd50191143@pks.im>

I have sent out v2 just now to fix a single grammar issue in a commit message as pointed out by Elijah. So I think it's ready.

Show 9 quoted lines

>  * ps/refname-avail-check-optim (2025-04-09) 1 commit
>   - refs/packed: fix BUG when seeking refs with UTF-8 characters
> 
>   Incorrect sorting of refs with bytes with high-bit set on platforms
>   with signed char led to a BUG, which has been corrected.
> 
>   Will merge to 'next'.
>   cf. <20250404205740.GA780449@coredump.intra.peff.net>
>   source: <20250404-b4-pks-packed-backend-seek-with-utf8-v1-1-6ceb694e3bd7@pks.im>

Yup, makes sense.

Show 19 quoted lines

>  * rj/build-tweaks (2025-04-09) 13 commits
>   - config.mak.uname: set CSPRNG_METHOD to getrandom on Linux
>   - config.mak.uname: add arc4random to the cygwin build
>   - config.mak.uname: add sysinfo() configuration for cygwin
>   - builtin/gc.c: correct RAM calculation when using sysinfo
>   - config.mak.uname: add clock_gettime() to the cygwin build
>   - config.mak.uname: add HAVE_GETDELIM to the cygwin section
>   - config.mak.uname: only set NO_REGEX on cygwin for v1.7
>   - config.mak.uname: add a note about NO_STRLCPY for Linux
>   - Makefile: remove NEEDS_LIBRT build variable
>   - meson.build: set default help format to html on windows
>   - meson.build: only set build variables for non-default values
>   - Makefile: only set some BASIC_CFLAGS when RUNTIME_PREFIX is set
>   - meson.build: remove -DCURL_DISABLE_TYPECHECK
> 
>   Various build tweaks, including CSPRNG selection on some platforms.
> 
>   Comments?
>   source: <cover.1743859985.git.ramsay@ramsayjones.plus.com>

I'm currently traveling, so I didn't yet have time to review this. But I do plan to do so on Monday.

Patrick