From 61d093f87a3f3d52435731e3c49036ed5a2ab832 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Aur=C3=A9lien=20Dunand?= <aurelien.dunand@smartjog.com>
Date: Thu, 4 Aug 2011 09:06:00 +0200
Subject: [PATCH] Cosmetics changes for list command

Signed-off-by: Seblu <sebastien.luttringer@smartjog.com>
---
 bin/is                       | 29 +++++++++++++++--------------
 installsystems/repository.py | 18 +++++++++---------
 2 files changed, 24 insertions(+), 23 deletions(-)

diff --git a/bin/is b/bin/is
index 50c2651..30a9b61 100755
--- a/bin/is
+++ b/bin/is
@@ -143,37 +143,37 @@ def c_list(parser, args):
     List images in repository or image content
     '''
     # List available repositories
-    if len(args.list_args) == 0:
+    if len(args.values) == 0:
         repoman = load_repositories(args)
         arrow("Repositories")
         repoman.show(verbose=args.detail)
         return
     # list direct image or reposity content
-    elif len(args.list_args) == 1:
+    elif len(args.values) == 1:
         # list image file content
-        if (istools.pathtype(args.list_args[0]) == "file" and
-            os.path.isfile(args.list_args[0])):
-            pkg = PackageImage(args.list_args[0])
+        if (istools.pathtype(args.values[0]) == "file" and
+            os.path.isfile(args.values[0])):
+            pkg = PackageImage(args.values[0])
             arrow("Image %s v%s" % (pkg.name, pkg.version))
             pkg.show(verbose=args.detail)
         # list repository content
         else:
-            args.repo_filter = args.list_args[0]
+            args.repo_filter = args.values[0]
             repoman = load_repositories(args)
             for repo in repoman:
                 arrow("Image in %s" % repo.config.name)
                 repo.show(args.detail)
         return
     # list last image version
-    elif len(args.list_args) == 2:
-        args.repo_filter = args.list_args[0]
-        args.image = args.list_args[1]
+    elif len(args.values) == 2:
+        args.repo_filter = args.values[0]
+        args.image = args.values[1]
         args.image_version = None
     # list specific image version
-    elif len(args.list_args) == 3:
-        args.repo_filter = args.list_args[0]
-        args.image = args.list_args[1]
-        args.image_version = args.list_args[2]
+    elif len(args.values) == 3:
+        args.repo_filter = args.values[0]
+        args.image = args.values[1]
+        args.image_version = args.values[2]
     else:
         args.subparser.print_usage()
         exit(1)
@@ -330,7 +330,8 @@ p_install.set_defaults(func=c_install, subparser=p_install)
 p_list = subparsers.add_parser("list", help=c_list.__doc__.lower())
 p_list.add_argument("-l", action="store_true", dest="detail",
                     default=False, help="detailled list output")
-p_list.add_argument("list_args", nargs="*", help="list content of image or repositories")
+p_list.add_argument("values", nargs="*",
+                    help="image or repositories to list")
 p_list.set_defaults(func=c_list, subparser=p_list)
 
 # update command parser
diff --git a/installsystems/repository.py b/installsystems/repository.py
index 5579fdb..2e4a94b 100644
--- a/installsystems/repository.py
+++ b/installsystems/repository.py
@@ -188,18 +188,18 @@ class Repository(object):
 
         for (image_md5, image_name, image_version, image_date, image_author,
              image_description, image_size) in images:
-            out('#light##yellow#%s #reset#v%s' % (image_name, image_version))
+            out("#light##yellow#%s #reset#v%s" % (image_name, image_version))
             if verbose:
-                out('  #yellow#Date:#reset# %s' % time.asctime(time.gmtime(image_date)))
-                out('  #yellow#Description:#reset# %s' % image_description)
-                out('  #yellow#Author:#reset# %s' % image_author)
-                out('  #yellow#MD5:#reset# %s' % image_md5)
+                out("  #yellow#Date:#reset# %s" % time.asctime(time.gmtime(image_date)))
+                out("  #yellow#Description:#reset# %s" % image_description)
+                out("  #yellow#Author:#reset# %s" % image_author)
+                out("  #yellow#MD5:#reset# %s" % image_md5)
                 payloads = self.db.ask("SELECT md5, name, size FROM payload\
                                     WHERE image_md5 = ?", (image_md5,)).fetchall()
                 for payload_md5, payload_name, payload_size in payloads:
-                    out('  #light##yellow#Payload:#reset# %s' % payload_name)
-                    out('    #yellow#Size:#reset# %s' % (istools.human_size(payload_size)))
-                    out('    #yellow#MD5:#reset# %s' % payload_md5)
+                    out("  #light##yellow#Payload:#reset# %s" % payload_name)
+                    out("    #yellow#Size:#reset# %s" % (istools.human_size(payload_size)))
+                    out("    #yellow#MD5:#reset# %s" % payload_md5)
                 out()
 
     def search(self, pattern):
@@ -555,5 +555,5 @@ class RepositoryManager(object):
             repo.config.name
             s = "#light##blue#%s#reset#"% repo.config.name
             if verbose:
-                s += ' (%s)' % repo.config.path
+                s += " (%s)" % repo.config.path
             out(s)
-- 
GitLab