On 23/07/2025 06:35, Patrick Steinhardt wrote:
On Tue, Jul 22, 2025 at 05:11:35PM -0700, Junio C Hamano wrote:
* ps/config-wo-the-repository (2025-07-17) 22 commits
- config: fix sign comparison warnings
- config: move Git config parsing into "environment.c"
- config: remove unused `the_repository` wrappers
- config: drop `git_config_set_multivar()` wrapper
- config: drop `git_config_get_multivar_gently()` wrapper
- config: drop `git_config_set_multivar_in_file_gently()` wrapper
- config: drop `git_config_set_in_file_gently()` wrapper
- config: drop `git_config_set()` wrapper
- config: drop `git_config_set_gently()` wrapper
- config: drop `git_config_set_in_file()` wrapper
- config: drop `git_config_get_bool()` wrapper
- config: drop `git_config_get_ulong()` wrapper
- config: drop `git_config_get_int()` wrapper
- config: drop `git_config_get_string()` wrapper
- config: drop `git_config_get_string()` wrapper
- config: drop `git_config_get_string_multi()` wrapper
- config: drop `git_config_get_value()` wrapper
- config: drop `git_config_get_value()` wrapper
- config: drop `git_config_get()` wrapper
- config: drop `git_config_clear()` wrapper
- config: drop `git_config()` wrapper
- Merge branch 'bc/use-sha256-by-default-in-3.0' into ps/config-wo-the-repository
The config API had a set of convenience wrapper functions that
implicitly use the_repository instance; they have been removed and
inlined at the calling sites.
Will merge to 'next'?
source: <20250717-pks-config-wo-the-repository-v1-0-d888e4a17de1@xxxxxx>
I think the only commit that really needs review is the last one that
fixes the sign comparison warnings. As long as people are fine with that
commit I'm happy to have it merged.
I think that commit needs a re-roll c.f.
<2b914b60-3417-4a49-b545-3109c9a05257@xxxxxxxxx>. I haven't looked at
the rest of the patches but I like the general idea behind this series.
Thanks
Phillip
Patrick
