Merge lp:~jameinel/hydrazine/arg-parsing into lp:hydrazine
Proposed by
John A Meinel
Status: | Merged |
---|---|
Approved by: | Martin Pool |
Approved revision: | 71 |
Merged at revision: | 73 |
Proposed branch: | lp:~jameinel/hydrazine/arg-parsing |
Merge into: | lp:hydrazine |
Diff against target: |
36 lines (+7/-8) 1 file modified
feed-pqm (+7/-8) |
To merge this branch: | bzr merge lp:~jameinel/hydrazine/arg-parsing |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Martin Pool | Approve | ||
Review via email:
|
Description of the change
This fixes up some of the argument parsing.
Basically, there was code that said "if you have != 1 argument, abort", which doesn't work when you can supply stuff like --queued.
It then always used 'argv[1]' rather than using 'args[0]'. (So if you did 'feed-pqm --queued' it would parse the queued request, and then try and look at the --queued project in Launchpad.)
This should fix it up, and changes how it displays usage, etc. as well.
To post a comment you must log in.
Thanks.