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

Re: [PATCH] builtin/repack.c: set a default factor for '--geometric'

From
Taylor Blau <me@ttaylorr.com>
Date
Apr 20, 2021, 20:31 UTC
Message-ID
<YH86ENkPrTeWnnBb@nand.local>
In-Reply-To
<xmqqv98g4sbl.fsf@gitster.g>
On Tue, Apr 20, 2021 at 01:25:02PM -0700, Junio C Hamano wrote:
Show 11 quoted lines
> Taylor Blau <me@ttaylorr.com> writes:
>
> > --g=<factor>::
> > ---geometric=<factor>::
> > +-g=[<factor>]::
> > +--geometric[=<factor>]::
>
> Do you mean "git repack -g= -d" is (1) accepted and/or (2)
> recommended?
>
> If not, we'd want "-g[=<factor>]" similar to the longer form, no?

Thanks for spotting. The '[' should come before the '=' character, not after. I'll send a reroll with this (and a suggestion from Stolee).

Thanks, Taylor

Previous: Junio C HamanoNext: Taylor Blau
Message 4 of 6 in “builtin/repack.c: set a default factor for '--geometric'”
  1. builtin/repack.c: set a default factor for '--geometric'Taylor Blau, Apr 20, 2021
  2. Derrick StoleeApr 20, 2021
  3. Junio C HamanoApr 20, 2021
  4. Taylor BlauApr 20, 2021
  5. builtin/repack.c: set a default factor for '--geometric'Taylor Blau, Apr 20, 2021
  6. Andreas SchwabApr 20, 2021

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.