Re: [PATCH v2 5/5] remote: announce removal of "branches/" and "remotes/"

From: Patrick Steinhardt <ps@pks.im>
Date: Jan 8, 2025, 06:36 UTC
Message-ID: <Z34c3rj0E6hP_kHN@pks.im>
In-Reply-To: <006701db6124$f16f9420$d44ebc60$@nexbridge.com>

On Tue, Jan 07, 2025 at 11:55:16AM -0500, rsbecker@nexbridge.com wrote:

Show 13 quoted lines

> On January 7, 2025 11:50 AM, Junio C Hamano wrote:
> >Junio C Hamano <gitster@pobox.com> writes:
> >
> >> Patrick Steinhardt <ps@pks.im> writes:
> >>
> >>> Makes sense indeed. We can easily add for something like below diff.
> >>> I'll roll that into the next version, thanks!
> >>
> >> It is a good start, but is probably a bit too noisy.  Can we make them
> >> appear ONLY when the definitions read from these older sources are
> >> actually USED?
> >>
> >> Thanks.

Fair enough, can do.

> I like this but wonder whether there might be some way to inhibit the
> warnings one a user gets it and decides they will act but do not want
> to see the warnings any longer? I have had requests like this on other
> products. Just a thought.

I guess the best idea I have here is to use an environment variable, e.g. "GIT_ALLOW_DEPRECATED_REMOTES=true", along with a hint for how to enable it.

Patrick