pack-redundant: escalate deprecation warning to an error
commit4406522b76138c2bd3d110ac9de0d860ba2be5ed
authorJeff King <peff@peff.net>
Thu, 23 Mar 2023 20:40:47 +0000 (23 16:40 -0400)
committerJunio C Hamano <gitster@pobox.com>
Thu, 23 Mar 2023 20:56:02 +0000 (23 13:56 -0700)
tree11d0c88bc56ef6731acca5e6d611c94e379702e5
parent73876f4861cd3d187a4682290ab75c9dccadbc56
pack-redundant: escalate deprecation warning to an error

In c3b58472be2 (pack-redundant: gauge the usage before proposing its
removal, 2020-08-25), we added a big, ugly warning when pack-redundant
is run. The plan there indicated that we would ratchet that up to an
error before finally removing it. Since it has been 2.5 years (and 9
releases) since then, let's continue with the plan.

Note that we did get one bite on the warning, which was somebody asking
about alternatives:

  https://lore.kernel.org/git/CAKvOHKAFXQwt4D8yUCCkf_TQL79mYaJ=KAKhtpDNTvHJFuX1NA@mail.gmail.com/

but we didn't undo the ugly warning (and the advice continues to be "use
repack -d" instead).

There was also some discussion around the time of the deprecation that
pack-redundant was invoked by the bitbake tool, and it still seems to do
so now:

  https://git.openembedded.org/bitbake

That use should probably just go away in favor of an occasional repack
(which probably even happens via auto-gc after fetch these days).

But since neither of those data points caused us to cancel the
deprecation plan by dropping the warning, it seems like we should
proceed with the next step.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/pack-redundant.c