{"thread":{"id":"64792","subject":"git shortlog --committer vs --committer=<pattern>","startedAt":"2026-01-13T15:22:35Z","lastAt":"2026-01-13T18:01:47Z","messageCount":5,"participants":["Manuel Rego Casasnovas","Junio C Hamano","rsbecker@nexbridge.com","Pushkar Singh"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"533747","messageId":"e2733434-f632-4be5-a9e0-28412b4043c4@igalia.com","threadId":"64792","inReplyTo":null,"subject":"git shortlog --committer vs --committer=<pattern>","fromName":"Manuel Rego Casasnovas","fromEmail":"rego@igalia.com","sentAt":"2026-01-13T15:22:31Z","receivedAt":"2026-01-13T15:22:35Z","isPatch":false,"sender":{"key":"rego@igalia.com","avatar":null},"body":"Hi,\n\nDoing something like this in a Git repository returns an error (git \nversion 2.51.0):\n$ git shortlog --committer=foo\nerror: option `committer' takes no value\n\nHowever the option \"--committer=<pattern>\" is in the documentation:\nhttps://git-scm.com/docs/git-shortlog#Documentation/git-shortlog.txt---committerpattern\n\nBut it seems to be in conflict with \"--committer\":\nhttps://git-scm.com/docs/git-shortlog#Documentation/git-shortlog.txt---committer\n\nIt would be nice to either allow using \"--committer=<pattern>\" for \nshortlog, or if that's not possible fix the documentation.\n\nThanks,\n   Rego\n\n"},{"id":"533749","messageId":"xmqqms2hjyrg.fsf@gitster.g","threadId":"64792","inReplyTo":"e2733434-f632-4be5-a9e0-28412b4043c4@igalia.com","subject":"Re: git shortlog --committer vs --committer=<pattern>","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2026-01-13T15:52:35Z","receivedAt":"2026-01-13T15:52:38Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Manuel Rego Casasnovas <rego@igalia.com> writes:\n\n> Doing something like this in a Git repository returns an error (git \n> version 2.51.0):\n> $ git shortlog --committer=foo\n> error: option `committer' takes no value\n\nThanks for noticing and reporting.\n\n> It would be nice to either allow using \"--committer=<pattern>\" for \n> shortlog\n\nI took a brief look at the command line parser's source.  It\nshouldn't be impossible to improve the status quo, but it would be\nimpossible to match what other commands in the \"git regv-lits\"\nfamily of commands like \"git log\" would do, because \"--committer\"\ncan take its <pattern> as a separate parameter.\n\nSo it is clear that \"git log --committer\" (no other parameters) is a\nsyntax error, \"git log --committer -n\" (no other parameters) is not\nan error but is looking for a committer that match the pattern -n,\nbut the same reasoning cannot apply for \"git shortlog\".  Most\nnotably, \"git shortlog --committer -n\" cannot behave the same way as\n\"git shortlog --committer=-n\".\n\n\n\n\n"},{"id":"533750","messageId":"018701dc84a4$39555970$ac000c50$@nexbridge.com","threadId":"64792","inReplyTo":"e2733434-f632-4be5-a9e0-28412b4043c4@igalia.com","subject":"RE: git shortlog --committer vs --committer=<pattern>","fromName":"","fromEmail":"rsbecker@nexbridge.com","sentAt":"2026-01-13T15:49:32Z","receivedAt":"2026-01-13T15:58:21Z","isPatch":false,"sender":{"key":"randall.becker@nexbridge.ca","avatar":"https://avatars.githubusercontent.com/u/28956764?v=4"},"body":"On January 13, 2026 10:23 AM, Manuel Rego Casasnovas wrote:\n>Doing something like this in a Git repository returns an error (git version 2.51.0):\n>$ git shortlog --committer=foo\n>error: option `committer' takes no value\n>\n>However the option \"--committer=<pattern>\" is in the documentation:\n>https://git-scm.com/docs/git-shortlog#Documentation/git-shortlog.txt---\n>committerpattern\n>\n>But it seems to be in conflict with \"--committer\":\n>https://git-scm.com/docs/git-shortlog#Documentation/git-shortlog.txt---\n>committer\n>\n>It would be nice to either allow using \"--committer=<pattern>\" for shortlog, or if\n>that's not possible fix the documentation.\n\nI do not see --committer=<pattern>, but do see --committer pattern in the online\nhelp and documentation. --committer is an alias for --group=committer.\nPattern applies to the log entries.\n\n"},{"id":"533752","messageId":"CALE2CrSUFt_HGy1+9H08OMZLTjdZt=NZsqvqa1a-rbvDOzjVWw@mail.gmail.com","threadId":"64792","inReplyTo":"018701dc84a4$39555970$ac000c50$@nexbridge.com","subject":"Re: git shortlog --committer vs --committer=<pattern>","fromName":"Pushkar Singh","fromEmail":"pushkarkumarsingh1970@gmail.com","sentAt":"2026-01-13T16:20:38Z","receivedAt":"2026-01-13T16:20:50Z","isPatch":false,"sender":{"key":"pushkarkumarsingh1970@gmail.com","avatar":"https://avatars.githubusercontent.com/u/173247767?v=4"},"body":"Hi Junio,\n\nThanks for the clarification.\n\nI confirmed that this comes from git-shortlog pulling in\nDocumentation/rev-list-options.adoc, which documents\n--author=<pattern> and --committer=<pattern>, even though\ngit shortlog treats --committer as an alias for --group=committer\nand therefore cannot accept a value.\n\nSo this looks like a documentation mismatch rather than just\nthe website.\n\nI will prepare a documentation patch to make git-shortlog’s\noptions unambiguous.\n\nThanks,\nPushkar\n\nOn Tue, Jan 13, 2026 at 9:28 PM <rsbecker@nexbridge.com> wrote:\n>\n> On January 13, 2026 10:23 AM, Manuel Rego Casasnovas wrote:\n> >Doing something like this in a Git repository returns an error (git version 2.51.0):\n> >$ git shortlog --committer=foo\n> >error: option `committer' takes no value\n> >\n> >However the option \"--committer=<pattern>\" is in the documentation:\n> >https://git-scm.com/docs/git-shortlog#Documentation/git-shortlog.txt---\n> >committerpattern\n> >\n> >But it seems to be in conflict with \"--committer\":\n> >https://git-scm.com/docs/git-shortlog#Documentation/git-shortlog.txt---\n> >committer\n> >\n> >It would be nice to either allow using \"--committer=<pattern>\" for shortlog, or if\n> >that's not possible fix the documentation.\n>\n> I do not see --committer=<pattern>, but do see --committer pattern in the online\n> help and documentation. --committer is an alias for --group=committer.\n> Pattern applies to the log entries.\n>\n>\n"},{"id":"533762","messageId":"CALE2CrS9YAqnX8ikNYzyD_i3H1rzhFQvKLijbnbkZh7VqOSZOQ@mail.gmail.com","threadId":"64792","inReplyTo":"CALE2CrSUFt_HGy1+9H08OMZLTjdZt=NZsqvqa1a-rbvDOzjVWw@mail.gmail.com","subject":"Re: git shortlog --committer vs --committer=<pattern>","fromName":"Pushkar Singh","fromEmail":"pushkarkumarsingh1970@gmail.com","sentAt":"2026-01-13T18:01:34Z","receivedAt":"2026-01-13T18:01:47Z","isPatch":false,"sender":{"key":"pushkarkumarsingh1970@gmail.com","avatar":"https://avatars.githubusercontent.com/u/173247767?v=4"},"body":"Follow up to my previous mail.\n\nAfter checking the generated git-shortlog(1) and git shortlog --help, both\n--committer (grouping) and --committer=<pattern> (commit limiting) are\nindeed documented correctly.\n\nThe real issue is the option parser ambiguity that Junio described, where\n--committer=foo is parsed as the grouping option and rejected, even though\nthe filtering form exists.\n\nSo this is not a documentation error but a command line parsing limitation.\nThanks for the clarification.\n\nOn Tue, Jan 13, 2026 at 9:50 PM Pushkar Singh\n<pushkarkumarsingh1970@gmail.com> wrote:\n>\n> Hi Junio,\n>\n> Thanks for the clarification.\n>\n> I confirmed that this comes from git-shortlog pulling in\n> Documentation/rev-list-options.adoc, which documents\n> --author=<pattern> and --committer=<pattern>, even though\n> git shortlog treats --committer as an alias for --group=committer\n> and therefore cannot accept a value.\n>\n> So this looks like a documentation mismatch rather than just\n> the website.\n>\n> I will prepare a documentation patch to make git-shortlog’s\n> options unambiguous.\n>\n> Thanks,\n> Pushkar\n>\n> On Tue, Jan 13, 2026 at 9:28 PM <rsbecker@nexbridge.com> wrote:\n> >\n> > On January 13, 2026 10:23 AM, Manuel Rego Casasnovas wrote:\n> > >Doing something like this in a Git repository returns an error (git version 2.51.0):\n> > >$ git shortlog --committer=foo\n> > >error: option `committer' takes no value\n> > >\n> > >However the option \"--committer=<pattern>\" is in the documentation:\n> > >https://git-scm.com/docs/git-shortlog#Documentation/git-shortlog.txt---\n> > >committerpattern\n> > >\n> > >But it seems to be in conflict with \"--committer\":\n> > >https://git-scm.com/docs/git-shortlog#Documentation/git-shortlog.txt---\n> > >committer\n> > >\n> > >It would be nice to either allow using \"--committer=<pattern>\" for shortlog, or if\n> > >that's not possible fix the documentation.\n> >\n> > I do not see --committer=<pattern>, but do see --committer pattern in the online\n> > help and documentation. --committer is an alias for --group=committer.\n> > Pattern applies to the log entries.\n> >\n> >\n"}]}