1230569 - ./mozregression -h displays wrong usage

Status: RESOLVED FIXED

(Testing :: mozregression, defect)

Description

[Julien Pagès (:parkouss)]

From :ryanvm:

    $ mozregression --good-release 43 --bad-release 44
    usage:
     mozregression-script.py [OPTIONS] [[--bad BAD_DATE]|[--bad-release BAD_RELEASE]] [[--good GOOD_DATE]|[--good-release GOOD_RELEASE]]
     mozregression-script.py [OPTIONS] --bad-rev BAD_REV --good-rev GOOD_REV
     mozregression-script.py [OPTIONS] --launch DATE_OR_REV
     mozregression-script.py --list-build-types
     mozregression-script.py --list-releases
     mozregression-script.py --write-conf
    mozregression-script.py: error: unrecognized arguments: --good-release 43 --bad-release 44 

Needs updating since some options are not anymore in there. :)

Comment 1

[Julien Pagès (:parkouss)]

Attachment: fix usage

With the patch:

    mozregression --good-release 43 --bad-release 44
    usage:
     mozregression [OPTIONS] [--bad DATE|BUILDID|RELEASE|CHANGESET] [--good DATE|BUILDID|RELEASE|CHANGESET]
     mozregression [OPTIONS] --launch DATE|BUILDID|RELEASE|CHANGESET
     mozregression --list-build-types
     mozregression --list-releases
     mozregression --write-conf
    mozregression: error: unrecognized arguments: --good-release 43 --bad-release 44

Comment 2

[William Lachance (:wlach)]

Comment on attachment 8695907 [details] [review]
fix usage

LGTM!

Comment 3

[Julien Pagès (:parkouss)]

Cool, landed in https://github.com/mozilla/mozregression/commit/4e0133519162bff03de61ea853c9db8369576b47.

Once bug 1229976 is landed, I will do a patch release with that fix.