[PATCH] docs: document caveats of rev-list's object-name output
- From
Jeff King <peff@peff.net>
- Date
- Mar 28, 2023, 18:26 UTC
- Message-ID
- <20230328182650.GC18558@coredump.intra.peff.net>
- In-Reply-To
- <AS1PR02MB8185DF947EBC583318481E1994889@AS1PR02MB8185.eurprd02.prod.outlook.com>
On Tue, Mar 28, 2023 at 08:08:02AM +0000, Baumann, Moritz wrote:
Show 5 quoted lines
> > Those names are really just intended as hints for pack-objects. I > > suspect the documentation could be more clear about these limitations. > > That would indeed be great and would have likely prevented the obvious > misconceptions on my side.
Here's what I came up with.
-- >8 -- Subject: docs: document caveats of rev-list's object-name output
At first glance, the names given by "rev-list --objects" seem like a good way to see which paths are present in a set of commits. But there are some subtle gotchas there. We do not document the format of the names at all, so let's do so, along with warning of these problems.
I intentionally did not document the exact format of the names here, as I don't think it's something we want people to rely on (though I doubt in practice that we'd change it at this point).
Though all of this is historically tied to "--objects", these days we have a separate "--object-names" flag which can turn the names off or on. So I put the detailed documentation there, but added a note from --objects (which did not otherwise mention the names at all, even though they are on by default).
Signed-off-by: Jeff King <peff@peff.net> --- I also considered adding a specific "if you want the names of each file in a range of commits, pipe to diff-tree" example. But it seemed like it would clutter up this section. It might be OK as a stand-alone in the EXAMPLES section, but should probably be done as a separate patch if anyone is interested.
Documentation/rev-list-options.txt | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/Documentation/rev-list-options.txt b/Documentation/rev-list-options.txt index 90c73d6708b..3000888a908 100644 --- a/Documentation/rev-list-options.txt +++ b/Documentation/rev-list-options.txt @@ -890,7 +890,7 @@ ifdef::git-rev-list[] Print the object IDs of any object referenced by the listed commits. `--objects foo ^bar` thus means ``send me all object IDs which I need to download if I have the commit - object _bar_ but not _foo_''. + object _bar_ but not _foo_''. See also `--object-names` below. --in-commit-order:: Print tree and blob ids in order of the commits. The tree @@ -920,7 +920,12 @@ ifdef::git-rev-list[] --object-names:: Only useful with `--objects`; print the names of the object IDs - that are found. This is the default behavior. + that are found. This is the default behavior. Note that the + "name" of each object is ambiguous, and mostly intended as a + hint for packing objects. In particular: no distinction is made between + the names of tags, trees, and blobs; path names may be modified + to remove newlines; and if an object would appear multiple times + with different names, only one name is shown. --no-object-names:: Only useful with `--objects`; does not print the names of the object
-- 2.40.0.616.gf524ec75088