Re: [PATCH v5 00/27] multi-pack reachability bitmaps

From: Junio C Hamano <gitster@pobox.com>
Date: Sep 1, 2021, 19:23 UTC
Message-ID: <xmqq1r68qevl.fsf@gitster.g>
In-Reply-To: <YS/Pqc7lkMlnlBYR@nand.local>

Taylor Blau <me@ttaylorr.com> writes:

Show 6 quoted lines

> I'm comfortable with saying what's written in the documentation, since
> even though it happens to work today, we should leave ourselves open to
> not supporting directories which aren't alternates.
>
> But I'm equally OK if you would rather drop this hunk from the
> documentation when staging.

Oh, no, don't get me wrong. I am comfortable with the documented limitation, as that is what the area experts have agreed that is reasonable given the expected use case.

I however am much less comfortable with a documented limitation that we make no attempt to enforce, and that is why the first thing I looked for after seeing the documentation update was new code to make sure we reject a random directory that is not our alternate object store.

Thanks.