Re: [PATCH] submodule: allow gitsubmodules to be pointed at tags

From: Junio C Hamano <gitster@pobox.com>
Date: Jun 24, 2024, 16:40 UTC
Message-ID: <xmqq4j9icn0t.fsf@gitster.g>
In-Reply-To: <m0sex2pi7q.fsf@epic96565.epic.com>

Sean Allred <allred.sean@gmail.com> writes:

> If you're ok with that behavior, then does it make sense to
> generalize it to the concept of a ref instead of just a tag name?
> This may do a better job of communicating the idea that no ref is
> inherently immovable.

Yup, I only took a brief glance at the patch, and I didn't quite see the point of duplicationg an almost parallel mechanism to an existing (--single-branch, set-branch) pair of the feature, only to replace "branch" with "tag" (the patch noise / code churn did not seem worth it).

In other words, is there a situation where a branch cannot be used (or using a branch is more cumbersome or awkward) when the user wants to use a tag with this patch? As users can just as easily move tags as they can move branches, the verb "freeze" used in the proposed log message probably does not reflect reality anyway, as you already pointed out.

Thanks.