{"thread":{"id":"66224","subject":"[PATCH] you_still_use_that(): reword the instructions","startedAt":"2026-08-26T14:33:33Z","lastAt":"2026-08-27T18:15:58Z","messageCount":15,"participants":["Junio C Hamano","Michael Montalbo","Elijah Newren","Kristoffer Haugsbakk"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"551285","messageId":"xmqqo6epj6is.fsf@gitster.g","threadId":"66224","inReplyTo":null,"subject":"[PATCH] you_still_use_that(): reword the instructions","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2026-08-26T14:33:31Z","receivedAt":"2026-08-26T14:33:33Z","isPatch":true,"body":"The message is overly long and may mislead readers into thinking\nthere is recourse other than adopting the new workflow. Clarify\nthat it merely helps them find a replacement, rather than offering\nto reconsider a decision that has already taken effect.\n\nSigned-off-by: Junio C Hamano \n---\n usage.c | 14 ++++++--------\n 1 file changed, 6 insertions(+), 8 deletions(-)\n\ndiff --git c/usage.c w/usage.c\nindex 3f0118ab2a..9933b9911c 100644\n--- c/usage.c\n+++ w/usage.c\n@@ -392,15 +392,13 @@ NORETURN void you_still_use_that(const char *command_name, const char *hint)\n \t\tfputs(hint, stderr);\n \n \tfprintf(stderr,\n-\t\t_(\"If you still use this command, here's what you can do:\\n\"\n+\t\t_(\"If you need a replacement:\\n\"\n \t\t \"\\n\"\n-\t\t \"- read https://git-scm.com/docs/BreakingChanges.html\\n\"\n-\t\t \"- check if anyone has discussed this on the mailing\\n\"\n-\t\t \" list and if they came up with something that can\\n\"\n-\t\t \" help you: https://lore.kernel.org/git/?q=%s\\n\"\n-\t\t \"- send an email to to let us\\n\"\n-\t\t \" know that you still use this command and were unable\\n\"\n-\t\t \" to determine a suitable replacement\\n\"\n+\t\t \"- Read https://git-scm.com/docs/BreakingChanges.html.\\n\\n\"\n+\t\t \"- Check what others on the mailing suggest as a replacement:\\n\"\n+\t\t \" https://lore.kernel.org/git/?q=%s\\n\\n\"\n+\t\t \"- Send an email to asking for help, only if\\n\"\n+\t\t \" suggestions by others do not work for you.\\n\"\n \t\t \"\\n\"),\n \t\tpercent_encoded.buf);\n \tstrbuf_release(&percent_encoded);\n"},{"id":"551290","messageId":"CAC2QwmJ_fjNw9z+8an9Doq6Mx_29R5mcGXT1=NnVvu-g71QByA@mail.gmail.com","threadId":"66224","inReplyTo":"xmqqo6epj6is.fsf@gitster.g","subject":"Re: [PATCH] you_still_use_that(): reword the instructions","fromName":"Michael Montalbo","fromEmail":"mmontalbo@gmail.com","sentAt":"2026-08-26T16:04:31Z","receivedAt":"2026-08-26T16:04:44Z","isPatch":true,"body":"On Wed, Aug 26, 2026 at 7:33 AM Junio C Hamano wrote:\n>\n> The message is overly long and may mislead readers into thinking\n> there is recourse other than adopting the new workflow. Clarify\n> that it merely helps them find a replacement, rather than offering\n> to reconsider a decision that has already taken effect.\n>\n\nI think this change makes sense.\n\n> + \"- Send an email to asking for help, only if\\n\"\n> + \" suggestions by others do not work for you.\\n\"\n\nMaybe a slight reword to reinforce the idea that the command is not coming\nback:\n\n \"Send an email... asking for help, only if suggested replacements\n for the [now?] deprecated command do not work for you.\"\n\nOther than that suggestion, I think the change looks good.\n"},{"id":"551291","messageId":"CAC2Qwm+Gg3nhxtwhQjrvrk6Pm_R=XEpAwYxVKqT0Ps9UZ-DAyg@mail.gmail.com","threadId":"66224","inReplyTo":"xmqqo6epj6is.fsf@gitster.g","subject":"Re: [PATCH] you_still_use_that(): reword the instructions","fromName":"Michael Montalbo","fromEmail":"mmontalbo@gmail.com","sentAt":"2026-08-26T16:06:56Z","receivedAt":"2026-08-26T16:07:10Z","isPatch":true,"body":"On Wed, Aug 26, 2026 at 7:33 AM Junio C Hamano wrote:\n>\n> diff --git c/usage.c w/usage.c\n> index 3f0118ab2a..9933b9911c 100644\n> --- c/usage.c\n> +++ w/usage.c\n> @@ -392,15 +392,13 @@ NORETURN void you_still_use_that(const char *command_name, const char *hint)\n> fputs(hint, stderr);\n>\n> fprintf(stderr,\n> - _(\"If you still use this command, here's what you can do:\\n\"\n> + _(\"If you need a replacement:\\n\"\n> \"\\n\"\n> - \"- read https://git-scm.com/docs/BreakingChanges.html\\n\"\n> - \"- check if anyone has discussed this on the mailing\\n\"\n> - \" list and if they came up with something that can\\n\"\n> - \" help you: https://lore.kernel.org/git/?q=%s\\n\"\n> - \"- send an email to to let us\\n\"\n> - \" know that you still use this command and were unable\\n\"\n> - \" to determine a suitable replacement\\n\"\n> + \"- Read https://git-scm.com/docs/BreakingChanges.html.\\n\\n\"\n\nI noticed there are now two newlines at the end instead of one like before. Same\nfor the lore link below. Is that intentional?\n\n> + \"- Check what others on the mailing suggest as a replacement:\\n\"\n> + \" https://lore.kernel.org/git/?q=%s\\n\\n\"\n> + \"- Send an email to asking for help, only if\\n\"\n> + \" suggestions by others do not work for you.\\n\"\n> \"\\n\"),\n> percent_encoded.buf);\n> strbuf_release(&percent_encoded);\n>\n"},{"id":"551293","messageId":"xmqqv78wj1ez.fsf@gitster.g","threadId":"66224","inReplyTo":"CAC2Qwm+Gg3nhxtwhQjrvrk6Pm_R=XEpAwYxVKqT0Ps9UZ-DAyg@mail.gmail.com","subject":"Re: [PATCH] you_still_use_that(): reword the instructions","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2026-08-26T16:23:48Z","receivedAt":"2026-08-26T16:23:50Z","isPatch":true,"body":"Michael Montalbo writes:\n\n> On Wed, Aug 26, 2026 at 7:33 AM Junio C Hamano wrote:\n>>\n>> diff --git c/usage.c w/usage.c\n>> index 3f0118ab2a..9933b9911c 100644\n>> --- c/usage.c\n>> +++ w/usage.c\n>> @@ -392,15 +392,13 @@ NORETURN void you_still_use_that(const char *command_name, const char *hint)\n>> fputs(hint, stderr);\n>>\n>> fprintf(stderr,\n>> - _(\"If you still use this command, here's what you can do:\\n\"\n>> + _(\"If you need a replacement:\\n\"\n>> \"\\n\"\n>> - \"- read https://git-scm.com/docs/BreakingChanges.html\\n\"\n>> - \"- check if anyone has discussed this on the mailing\\n\"\n>> - \" list and if they came up with something that can\\n\"\n>> - \" help you: https://lore.kernel.org/git/?q=%s\\n\"\n>> - \"- send an email to to let us\\n\"\n>> - \" know that you still use this command and were unable\\n\"\n>> - \" to determine a suitable replacement\\n\"\n>> + \"- Read https://git-scm.com/docs/BreakingChanges.html.\\n\\n\"\n>\n> I noticed there are now two newlines at the end instead of one like before. Same\n> for the lore link below. Is that intentional?\n\nAbsolutely.\n\n>\n>> + \"- Check what others on the mailing suggest as a replacement:\\n\"\n>> + \" https://lore.kernel.org/git/?q=%s\\n\\n\"\n>> + \"- Send an email to asking for help, only if\\n\"\n>> + \" suggestions by others do not work for you.\\n\"\n>> \"\\n\"),\n>> percent_encoded.buf);\n>> strbuf_release(&percent_encoded);\n>>\n"},{"id":"551297","messageId":"xmqqik4wizqf.fsf@gitster.g","threadId":"66224","inReplyTo":"CAC2QwmJ_fjNw9z+8an9Doq6Mx_29R5mcGXT1=NnVvu-g71QByA@mail.gmail.com","subject":"Re: [PATCH] you_still_use_that(): reword the instructions","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2026-08-26T17:00:08Z","receivedAt":"2026-08-26T17:00:15Z","isPatch":true,"body":"Michael Montalbo writes:\n\n> I think this change makes sense.\n>\n>> + \"- Send an email to asking for help, only if\\n\"\n>> + \" suggestions by others do not work for you.\\n\"\n>\n> Maybe a slight reword to reinforce the idea that the command is not coming\n> back:\n>\n> \"Send an email... asking for help, only if suggested replacements\n> for the [now?] deprecated command do not work for you.\"\n>\n> Other than that suggestion, I think the change looks good.\n\nNot repeating 'replacement' there was deliberate for two reasons.\nFirst, the preceding instruction mentions others suggesting\nreplacements, presenting an obvious opportunity to keep the wording\nconcise. Second, this message is meant to be generic. For a case\nlike 'whatschanged', the \"new workflow\" is to use another command\n(i.e., 'replacement'), but other cases may involve doing things\ndifferently and are not limited to a simple replacement with another\ncommand. \n\nAnd that is why the message only says \"suggestions by others\".\n"},{"id":"551298","messageId":"CAC2Qwm+Dx1UxLO3EOkQ-9uBokxacVfgrs7+XCj3BeM9vt8HcPw@mail.gmail.com","threadId":"66224","inReplyTo":"xmqqik4wizqf.fsf@gitster.g","subject":"Re: [PATCH] you_still_use_that(): reword the instructions","fromName":"Michael Montalbo","fromEmail":"mmontalbo@gmail.com","sentAt":"2026-08-26T17:12:13Z","receivedAt":"2026-08-26T17:12:34Z","isPatch":true,"body":"On Wed, Aug 26, 2026 at 10:00 AM Junio C Hamano wrote:\n>\n> Michael Montalbo writes:\n>\n> > I think this change makes sense.\n> >\n> >> + \"- Send an email to asking for help, only if\\n\"\n> >> + \" suggestions by others do not work for you.\\n\"\n> >\n> > Maybe a slight reword to reinforce the idea that the command is not coming\n> > back:\n> >\n> > \"Send an email... asking for help, only if suggested replacements\n> > for the [now?] deprecated command do not work for you.\"\n> >\n> > Other than that suggestion, I think the change looks good.\n>\n> Not repeating 'replacement' there was deliberate for two reasons.\n> First, the preceding instruction mentions others suggesting\n> replacements, presenting an obvious opportunity to keep the wording\n> concise. Second, this message is meant to be generic. For a case\n> like 'whatschanged', the \"new workflow\" is to use another command\n> (i.e., 'replacement'), but other cases may involve doing things\n> differently and are not limited to a simple replacement with another\n> command.\n>\n> And that is why the message only says \"suggestions by others\".\n\nGot it, the reasoning for removing 'replacement' makes sense. Thank you.\n\nAdding something like \"[now] deprecated\" or otherwise more explicitly\nsuggesting the command does not intend to return is my main feedback.\n"},{"id":"551302","messageId":"CAC2QwmL58JbGLVPX+bqrANvNtn1swwzLU+P+Pu=+aJ5euUwRXg@mail.gmail.com","threadId":"66224","inReplyTo":"CAC2Qwm+Dx1UxLO3EOkQ-9uBokxacVfgrs7+XCj3BeM9vt8HcPw@mail.gmail.com","subject":"Re: [PATCH] you_still_use_that(): reword the instructions","fromName":"Michael Montalbo","fromEmail":"mmontalbo@gmail.com","sentAt":"2026-08-26T17:36:30Z","receivedAt":"2026-08-26T17:36:45Z","isPatch":true,"body":"On Wed, Aug 26, 2026 at 10:12 AM Michael Montalbo wrote:\n>\n> Adding something like \"[now] deprecated\" or otherwise more explicitly\n> suggesting the command does not intend to return is my main feedback.\n\nMaybe:\n \"Send an email..., only if suggested alternatives to the deprecated workflow\n do not work for you.\"\n\nOnly a minor suggestion in the end. The general cleanup to the text seems good.\n"},{"id":"551303","messageId":"xmqqwltchj2p.fsf@gitster.g","threadId":"66224","inReplyTo":"CAC2Qwm+Dx1UxLO3EOkQ-9uBokxacVfgrs7+XCj3BeM9vt8HcPw@mail.gmail.com","subject":"Re: [PATCH] you_still_use_that(): reword the instructions","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2026-08-26T17:45:18Z","receivedAt":"2026-08-26T17:45:24Z","isPatch":true,"body":"Michael Montalbo writes:\n\n> On Wed, Aug 26, 2026 at 10:00 AM Junio C Hamano wrote:\n>>\n>> Michael Montalbo writes:\n>>\n>> > I think this change makes sense.\n>> >\n>> >> + \"- Send an email to asking for help, only if\\n\"\n>> >> + \" suggestions by others do not work for you.\\n\"\n>> >\n>> > Maybe a slight reword to reinforce the idea that the command is not coming\n>> > back:\n>> >\n>> > \"Send an email... asking for help, only if suggested replacements\n>> > for the [now?] deprecated command do not work for you.\"\n>> ...\n> Adding something like \"[now] deprecated\" or otherwise more explicitly\n> suggesting the command does not intend to return is my main feedback.\n\nThe same function emits a message that says \"this command is\nnominated for removal.\" Perhaps we want to strengthen that part by\nsaying \"will be removed\" instead?\n"},{"id":"551304","messageId":"xmqqjypchixe.fsf@gitster.g","threadId":"66224","inReplyTo":"xmqqo6epj6is.fsf@gitster.g","subject":"[PATCH v2] you_still_use_that(): reword the instructions","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2026-08-26T17:48:29Z","receivedAt":"2026-08-26T17:48:35Z","isPatch":true,"body":"The message is overly long and may mislead readers into thinking\nthere is recourse other than adopting the new workflow. Clarify\nthat the message is there merely to help them find a replacement\nworkflow, and is not offering to reconsider a decision that has\nalready taken effect.\n\nSigned-off-by: Junio C Hamano \n---\n usage.c | 16 +++++++---------\n 1 file changed, 7 insertions(+), 9 deletions(-)\n\ndiff --git c/usage.c w/usage.c\nindex 3f0118ab2a..365c0bcb90 100644\n--- c/usage.c\n+++ w/usage.c\n@@ -386,21 +386,19 @@ NORETURN void you_still_use_that(const char *command_name, const char *hint)\n \t\t\t\t STRBUF_ENCODE_SLASH);\n \n \tfprintf(stderr,\n-\t\t_(\"'%s' is nominated for removal.\\n\"), command_name);\n+\t\t_(\"'%s' will be removed soon.\\n\"), command_name);\n \n \tif (hint)\n \t\tfputs(hint, stderr);\n \n \tfprintf(stderr,\n-\t\t_(\"If you still use this command, here's what you can do:\\n\"\n+\t\t_(\"If you need a replacement:\\n\"\n \t\t \"\\n\"\n-\t\t \"- read https://git-scm.com/docs/BreakingChanges.html\\n\"\n-\t\t \"- check if anyone has discussed this on the mailing\\n\"\n-\t\t \" list and if they came up with something that can\\n\"\n-\t\t \" help you: https://lore.kernel.org/git/?q=%s\\n\"\n-\t\t \"- send an email to to let us\\n\"\n-\t\t \" know that you still use this command and were unable\\n\"\n-\t\t \" to determine a suitable replacement\\n\"\n+\t\t \"- Read https://git-scm.com/docs/BreakingChanges.html.\\n\\n\"\n+\t\t \"- Check what others on the mailing suggest as a replacement:\\n\"\n+\t\t \" https://lore.kernel.org/git/?q=%s\\n\\n\"\n+\t\t \"- Send an email to asking for help, only if\\n\"\n+\t\t \" suggestions by others do not work for you.\\n\"\n \t\t \"\\n\"),\n \t\tpercent_encoded.buf);\n \tstrbuf_release(&percent_encoded);\n"},{"id":"551305","messageId":"CAC2QwmL6GvnLoW7xOc19vQ_Nj+0B=fiqAmz2sRnaev-boHyc3Q@mail.gmail.com","threadId":"66224","inReplyTo":"xmqqwltchj2p.fsf@gitster.g","subject":"Re: [PATCH] you_still_use_that(): reword the instructions","fromName":"Michael Montalbo","fromEmail":"mmontalbo@gmail.com","sentAt":"2026-08-26T17:48:39Z","receivedAt":"2026-08-26T17:48:58Z","isPatch":true,"body":"On Wed, Aug 26, 2026 at 10:45 AM Junio C Hamano wrote:\n>\n> Michael Montalbo writes:\n>\n> > On Wed, Aug 26, 2026 at 10:00 AM Junio C Hamano wrote:\n> >>\n> >> Michael Montalbo writes:\n> >>\n> >> > I think this change makes sense.\n> >> >\n> >> >> + \"- Send an email to asking for help, only if\\n\"\n> >> >> + \" suggestions by others do not work for you.\\n\"\n> >> >\n> >> > Maybe a slight reword to reinforce the idea that the command is not coming\n> >> > back:\n> >> >\n> >> > \"Send an email... asking for help, only if suggested replacements\n> >> > for the [now?] deprecated command do not work for you.\"\n> >> ...\n> > Adding something like \"[now] deprecated\" or otherwise more explicitly\n> > suggesting the command does not intend to return is my main feedback.\n>\n> The same function emits a message that says \"this command is\n> nominated for removal.\" Perhaps we want to strengthen that part by\n> saying \"will be removed\" instead?\n\nYes, saying it \"will be removed\" sounds great to me.\n"},{"id":"551335","messageId":"CABPp-BEnTWbk7m7jRoBG7Bp6vT62GRHoEfXg7=bJh111BqSx_A@mail.gmail.com","threadId":"66224","inReplyTo":"xmqqjypchixe.fsf@gitster.g","subject":"Re: [PATCH v2] you_still_use_that(): reword the instructions","fromName":"Elijah Newren","fromEmail":"newren@gmail.com","sentAt":"2026-08-27T01:22:54Z","receivedAt":"2026-08-27T01:23:07Z","isPatch":true,"body":"On Wed, Aug 26, 2026 at 10:50 AM Junio C Hamano wrote:\n>\n> The message is overly long and may mislead readers into thinking\n> there is recourse other than adopting the new workflow. Clarify\n> that the message is there merely to help them find a replacement\n> workflow, and is not offering to reconsider a decision that has\n> already taken effect.\n>\n> Signed-off-by: Junio C Hamano \n> ---\n> usage.c | 16 +++++++---------\n> 1 file changed, 7 insertions(+), 9 deletions(-)\n>\n> diff --git c/usage.c w/usage.c\n> index 3f0118ab2a..365c0bcb90 100644\n> --- c/usage.c\n> +++ w/usage.c\n> @@ -386,21 +386,19 @@ NORETURN void you_still_use_that(const char *command_name, const char *hint)\n> STRBUF_ENCODE_SLASH);\n>\n> fprintf(stderr,\n> - _(\"'%s' is nominated for removal.\\n\"), command_name);\n> + _(\"'%s' will be removed soon.\\n\"), command_name);\n>\n> if (hint)\n> fputs(hint, stderr);\n>\n> fprintf(stderr,\n> - _(\"If you still use this command, here's what you can do:\\n\"\n> + _(\"If you need a replacement:\\n\"\n> \"\\n\"\n> - \"- read https://git-scm.com/docs/BreakingChanges.html\\n\"\n> - \"- check if anyone has discussed this on the mailing\\n\"\n> - \" list and if they came up with something that can\\n\"\n> - \" help you: https://lore.kernel.org/git/?q=%s\\n\"\n> - \"- send an email to to let us\\n\"\n> - \" know that you still use this command and were unable\\n\"\n> - \" to determine a suitable replacement\\n\"\n> + \"- Read https://git-scm.com/docs/BreakingChanges.html.\\n\\n\"\n> + \"- Check what others on the mailing suggest as a replacement:\\n\"\n\nmissing \"list\"? (\"mailing [list] suggest\")\n\n> + \" https://lore.kernel.org/git/?q=%s\\n\\n\"\n> + \"- Send an email to asking for help, only if\\n\"\n> + \" suggestions by others do not work for you.\\n\"\n> \"\\n\"),\n> percent_encoded.buf);\n> strbuf_release(&percent_encoded);\n\nOtherwise, looks good.\n"},{"id":"551345","messageId":"cd741511-b457-4464-aab0-56d8554bf763@app.fastmail.com","threadId":"66224","inReplyTo":"xmqqjypchixe.fsf@gitster.g","subject":"Re: [PATCH v2] you_still_use_that(): reword the instructions","fromName":"Kristoffer Haugsbakk","fromEmail":"kristofferhaugsbakk@fastmail.com","sentAt":"2026-08-27T06:49:27Z","receivedAt":"2026-08-27T06:49:52Z","isPatch":true,"body":"On Wed, Aug 26, 2026, at 19:48, Junio C Hamano wrote:\n> The message is overly long and may mislead readers into thinking\n> there is recourse other than adopting the new workflow. Clarify\n> that the message is there merely to help them find a replacement\n> workflow, and is not offering to reconsider a decision that has\n> already taken effect.\n>\n> Signed-off-by: Junio C Hamano \n> ---\n\nFrom the responses on the thread there was one action point that was\nagreed on and that is changed in this version. See the range diff at\nthe end of this message.[1]\n\nSpotting the changes is harder than it needs to be when there are no\ninter-version comments, range diff, or interdiff (any of them).\n\n> usage.c | 16 +++++++---------\n> 1 file changed, 7 insertions(+), 9 deletions(-)\n>\n> diff --git c/usage.c w/usage.c\n> index 3f0118ab2a..365c0bcb90 100644\n> --- c/usage.c\n> +++ w/usage.c\n> @@ -386,21 +386,19 @@ NORETURN void you_still_use_that(const char\n> *command_name, const char *hint)\n> \t\t\t\t STRBUF_ENCODE_SLASH);\n>\n> \tfprintf(stderr,\n> -\t\t_(\"'%s' is nominated for removal.\\n\"), command_name);\n> +\t\t_(\"'%s' will be removed soon.\\n\"), command_name);\n\nI was thinking that maybe the final bullet point could say something\nlike\n\n ... and don’t send a message voting for it to come back because the\n decision has already been made\n\nBut it *could* be mentioned here instead. “The decision is final ...”\n\nI bring this up because some replies have come in where they demonstrate\nthat they know what to do about it. But instead of doing it they vote\nfor the decision to be undone.[2]\n\n Kernel audio driver maintainer, here. The first thing I want to know\n about a branch is \"whatchanged\" - it tells me exactly what I want to\n know with no distractions. So I was horrified when I saw the removal\n nomination message. Please reconsider - while git log can do it, I\n don't use git log... at all. I'd just have to add another alias that\n would be another thing to constantly collide with while using docker\n build chains with different (but usually older) versions of git.\n\n(Although this was before you could alias deprecated commands but still.)\n\nHonestly, it seems that some people would rather vote/lobby for the\ncontinued maintenance of git-whatchanged(1) [3] rather than execute\none config change. That is just my general impression.\n\nAnd I also bring this up because the commit message says:\n\n and [the message] is not offering to reconsider a decision that has\n already taken effect.\n\n🔗 2: https://lore.kernel.org/git/BL3P221MB0449288C8B0FA448A227FD48833AA@BL3P221MB0449.NAMP221.PROD.OUTLOOK.COM/\n\n† 3: This message is for the deprecated command git-pack-redundant(1)\n too but there hasn’t been any messages about wanting to get it back\n in the last year or so.\n\n>\n> \tif (hint)\n> \t\tfputs(hint, stderr);\n>\n> \tfprintf(stderr,\n> -\t\t_(\"If you still use this command, here's what you can do:\\n\"\n> +\t\t_(\"If you need a replacement:\\n\"\n> \t\t \"\\n\"\n> -\t\t \"- read https://git-scm.com/docs/BreakingChanges.html\\n\"\n> -\t\t \"- check if anyone has discussed this on the mailing\\n\"\n> -\t\t \" list and if they came up with something that can\\n\"\n> -\t\t \" help you: https://lore.kernel.org/git/?q=%s\\n\"\n> -\t\t \"- send an email to to let us\\n\"\n> -\t\t \" know that you still use this command and were unable\\n\"\n> -\t\t \" to determine a suitable replacement\\n\"\n> +\t\t \"- Read https://git-scm.com/docs/BreakingChanges.html.\\n\\n\"\n> +\t\t \"- Check what others on the mailing suggest as a replacement:\\n\"\n> +\t\t \" https://lore.kernel.org/git/?q=%s\\n\\n\"\n> +\t\t \"- Send an email to asking for help, only if\\n\"\n> +\t\t \" suggestions by others do not work for you.\\n\"\n> \t\t \"\\n\"),\n> \t\tpercent_encoded.buf);\n> \tstrbuf_release(&percent_encoded);\n\nThe rest looks good.\n\nWhat I like the most about this patch/change is that it is more\nassertive and less wordy, e.g. just “If you need a replacement”\nnot a long ceremonial sentence “If you still use this command, ...”.\n\n[1]:\n\n1: fd53eb2ca3a ! 1: dca3205b1f6 you_still_use_that(): reword the instructions\n @@ Commit message\n\n The message is overly long and may mislead readers into thinking\n there is recourse other than adopting the new workflow. Clarify\n - that it merely helps them find a replacement, rather than offering\n - to reconsider a decision that has already taken effect.\n + that the message is there merely to help them find a replacement\n + workflow, and is not offering to reconsider a decision that has\n + already taken effect.\n\n Signed-off-by: Junio C Hamano \n\n ## usage.c ##\n @@ usage.c: NORETURN void you_still_use_that(const char *command_name, const char *hint)\n + \t\t\t\t STRBUF_ENCODE_SLASH);\n +\n + \tfprintf(stderr,\n +-\t\t_(\"'%s' is nominated for removal.\\n\"), command_name);\n ++\t\t_(\"'%s' will be removed soon.\\n\"), command_name);\n +\n + \tif (hint)\n \t\tfputs(hint, stderr);\n\n \tfprintf(stderr,\n"},{"id":"551360","messageId":"xmqqqzjjbpct.fsf@gitster.g","threadId":"66224","inReplyTo":"cd741511-b457-4464-aab0-56d8554bf763@app.fastmail.com","subject":"Re: [PATCH v2] you_still_use_that(): reword the instructions","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2026-08-27T14:38:26Z","receivedAt":"2026-08-27T14:38:29Z","isPatch":true,"body":"\"Kristoffer Haugsbakk\" writes:\n\n> What I like the most about this patch/change is that it is more\n> assertive and less wordy, e.g. just “If you need a replacement”\n> not a long ceremonial sentence “If you still use this command, ...”.\n\nYes, that was exactly what I aimed for. Thanks.\n"},{"id":"551370","messageId":"xmqqecfjbliz.fsf@gitster.g","threadId":"66224","inReplyTo":"cd741511-b457-4464-aab0-56d8554bf763@app.fastmail.com","subject":"Re: [PATCH v2] you_still_use_that(): reword the instructions","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2026-08-27T16:01:08Z","receivedAt":"2026-08-27T16:01:11Z","isPatch":true,"body":"\"Kristoffer Haugsbakk\" writes:\n\n> I was thinking that maybe the final bullet point could say something\n> like\n>\n> ... and don’t send a message voting for it to come back because the\n> decision has already been made\n>\n> But it *could* be mentioned here instead. “The decision is final ...”\n> ...\n> And I also bring this up because the commit message says:\n>\n> and [the message] is not offering to reconsider a decision that has\n> already taken effect.\n\nI forgot to respond to this part. Yes, we could be more blunt and\nexplicitly state that bringing the issue back to the discussion\nstage is way too late and unwelcome.\n\nHowever, offering advice only on finding a replacement should be a\nsufficient hint for any reasonably clueful person to understand that\nit is far too late to suggest a reversal. There will always be\nthose who demand that we revisit our decisions regardless of what we\ntell them, and those people are unsalvageable.\n"},{"id":"551383","messageId":"xmqqse3z8m5g.fsf_-_@gitster.g","threadId":"66224","inReplyTo":"xmqqjypchixe.fsf@gitster.g","subject":"[PATCH v3] you_still_use_that(): reword the instructions","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2026-08-27T18:15:55Z","receivedAt":"2026-08-27T18:15:58Z","isPatch":true,"body":"The message is overly long and may mislead readers into thinking\nthere is recourse other than adopting the new workflow. Clarify\nthat the message is there merely to help them find a replacement\nworkflow, and is not offering to reconsider a decision that has\nalready taken effect.\n\nSigned-off-by: Junio C Hamano \n---\n\n * The text is unchanged from v2 except for fixing \"mailing list\";\n two tests that looked for \"nominated for removal\" string have\n been corrected.\n\n t/t4013-diff-various.sh | 2 +-\n t/t5323-pack-redundant.sh | 2 +-\n usage.c | 16 +++++++---------\n 3 files changed, 9 insertions(+), 11 deletions(-)\n\ndiff --git a/t/t4013-diff-various.sh b/t/t4013-diff-various.sh\nindex d35695f5b0..93d6e6e975 100755\n--- a/t/t4013-diff-various.sh\n+++ b/t/t4013-diff-various.sh\n@@ -478,7 +478,7 @@ EOF\n \n test_expect_success !WITH_BREAKING_CHANGES 'whatchanged needs --i-still-use-this' '\n \ttest_must_fail git whatchanged >message 2>&1 &&\n-\ttest_grep \"nominated for removal\" message\n+\ttest_grep \"will be removed soon\" message\n '\n \n test_expect_success 'log -m matches pure log' '\ndiff --git a/t/t5323-pack-redundant.sh b/t/t5323-pack-redundant.sh\nindex 2d96afd6f7..aff0bce099 100755\n--- a/t/t5323-pack-redundant.sh\n+++ b/t/t5323-pack-redundant.sh\n@@ -47,7 +47,7 @@ shared_repo=shared.git\n \n test_expect_success 'pack-redundant needs --i-still-use-this' '\n \ttest_must_fail git pack-redundant >message 2>&1 &&\n-\ttest_grep \"nominated for removal\" message\n+\ttest_grep \"will be removed soon\" message\n '\n \n git_pack_redundant='git pack-redundant --i-still-use-this'\ndiff --git a/usage.c b/usage.c\nindex 527edb1e79..90f392e89e 100644\n--- a/usage.c\n+++ b/usage.c\n@@ -386,21 +386,19 @@ NORETURN void you_still_use_that(const char *command_name, const char *hint)\n \t\t\t\t STRBUF_ENCODE_SLASH);\n \n \tfprintf(stderr,\n-\t\t_(\"'%s' is nominated for removal.\\n\"), command_name);\n+\t\t_(\"'%s' will be removed soon.\\n\"), command_name);\n \n \tif (hint)\n \t\tfputs(hint, stderr);\n \n \tfprintf(stderr,\n-\t\t_(\"If you still use this command, here's what you can do:\\n\"\n+\t\t_(\"If you need a replacement:\\n\"\n \t\t \"\\n\"\n-\t\t \"- read https://git-scm.com/docs/BreakingChanges.html\\n\"\n-\t\t \"- check if anyone has discussed this on the mailing\\n\"\n-\t\t \" list and if they came up with something that can\\n\"\n-\t\t \" help you: https://lore.kernel.org/git/?q=%s\\n\"\n-\t\t \"- send an email to to let us\\n\"\n-\t\t \" know that you still use this command and were unable\\n\"\n-\t\t \" to determine a suitable replacement\\n\"\n+\t\t \"- Read https://git-scm.com/docs/BreakingChanges.html.\\n\\n\"\n+\t\t \"- Check what others on the mailing list suggest as a replacement:\\n\"\n+\t\t \" https://lore.kernel.org/git/?q=%s\\n\\n\"\n+\t\t \"- Send an email to asking for help, only if\\n\"\n+\t\t \" suggestions by others do not work for you.\\n\"\n \t\t \"\\n\"),\n \t\tpercent_encoded.buf);\n \tstrbuf_release(&percent_encoded);\n\nInterdiff:\n diff --git a/t/t4013-diff-various.sh b/t/t4013-diff-various.sh\n index d35695f5b0..93d6e6e975 100755\n --- a/t/t4013-diff-various.sh\n +++ b/t/t4013-diff-various.sh\n @@ -478,7 +478,7 @@ EOF\n \n test_expect_success !WITH_BREAKING_CHANGES 'whatchanged needs --i-still-use-this' '\n \ttest_must_fail git whatchanged >message 2>&1 &&\n -\ttest_grep \"nominated for removal\" message\n +\ttest_grep \"will be removed soon\" message\n '\n \n test_expect_success 'log -m matches pure log' '\n diff --git a/t/t5323-pack-redundant.sh b/t/t5323-pack-redundant.sh\n index 2d96afd6f7..aff0bce099 100755\n --- a/t/t5323-pack-redundant.sh\n +++ b/t/t5323-pack-redundant.sh\n @@ -47,7 +47,7 @@ shared_repo=shared.git\n \n test_expect_success 'pack-redundant needs --i-still-use-this' '\n \ttest_must_fail git pack-redundant >message 2>&1 &&\n -\ttest_grep \"nominated for removal\" message\n +\ttest_grep \"will be removed soon\" message\n '\n \n git_pack_redundant='git pack-redundant --i-still-use-this'\n diff --git a/usage.c b/usage.c\n index d421ca426b..90f392e89e 100644\n --- a/usage.c\n +++ b/usage.c\n @@ -395,7 +395,7 @@ NORETURN void you_still_use_that(const char *command_name, const char *hint)\n \t\t_(\"If you need a replacement:\\n\"\n \t\t \"\\n\"\n \t\t \"- Read https://git-scm.com/docs/BreakingChanges.html.\\n\\n\"\n -\t\t \"- Check what others on the mailing suggest as a replacement:\\n\"\n +\t\t \"- Check what others on the mailing list suggest as a replacement:\\n\"\n \t\t \" https://lore.kernel.org/git/?q=%s\\n\\n\"\n \t\t \"- Send an email to asking for help, only if\\n\"\n \t\t \" suggestions by others do not work for you.\\n\"\n-- \n2.55.0-862-g3c6f97f7b9\n\n"}]}