Re: [Outreachy PATCH v6 1/3] environment: stop storing `core.attributesFile` globally
- From
Bello Olamide <belkid98@gmail.com>
- Date
- Feb 11, 2026, 12:05 UTC
- Message-ID
- <CAD=f0L8Mz2jLMnkyvAUbtDC65RwKSk6Q-ZgB3+pLFpJo_DN9qw@mail.gmail.com>
- In-Reply-To
- <37abfe28-ff89-460a-962d-4503ec3f7975@gmail.com>
On Wed, 11 Feb 2026 at 10:31, Phillip Wood <phillip.wood123@gmail.com> wrote:
Show 23 quoted lines
>
> Thanks for the backtrace which helped me spot the problem though I
> should have spotted this yesterday. The problem is in
> initialize_repository()
>
> > void initialize_repository(struct repository *repo)
> > {
> > + if (repo->initialized)
> > + BUG("repository initialized already");
> > + repo->initialized = true;
> > +
> > repo->remote_state = remote_state_new();
> > repo->parsed_objects = parsed_object_pool_new(repo);
> > ALLOC_ARRAY(repo->index, 1);
> > index_state_init(repo->index, repo);
> > repo->check_deprecated_config = true;
> > + repo_config_values_init(repo_config_values(repo));
>
> Here you need to use repo->config_values_private_ instead of using the
> accessor as it is fine to initialize the config values to their defaults
> in any instance, it is only when we read them that we want to assert
> that we're reading from "the_repository".
>Okay thank you very much. I will send an updated version.