{"thread":{"id":"46501","subject":"git send-email -v n fails","startedAt":"2017-08-02T13:46:46Z","lastAt":"2017-08-02T16:35:32Z","messageCount":2,"participants":["Olaf Hering","Junio C Hamano"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"325430","messageId":"20170802134635.GA19311@aepfle.de","threadId":"46501","inReplyTo":null,"subject":"git send-email -v n fails","fromName":"Olaf Hering","fromEmail":"olaf@aepfle.de","sentAt":"2017-08-02T13:46:35Z","receivedAt":"2017-08-02T13:46:46Z","isPatch":false,"sender":{"key":"olaf@aepfle.de","avatar":"https://avatars.githubusercontent.com/u/942324?v=4"},"body":"I think send-email should understand all options of format-patch.\nAt least for '-v n' this fails, one has to type '--reroll-count n' as a\nworkaround with git version 2.13.3:\n\n+ git send-email -v 2 --annotate --confirm=always --to-cmd=scripts/get_maintainer.pl 'HEAD^'\nfatal: ambiguous argument '2': unknown revision or path not in the working tree.\nUse '--' to separate paths from revisions, like this:\n'git <command> [<revision>...] -- [<file>...]'\nformat-patch -o /tmp/fVUtbB6OWI 1 HEAD^: command returned error: 128\n\nOlaf\n"},{"id":"325440","messageId":"xmqqy3r2c4hl.fsf@gitster.mtv.corp.google.com","threadId":"46501","inReplyTo":"20170802134635.GA19311@aepfle.de","subject":"Re: git send-email -v n fails","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-08-02T16:35:18Z","receivedAt":"2017-08-02T16:35:32Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Olaf Hering <olaf@aepfle.de> writes:\n\n> I think send-email should understand all options of format-patch.\n\nI actually think it was a mistake to allow send-email drive\nformat-patch ;-).\n\n> At least for '-v n' this fails, one has to type '--reroll-count n' as a\n> workaround with git version 2.13.3:\n\nI wonder if \"-v2\" would work here, though.  It seems that the hack\nsend-email uses to \"understand\" format-patch options (i.e. the\nsubroutine is_format_patch_arg) is not prepared to take an option\nwith parameter.\n\n"}]}