doc/git-repack: small fixes for geometric repacks
cover doc/git-repack: small fixes for geometric repacks
Hi,
this small series addresses two issues in our documentation for the git-repack(1) command that relate to geometric repacks.
Patrick
Patrick Steinhardt (2):
doc/git-repack: fix syntax for -g shorthand option doc/git-repack: don't mention nonexistent "--unpacked" option
Documentation/git-repack.txt | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-)
Signature · 1 line
2.42.0
PATCH 1/2 doc/git-repack: fix syntax for `-g` shorthand option
The -g switch is a shorthand for --geometric= and allows the user to specify the geometric. The documentation is wrong though and indicates that the syntax for the shorthand is -g=<factor>. In fact though, the option must be specified without the equals sign via -g<factor>.
Fix the syntax accordingly.
--- Documentation/git-repack.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Documentation/git-repack.txt +1 -1
@@ -209,7 +209,7 @@ depth is 4095. Pass the `--delta-islands` option to `git-pack-objects`, see linkgit:git-pack-objects[1]. --g=<factor>:: +-g<factor>:: --geometric=<factor>:: Arrange resulting pack structure so that each successive pack contains at least `<factor>` times the number of objects as the
Signature · 1 line
2.42.0
PATCH 2/2 doc/git-repack: don't mention nonexistent "--unpacked" option
In reply to Patrick SteinhardtThe 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.
--- Documentation/git-repack.txt | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-)
Documentation/git-repack.txt +2 -5
@@ -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. + When writing a multi-pack bitmap, `git repack` selects the largest resulting pack as the preferred pack for object selection by the MIDX (see
Signature · 1 line
2.42.0
Re: [PATCH 2/2] doc/git-repack: don't mention nonexistent "--unpacked" option
Quoted 18 lines · Patrick Steinhardt
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 <[email protected]>--- Documentation/git-repack.txt | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-)Documentation/git-repack.txt +0
-0@@ -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
Documentation/git-repack.txt uniquely by the set of packs being "rolled-up"; in other words, the · line 231
Quoted 10 lines
Documentation/git-repack.txt +2
-5progression. + -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
Quoted 6 lines · Patrick Steinhardt
Patrick Steinhardt (2):
doc/git-repack: fix syntax for-gshorthand option doc/git-repack: don't mention nonexistent "--unpacked" optionDocumentation/git-repack.txt | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-)
Sorry that this one fell off of my queue. These are both looking good to me, thanks for finding and fixing them!
Thanks,
Taylor