[v1,0/4] drop "name-rev --stdin" support

Message

Junio C Hamano March 10, 2025, 11:16 p.m. UTC

[Administrivia]
    Yes, yes, I know I shouldn't be playing with shiny new toys
    during the pre-release freeze time.  But just to show others who
    may still be doing so that the first patch in this series has
    already be written to avoid duplicated and conflicting work, I
    am sending them out.  I have no intention to push the topic
    further before the final.

During Git 2.40 timeframe, we deprecated "--stdin" option of the
"name-rev" command in preference to "--annotate-stdin", and removed
the mention of it from the documentation.

Let's prepare for Git 3.0 to stop supporting it.

The real motive of these patches is not really about that option but
make sure we have, with WITH_BREAKING_CHANGES compilation knob,
enough support to keep preparing these changes.  The first patch
renames the WITHOUT_BREAKING_CHANGES prerequisite that unfortunately
invites double negations easily and changes existing users of it,
then two patches to a test script minimally modernizes it.  The last
step introduces the real change, guarded by WITH_BREAKING_CHANGES
compilation knob.

Junio C Hamano (4):
  t: introduce WITH_BREAKING_CHANGES prerequisite
  t6120: avoid hiding "git" exit status
  t6120: further modernize
  name-rev: remove "--stdin" support

 Documentation/BreakingChanges.adoc |  6 ++++++
 builtin/name-rev.c                 |  2 ++
 t/t5323-pack-redundant.sh          |  2 +-
 t/t5505-remote.sh                  |  6 +++---
 t/t5515-fetch-merge-logic.sh       |  2 +-
 t/t5516-fetch-push.sh              |  8 ++++----
 t/t6120-describe.sh                | 18 +++++++++++++-----
 t/test-lib.sh                      |  5 +++++
 8 files changed, 35 insertions(+), 14 deletions(-)