summaryrefslogtreecommitdiff
path: root/t/t5100/patch0002
diff options
context:
space:
mode:
authorLibravatar Christian Couder <christian.couder@gmail.com>2018-04-25 18:10:25 +0200
committerLibravatar Junio C Hamano <gitster@pobox.com>2018-04-26 11:07:16 +0900
commit38368cba2650bc51570356dfd660dfe192164082 (patch)
treebe3a8ced07d86c0a9eb4f370b62a01e9ad02d136 /t/t5100/patch0002
parentThe fourth batch for 2.18 (diff)
downloadtgif-38368cba2650bc51570356dfd660dfe192164082.tar.xz
perf/aggregate: use Getopt::Long for option parsing
When passing an option '--foo' that it does not recognize, the aggregate.perl script should die with an helpful error message like: Unknown option: foo ./aggregate.perl [options] [--] [<dir_or_rev>...] [--] \ [<test_script>...] > Options: --codespeed * Format output for Codespeed --reponame <str> * Send given reponame to codespeed --sort-by <str> * Sort output (only "regression" \ criteria is supported) rather than: fatal: Needed a single revision rev-parse --verify --foo: command returned error: 128 To implement that let's use Getopt::Long for option parsing instead of the current manual and sloppy parsing. This should save some code and make option parsing simpler, tighter and safer. This will avoid something like 'foo--sort-by=regression' to be handled as if '--sort-by=regression' had been used, for example. As Getopt::Long eats '--' at the end of options, this changes a bit the way '--' is handled as we can now have '--' both after the options and before the scripts. Signed-off-by: Christian Couder <chriscool@tuxfamily.org> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t5100/patch0002')
0 files changed, 0 insertions, 0 deletions