changeset 7256:1bdda21f186a

repo-scan: don't print empty list of results If 'gearbox repo-scan' does not add nor remove anything, the output is: Now scanning root location for new repos ... Scan completed. Added: - Missing: - These empty lists of results are not very helpful. Instead, rework the code so that a list of added/removed/missing repositories is only printed if it actually contains something.
author Thomas De Schampheleire <thomas.de_schampheleire@nokia.com>
date Tue, 01 May 2018 21:51:54 +0200
parents ca4b2c3c2ffb
children 861ef58bc36e
files kallithea/lib/paster_commands/repo_scan.py
diffstat 1 files changed, 5 insertions(+), 7 deletions(-) [+]
line wrap: on
line diff
--- a/kallithea/lib/paster_commands/repo_scan.py	Mon Apr 23 01:02:07 2018 +0200
+++ b/kallithea/lib/paster_commands/repo_scan.py	Tue May 01 21:51:54 2018 +0200
@@ -46,14 +46,12 @@
         print 'Now scanning root location for new repos ...'
         added, removed = repo2db_mapper(ScmModel().repo_scan(),
                                         remove_obsolete=rm_obsolete)
-        added = ', '.join(added) or '-'
-        removed = ', '.join(removed) or '-'
         print 'Scan completed.'
-        print 'Added: %s' % added
-        if rm_obsolete:
-            print 'Removed: %s' % removed
-        else:
-            print 'Missing: %s' % removed
+        if added:
+            print 'Added: %s' % ', '.join(added)
+        if removed:
+            print '%s: %s' % ('Removed' if rm_obsolete else 'Missing',
+                              ', '.join(removed))
 
     def get_parser(self, prog_name):
         parser = super(Command, self).get_parser(prog_name)