From 49dd9393919969971d51c647199cc99da92c1777 Mon Sep 17 00:00:00 2001
From: Sebastien Luttringer <sebastien.luttringer@smartjog.com>
Date: Thu, 1 Dec 2011 18:06:08 +0100
Subject: [PATCH] clean command options

all options have now a long mode
some options name have changed
---
 bin/is             | 16 ++++++++--------
 completion/bash/is |  8 ++++----
 2 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/bin/is b/bin/is
index 9c88288..c85f920 100755
--- a/bin/is
+++ b/bin/is
@@ -184,7 +184,7 @@ def c_extract(parser, args):
     repoman = load_repositories(args)
     img, repo = select_image(args.image, repoman, args.best)
     img.extract(args.path, payload=args.payload, force=args.force,
-                gendescription=args.gendescription)
+                gendescription=args.gen_description)
 
 def c_get(parser, args):
     '''
@@ -193,7 +193,7 @@ def c_get(parser, args):
     repoman = load_repositories(args)
     for image in args.image:
         img, repo = select_image(image, repoman, args.best)
-        img.download(".", image=not args.noimage, payload=args.payload, force=args.force)
+        img.download(".", image=not args.no_image, payload=args.payload, force=args.force)
 
 def c_help(parser, args):
     '''
@@ -438,9 +438,9 @@ p_diff.set_defaults(func=c_diff)
 
 # extract command parser
 p_extract = subparsers.add_parser("extract", help=c_extract.__doc__.lower())
-p_extract.add_argument("-D", action="store_true", dest="gendescription", default=False,
+p_extract.add_argument("-g", "--gen-description", action="store_true", default=False,
                        help="generate a description file from metadata")
-p_extract.add_argument("-p", action="store_true", dest="payload", default=False,
+p_extract.add_argument("-p", "--payload", action="store_true", default=False,
                        help="extract payloads")
 p_extract.add_argument("-f", "--force", action="store_true", default=False,
                        help="overwrite existing destinations")
@@ -453,9 +453,9 @@ p_extract.set_defaults(func=c_extract)
 
 # get command parser
 p_get = subparsers.add_parser("get", help=c_get.__doc__.lower())
-p_get.add_argument("-p", action="store_true", dest="payload", default=False,
+p_get.add_argument("-p", "--payload", action="store_true", default=False,
                    help="get payloads")
-p_get.add_argument("-I", action="store_true", dest="noimage", default=False,
+p_get.add_argument("-I", "--no-image", action="store_true", default=False,
                    help="do not get image")
 p_get.add_argument("-f", "--force", action="store_true", default=False,
                    help="overwrite existing destinations")
@@ -472,7 +472,7 @@ p_help.set_defaults(func=c_help, subparser=subparsers)
 
 # info command parser
 p_info = subparsers.add_parser("info", help=c_info.__doc__.lower())
-p_info.add_argument("-v", action="store_true", dest="verbose", default=False,
+p_info.add_argument("-v", "--verbose", action="store_true", default=False,
                     help="verbose output")
 p_info.add_argument("-c", "--changelog", action="store_true", default=False,
                     help="display image changelog")
@@ -499,7 +499,7 @@ p_install.set_defaults(func=c_install, subparser=p_install)
 
 # list command parser
 p_list = subparsers.add_parser("list", help=c_list.__doc__.lower())
-p_list.add_argument("-v", action="store_true", dest="verbose", default=False,
+p_list.add_argument("-v", "--verbose", action="store_true", default=False,
                     help="verbose output")
 p_list.add_argument("-b", "--best", action="store_true", default=False,
                     help="in best mode, image is the most recent in all repositories")
diff --git a/completion/bash/is b/completion/bash/is
index 0dacac0..eb7f332 100644
--- a/completion/bash/is
+++ b/completion/bash/is
@@ -88,13 +88,13 @@ _is() {
             (( args < 4 )) && c_args=$(images)
             ;;
 	extract)
-            [[ "$cur" == -* ]] && _opt '-h --help -f --force -b --best -p -D' && return 0
+            [[ "$cur" == -* ]] && _opt '-h --help -f --force -b --best -p --payload -g --gen-description' && return 0
 	    _count_args
 	    (( args == 2 )) && _image
             (( args == 3 )) && _filedir -d
             ;;
 	get)
-            [[ "$cur" == -* ]] && _opt '-h --help -f --force -b --best -p' && return 0
+            [[ "$cur" == -* ]] && _opt '-h --help -f --force -b --best --payload -I --no-image' && return 0
             _image
             ;;
 	help)
@@ -102,7 +102,7 @@ _is() {
 	    (( args == 2 )) && _opt "${cmds[@]}"
             ;;
 	info)
-            [[ "$cur" == -* ]] && _opt '-v -c --changelog -b --best' && return 0
+            [[ "$cur" == -* ]] && _opt '-v --verbose -c --changelog -b --best' && return 0
             _image
             ;;
 	init)
@@ -116,7 +116,7 @@ _is() {
 	    (( args > 2 )) && _filedir
             ;;
 	list)
-            [[ "$cur" == -* ]] && _opt '-h --help -b --best -v -c --changelog -p --as-path' && return 0
+            [[ "$cur" == -* ]] && _opt '-h --help -b --best --verbose -c --changelog -p --as-path' && return 0
 	    _repo
 	    _image
             ;;
-- 
GitLab