diff options
author | Jakob Gruber <jakob.gruber@gmail.com> | 2010-10-08 22:34:17 +0200 |
---|---|---|
committer | Dan McGee <dan@archlinux.org> | 2010-10-11 20:57:26 -0500 |
commit | 827258f32ee898ed5422bc9219d9a42bd9bd2360 (patch) | |
tree | af282868e81343ecf5a859b3084e13d1a1a7eda0 /test | |
parent | 0ff2a9149711b9179d288f579c792890cd81d79b (diff) | |
download | pacman-827258f32ee898ed5422bc9219d9a42bd9bd2360.tar.xz |
CLI args: reorganize parsing
Split parsing of CLI arguments into separate functions:
parsearg_op (operations)
parsearg_global (global options)
parsearg_{database,query,remove,sync,deptest,upgrade}
Organization strictly follows the manpage (even where the manpage is
incorrect) - these cases will be fixed in the following commits.
Switch cases are copy/pasted and statements unrelated to chosen
operation are deleted.
Parsing logic adjusted as follows:
1) Parse operation
2) If we can bail out early (duplicate op, help/version requested) do so
3) Parse arguments again:
foreach arg:
if arg is operation:
continue
tryparse_args_specific_to_op
if unsuccessful tryparse_args_global
if unsuccessful print error message and exit
Signed-off-by: Xavier Chantry <chantry.xavier@gmail.com>
Signed-off-by: Dan McGee <dan@archlinux.org>
Diffstat (limited to 'test')
-rw-r--r-- | test/pacman/tests/pacman005.py | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/test/pacman/tests/pacman005.py b/test/pacman/tests/pacman005.py index 99c25b26..bb21ad4f 100644 --- a/test/pacman/tests/pacman005.py +++ b/test/pacman/tests/pacman005.py @@ -6,6 +6,3 @@ self.addpkg2db("local", p) self.args = "-Qy" self.addrule("PACMAN_RETCODE=1") - -self.expectfailure = True - |