Re: [PATCH v2 1/4] promisor-remote: read partialClone config here
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Jun 9, 2021, 09:30 UTC
- Message-ID
- <xmqqeedbidvy.fsf@gitster.g>
- In-Reply-To
- <20210609042649.2322758-1-jonathantanmy@google.com>
Jonathan Tan <jonathantanmy@google.com> writes:
> I'm reluctant to add more fields to struct repository_format.
I am only suggesting to add one new member to either struct repository or struct repo_settings, so that it becomes crystal clear that struct repository_format is about each single repository, not the global the_repository. Other things that partial repository support needs to keep *and* do not directly come from extensions would not belong to repository_format and should not be added there, but what we read from extensions.* for each repository belongs to each instance of in-core repository structure and should be discoverable starting from "struct repository", no?