Re: [Outreachy PATCH] environment: move "core.attributesFile" into repo-setting
- From
Karthik Nayak <karthik.188@gmail.com>
- Date
- Jan 2, 2026, 08:49 UTC
- Message-ID
- <CAOLa=ZQ8zQKk7Jmu_Pwm-VbCY9u3WP7oC51JBVUSXEi9pk_UfA@mail.gmail.com>
- In-Reply-To
- <CAD=f0L88QW_tL2iKg8ru3mU7t-vmY=p61S33GN+6tSQBMQAjqw@mail.gmail.com>
Bello Olamide <belkid98@gmail.com> writes:
Show 30 quoted lines
> On Thu, 18 Dec 2025 at 09:30, Olamide Caleb Bello <belkid98@gmail.com> wrote: >> >> When handling multiple repositories within the same process, relying on >> global state for accessing the "core.attributesFile" configuration can >> lead to incorrect values being used. It also makes it harder to isolate >> repositories and hinders the libification of git. >> The functions `bootstrap_attr_stack()` and `git_attr_val_system()` >> retrieve "core.attributesFile" via `git_attr_global_file()` >> which reads from global state `git_attributes_file`. >> >> Move the "core.attributesFile" configuration into the >> `struct repo_settings` instead of relying on the global state. >> A new function `repo_settings_get_attributesfile_path()` is added >> and used to retrieve this setting in a repository-scoped manner. >> The functions to retrieve "core.attributesFile" are replaced with >> the new accessor function `repo_settings_get_attributesfile_path()` >> This improves multi-repository behaviour and aligns with the goal of >> libifying of Git. >> >> Note that in `bootstrap_attr_stack()`, the `index_state` is used only >> if it exists, else we default to `the_repository`. >> >> Based-on-patch-by: Ayush Chandekar <ayu.chandekar@gmail.com> >> Mentored-by: Christian Couder <christian.couder@gmail.com> >> Mentored-by: Usman Akinyemi <usmanakinyemi202@gmail.com> >> Signed-off-by: Olamide Caleb Bello <belkid98@gmail.com> > > Hello. > Please I am replying to this as no reviews have been done on this patch. > Thanks
Thanks for the bump, I think reviews are slowed down due to holidays. You should see a uptick henceforth :)