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

Re: [PATCH 2/2] doc/git-repack: don't mention nonexistent "--unpacked" option

From
Taylor Blau <me@ttaylorr.com>
Date
Oct 30, 2023, 21:51 UTC
Message-ID
<ZUAldFRSj/HLGjOQ@nand.local>
In-Reply-To
<aa0b4fef4d8397983676394472ff86e468bfc687.1697440686.git.ps@pks.im>
On Mon, Oct 16, 2023 at 09:19:56AM +0200, Patrick Steinhardt wrote:
Show 28 quoted lines
> The documentation for geometric repacking mentions a "--unpacked" option
> that supposedly changes how loose objects are rolled up. This option has
> never existed, and the implied behaviour, namely to include all unpacked
> objects into the resulting packfile, is in fact the default behaviour.
>
> Correct the documentation to not mention this option.
>
> Signed-off-by: Patrick Steinhardt <ps@pks.im>
> ---
>  Documentation/git-repack.txt | 7 ++-----
>  1 file changed, 2 insertions(+), 5 deletions(-)
>
> diff --git a/Documentation/git-repack.txt b/Documentation/git-repack.txt
> index dfd2a59c50..d61078b697 100644
> --- a/Documentation/git-repack.txt
> +++ b/Documentation/git-repack.txt
> @@ -226,11 +226,8 @@ uniquely by the set of packs being "rolled-up"; in other words, the
>  packs determined to need to be combined in order to restore a geometric
>  progression.
>  +
> -When `--unpacked` is specified, loose objects are implicitly included in
> -this "roll-up", without respect to their reachability. This is subject
> -to change in the future. This option (implying a drastically different
> -repack mode) is not guaranteed to work with all other combinations of
> -option to `git repack`.
> +Loose objects are implicitly included in this "roll-up", without respect to
> +their reachability. This is subject to change in the future.
>  +

Oops. This refers to the "--unpacked" option that pack-objects takes, not repack. I agree that mentioning "--unpacked" is too low-level a detail for this user-facing documentation, so even something like:

    When `repack` passes `--unpacked` down to `pack-objects` (which is
    the default) ...
would be too much detail for this man page.
I am very happy with the patch here as an alternative.

Thanks, Taylor

Previous: Patrick SteinhardtNext: Taylor Blau
Message 4 of 5 in “doc/git-repack: small fixes for geometric repacks”
  1. 0/2 doc/git-repack: small fixes for geometric repacksPatrick Steinhardt, Oct 16, 2023
  2. 1/2 doc/git-repack: fix syntax for `-g` shorthand optionPatrick Steinhardt, Oct 16, 2023
  3. 2/2 doc/git-repack: don't mention nonexistent "--unpacked" optionPatrick Steinhardt, Oct 16, 2023
  4. Taylor BlauOct 30, 2023
  5. Taylor BlauOct 30, 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.