Git development
 help / color / mirror / Atom feed
From: Junio C Hamano <gitster@pobox.com>
To: git@vger.kernel.org
Subject: [PATCH v3] you_still_use_that(): reword the instructions
Date: Thu, 27 Aug 2026 11:15:55 -0700	[thread overview]
Message-ID: <xmqqse3z8m5g.fsf_-_@gitster.g> (raw)
In-Reply-To: <xmqqjypchixe.fsf@gitster.g> (Junio C. Hamano's message of "Wed, 26 Aug 2026 10:48:29 -0700")

The message is overly long and may mislead readers into thinking
there is recourse other than adopting the new workflow.  Clarify
that the message is there merely to help them find a replacement
workflow, and is not offering to reconsider a decision that has
already taken effect.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
---

 * The text is unchanged from v2 except for fixing "mailing list";
   two tests that looked for "nominated for removal" string have
   been corrected.

 t/t4013-diff-various.sh   |  2 +-
 t/t5323-pack-redundant.sh |  2 +-
 usage.c                   | 16 +++++++---------
 3 files changed, 9 insertions(+), 11 deletions(-)

diff --git a/t/t4013-diff-various.sh b/t/t4013-diff-various.sh
index d35695f5b0..93d6e6e975 100755
--- a/t/t4013-diff-various.sh
+++ b/t/t4013-diff-various.sh
@@ -478,7 +478,7 @@ EOF
 
 test_expect_success !WITH_BREAKING_CHANGES 'whatchanged needs --i-still-use-this' '
 	test_must_fail git whatchanged >message 2>&1 &&
-	test_grep "nominated for removal" message
+	test_grep "will be removed soon" message
 '
 
 test_expect_success 'log -m matches pure log' '
diff --git a/t/t5323-pack-redundant.sh b/t/t5323-pack-redundant.sh
index 2d96afd6f7..aff0bce099 100755
--- a/t/t5323-pack-redundant.sh
+++ b/t/t5323-pack-redundant.sh
@@ -47,7 +47,7 @@ shared_repo=shared.git
 
 test_expect_success 'pack-redundant needs --i-still-use-this' '
 	test_must_fail git pack-redundant >message 2>&1 &&
-	test_grep "nominated for removal" message
+	test_grep "will be removed soon" message
 '
 
 git_pack_redundant='git pack-redundant --i-still-use-this'
diff --git a/usage.c b/usage.c
index 527edb1e79..90f392e89e 100644
--- a/usage.c
+++ b/usage.c
@@ -386,21 +386,19 @@ NORETURN void you_still_use_that(const char *command_name, const char *hint)
 				 STRBUF_ENCODE_SLASH);
 
 	fprintf(stderr,
-		_("'%s' is nominated for removal.\n"), command_name);
+		_("'%s' will be removed soon.\n"), command_name);
 
 	if (hint)
 		fputs(hint, stderr);
 
 	fprintf(stderr,
-		_("If you still use this command, here's what you can do:\n"
+		_("If you need a replacement:\n"
 		  "\n"
-		  "- read https://git-scm.com/docs/BreakingChanges.html\n"
-		  "- check if anyone has discussed this on the mailing\n"
-		  "  list and if they came up with something that can\n"
-		  "  help you: https://lore.kernel.org/git/?q=%s\n"
-		  "- send an email to <git@vger.kernel.org> to let us\n"
-		  "  know that you still use this command and were unable\n"
-		  "  to determine a suitable replacement\n"
+		  "- Read https://git-scm.com/docs/BreakingChanges.html.\n\n"
+		  "- Check what others on the mailing list suggest as a replacement:\n"
+		  "  https://lore.kernel.org/git/?q=%s\n\n"
+		  "- Send an email to <git@vger.kernel.org> asking for help, only if\n"
+		  "  suggestions by others do not work for you.\n"
 		  "\n"),
 		percent_encoded.buf);
 	strbuf_release(&percent_encoded);

Interdiff:
  diff --git a/t/t4013-diff-various.sh b/t/t4013-diff-various.sh
  index d35695f5b0..93d6e6e975 100755
  --- a/t/t4013-diff-various.sh
  +++ b/t/t4013-diff-various.sh
  @@ -478,7 +478,7 @@ EOF
   
   test_expect_success !WITH_BREAKING_CHANGES 'whatchanged needs --i-still-use-this' '
   	test_must_fail git whatchanged >message 2>&1 &&
  -	test_grep "nominated for removal" message
  +	test_grep "will be removed soon" message
   '
   
   test_expect_success 'log -m matches pure log' '
  diff --git a/t/t5323-pack-redundant.sh b/t/t5323-pack-redundant.sh
  index 2d96afd6f7..aff0bce099 100755
  --- a/t/t5323-pack-redundant.sh
  +++ b/t/t5323-pack-redundant.sh
  @@ -47,7 +47,7 @@ shared_repo=shared.git
   
   test_expect_success 'pack-redundant needs --i-still-use-this' '
   	test_must_fail git pack-redundant >message 2>&1 &&
  -	test_grep "nominated for removal" message
  +	test_grep "will be removed soon" message
   '
   
   git_pack_redundant='git pack-redundant --i-still-use-this'
  diff --git a/usage.c b/usage.c
  index d421ca426b..90f392e89e 100644
  --- a/usage.c
  +++ b/usage.c
  @@ -395,7 +395,7 @@ NORETURN void you_still_use_that(const char *command_name, const char *hint)
   		_("If you need a replacement:\n"
   		  "\n"
   		  "- Read https://git-scm.com/docs/BreakingChanges.html.\n\n"
  -		  "- Check what others on the mailing suggest as a replacement:\n"
  +		  "- Check what others on the mailing list suggest as a replacement:\n"
   		  "  https://lore.kernel.org/git/?q=%s\n\n"
   		  "- Send an email to <git@vger.kernel.org> asking for help, only if\n"
   		  "  suggestions by others do not work for you.\n"
-- 
2.55.0-862-g3c6f97f7b9


      parent reply	other threads:[~2026-08-27 18:15 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-08-26 14:33 [PATCH] you_still_use_that(): reword the instructions Junio C Hamano
2026-08-26 16:04 ` Michael Montalbo
2026-08-26 17:00   ` Junio C Hamano
2026-08-26 17:12     ` Michael Montalbo
2026-08-26 17:36       ` Michael Montalbo
2026-08-26 17:45       ` Junio C Hamano
2026-08-26 17:48         ` Michael Montalbo
2026-08-26 16:06 ` Michael Montalbo
2026-08-26 16:23   ` Junio C Hamano
2026-08-26 17:48 ` [PATCH v2] " Junio C Hamano
2026-08-27  1:22   ` Elijah Newren
2026-08-27  6:49   ` Kristoffer Haugsbakk
2026-08-27 14:38     ` Junio C Hamano
2026-08-27 16:01     ` Junio C Hamano
2026-08-27 18:15   ` Junio C Hamano [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=xmqqse3z8m5g.fsf_-_@gitster.g \
    --to=gitster@pobox.com \
    --cc=git@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox