{"thread":{"id":"38446","subject":"[PATCH] Documentation: what does \"git log --indexed-objects\" even mean?","startedAt":"2015-01-23T19:49:05Z","lastAt":"2015-01-23T19:52:53Z","messageCount":2,"participants":["Junio C Hamano","Jeff King"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"255213","messageId":"xmqqd265qnv2.fsf@gitster.dls.corp.google.com","threadId":"38446","inReplyTo":null,"subject":"[PATCH] Documentation: what does \"git log --indexed-objects\" even mean?","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2015-01-23T19:49:05Z","receivedAt":"2015-01-23T19:49:05Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"4fe10219 (rev-list: add --indexed-objects option, 2014-10-16) adds\n\"--indexed-objects\" option to \"rev-list\", and it is only useful in\nthe context of \"git rev-list\" and not \"git log\".  There are other\nobject traversal options that do not make sense for \"git log\" that\nare shown in the manual page.\n\nMove the description of \"--indexed-objects\" to the object traversal\nsection so that it sits together with its friends \"--objects\",\n\"--objects-edge\", etc. and then show them only in \"git rev-list\"\ndocumentation.\n\nSigned-off-by: Junio C Hamano <gitster@pobox.com>\n---\n Documentation/rev-list-options.txt | 12 +++++++-----\n 1 file changed, 7 insertions(+), 5 deletions(-)\n\ndiff --git a/Documentation/rev-list-options.txt b/Documentation/rev-list-options.txt\nindex 2984f40..97ef2e8 100644\n--- a/Documentation/rev-list-options.txt\n+++ b/Documentation/rev-list-options.txt\n@@ -172,11 +172,6 @@ explicitly.\n \tPretend as if all objects mentioned by reflogs are listed on the\n \tcommand line as `<commit>`.\n \n---indexed-objects::\n-\tPretend as if all trees and blobs used by the index are listed\n-\ton the command line.  Note that you probably want to use\n-\t`--objects`, too.\n-\n --ignore-missing::\n \tUpon seeing an invalid object name in the input, pretend as if\n \tthe bad input was not given.\n@@ -644,6 +639,7 @@ Object Traversal\n \n These options are mostly targeted for packing of Git repositories.\n \n+ifdef::git-rev-list[]\n --objects::\n \tPrint the object IDs of any object referenced by the listed\n \tcommits.  `--objects foo ^bar` thus means ``send me\n@@ -662,9 +658,15 @@ These options are mostly targeted for packing of Git repositories.\n \tcommits at the cost of increased time.  This is used instead of\n \t`--objects-edge` to build ``thin'' packs for shallow repositories.\n \n+--indexed-objects::\n+\tPretend as if all trees and blobs used by the index are listed\n+\ton the command line.  Note that you probably want to use\n+\t`--objects`, too.\n+\n --unpacked::\n \tOnly useful with `--objects`; print the object IDs that are not\n \tin packs.\n+endif::git-rev-list[]\n \n --no-walk[=(sorted|unsorted)]::\n \tOnly show the given commits, but do not traverse their ancestors.\n"},{"id":"255214","messageId":"20150123195253.GA6935@peff.net","threadId":"38446","inReplyTo":"xmqqd265qnv2.fsf@gitster.dls.corp.google.com","subject":"Re: [PATCH] Documentation: what does \"git log --indexed-objects\" even mean?","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2015-01-23T19:52:53Z","receivedAt":"2015-01-23T19:52:53Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Fri, Jan 23, 2015 at 11:49:05AM -0800, Junio C Hamano wrote:\n\n> 4fe10219 (rev-list: add --indexed-objects option, 2014-10-16) adds\n> \"--indexed-objects\" option to \"rev-list\", and it is only useful in\n> the context of \"git rev-list\" and not \"git log\".  There are other\n> object traversal options that do not make sense for \"git log\" that\n> are shown in the manual page.\n> \n> Move the description of \"--indexed-objects\" to the object traversal\n> section so that it sits together with its friends \"--objects\",\n> \"--objects-edge\", etc. and then show them only in \"git rev-list\"\n> documentation.\n\nYeah, that makes sense to me.\n\nAcked-by: Jeff King <peff@peff.net>\n\n-Peff\n"}]}