Re: [RFC GSoC PATCH v3 0/2] Make read_attr() repository-aware by introducing a lazy bare state

From: Ayush Jha <kumarayushjha123@gmail.com>
Date: Feb 19, 2026, 09:42 UTC
Message-ID: <CAFNBzOeUcSYBfQwDSV7ZVrAdDNqNohmVG=JuYzxQC0DgM2snUQ@mail.gmail.com>
In-Reply-To: <20260208062949.596-1-kumarayushjha123@gmail.com>

Hi all,

I wanted to gently follow up on this RFC series in case it may have been missed in the thread. I would really appreciate any feedback whenever you have time.

If there are concerns, suggestions, or areas that need clarification, I’d be happy to revise the series accordingly.

Thank you for your time and review.

Best regards, Ayush

On Sun, Feb 8, 2026 at 12:00 PM Ayush Jha <kumarayushjha123@gmail.com> wrote:

Show 28 quoted lines

>
> This patch series is v3 of an earlier RFC that aimed to remove the
> dependency on the global `the_repository` from read_attr().
>
> Based on feedback from Lucas Seiki Oshiro and Junio C Hamano, this
> iteration introduces a repository-scoped, lazily initialized "bare"
> state in struct repo_settings, exposed via repo_settings_get_is_bare().
> The read_attr() function is then updated to use this helper instead of
> is_bare_repository().
>
> The series is split as follows:
>   1/2 repo-settings: add repo_settings_get_is_bare
>   2/2 attr: use local repository state in read_attr
>
> Comments and suggestions are welcome.
>
> Ayush Jha (2):
>   repo-settings: add repo_settings_get_is_bare
>   attr: use local repository state in read_attr
>
>  attr.c          |  3 ++-
>  repo-settings.c | 12 ++++++++++++
>  repo-settings.h |  5 +++++
>  3 files changed, 19 insertions(+), 1 deletion(-)
>
> --
> 2.53.0.windows.1
>