git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] format-patch: clarify --creation-factor=<factor>

From
Eric Sunshine <sunshine@sunshineco.com>
Date
Jul 28, 2022, 21:09 UTC
Message-ID
<CAPig+cTxM2vr11XRU-zrNAJRwveP0W9d0x8Euvr6HKmQicKgPQ@mail.gmail.com>
In-Reply-To
<xmqqv8rh9co5.fsf@gitster.g>
On Thu, Jul 28, 2022 at 4:59 PM Junio C Hamano <gitster@pobox.com> wrote:
Show 25 quoted lines
> Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:
> >> diff --git a/Documentation/git-format-patch.txt b/Documentation/git-format-patch.txt
> >> @@ -27,7 +27,7 @@ SYNOPSIS
> >> -               [--range-diff=<previous> [--creation-factor=<percent>]]
> >> +               [--range-diff=<previous> [--creation-factor=<factor>]]
> >> @@ -321,7 +321,7 @@ product of `format-patch` is generated, and they are not passed to
> >> ---creation-factor=<percent>::
> >> +--creation-factor=<factor>::
> >
> > Looks good as far as it goes, looks like both of your patches need to
> > also tweak this bit though:
> >
> >       $ git -P grep 'percentage.*creation' -- '*.c'
> >       builtin/log.c:                      N_("percentage by which creation is weighted")),
> >       builtin/range-diff.c:                       N_("percentage by which creation is weighted")),
> >
> > Probably just s/percentage/factor/ in for those -h strings?
>
> Thanks for being extra careful.
>
> Eric, I am not sure if the other patch for the range-diff command as
> a standalone patch is worth it.  Perhaps you can help me by
> submitting a single combined patch to cover the above as well as
> what we wrote in the two patches, with you marked as the primary
> author and with Helped-by: that credits Ævar?
Will do.

(Slightly chagrined that I forgot to check the usage-string in the C files. Thanks, Ævar.)

Previous: Junio C HamanoNext: Eric Sunshine
Message 11 of 17 in “--creation-factor=100 does not show code”
  1. Eugen KonkovJul 26, 2022
  2. Johannes SchindelinJul 28, 2022
  3. Re* --creation-factor=100 does not show codeJunio C Hamano, Jul 28, 2022
  4. Ævar Arnfjörð BjarmasonJul 28, 2022
  5. Junio C HamanoJul 28, 2022
  6. Ævar Arnfjörð BjarmasonJul 28, 2022
  7. Junio C HamanoJul 28, 2022
  8. format-patch: clarify --creation-factor=<factor>Eric Sunshine, Jul 28, 2022
  9. Ævar Arnfjörð BjarmasonJul 28, 2022
  10. Junio C HamanoJul 28, 2022
  11. Eric SunshineJul 28, 2022
  12. Eric SunshineJul 30, 2022
  13. Junio C HamanoJul 31, 2022
  14. Johannes SchindelinJul 29, 2022
  15. Junio C HamanoJul 29, 2022
  16. Junio C HamanoAug 1, 2022
  17. Eugen KonkovApr 2, 2023

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.