Show summary only when specified

Since now added, removed and updated packages already show the amount of changed packages.
rpm-software-management · May 27, 2022 · 5f39a0e · 5f39a0e
1 parent 709b994
commit 5f39a0e
Showing 1 changed file with 17 additions and 15 deletions.
diff --git a/plugins/repodiff.py b/plugins/repodiff.py
@@ -78,6 +78,8 @@ def set_argparser(parser):
         parser.add_argument("--hide-author", action="store_true",
                             help=_("Hide the authors name and email address"
                                    "in the changelog."))
+        parser.add_argument("--summary", action="store_true",
+                            help=_("Sum up changes after all changes have been listed."))
 
     def configure(self):
         demands = self.cli.demands
@@ -281,19 +283,19 @@ def report_modified(pkg_old, pkg_new=None, sub_pkgs=[]):
                     report_modified(pkg_old, pkg_new,
                                     new_sub_packages[pkg_new.name]
                                     if pkg_new.name in new_sub_packages else None)
-
-        print(_("\n# Summary"))
-        print(_("- Added packages: {}").format(len(repodiff['added'])))
-        print(_("- Removed packages: {}").format(len(repodiff['removed'])))
-        if self.opts.downgrade:
-            print(_("- Upgraded packages: {}").format(len(repodiff['upgraded'])))
-            print(_("- Downgraded packages: {}").format(len(repodiff['downgraded'])))
-        else:
-            print(_("- Modified packages: {}").format(
-                len(repodiff['upgraded']) + len(repodiff['downgraded'])))
-        if self.opts.size:
-            print(_("- Size of added packages: {}").format(sizestr(sizes['added'])))
-            print(_("- Size of removed packages: {}").format(sizestr(sizes['removed'])))
+        if self.opts.summary:
+            print(_("\n# Summary"))
+            print(_("- Added packages: {}").format(len(repodiff['added'])))
+            print(_("- Removed packages: {}").format(len(repodiff['removed'])))
+            if self.opts.downgrade:
+                print(_("- Upgraded packages: {}").format(len(repodiff['upgraded'])))
+                print(_("- Downgraded packages: {}").format(len(repodiff['downgraded'])))
+            else:
+                print(_("- Modified packages: {}").format(
+                    len(repodiff['upgraded']) + len(repodiff['downgraded'])))
+            if self.opts.size:
+                print(_("- Size of added packages: {}").format(sizestr(sizes['added'])))
+                print(_("- Size of removed packages: {}").format(sizestr(sizes['removed'])))
             if not self.opts.downgrade:
                 print(_("Size of modified packages: {}").format(
                     sizestr(sizes['upgraded'] + sizes['downgraded'])))
@@ -302,8 +304,8 @@ def report_modified(pkg_old, pkg_new=None, sub_pkgs=[]):
                     sizestr(sizes['upgraded'])))
                 print(_("- Size of downgraded packages: {}").format(
                     sizestr(sizes['downgraded'])))
-            print(_("- Size change: {}").format(
-                sizestr(sizes['added'] + sizes['upgraded'] + sizes['downgraded'] -
+                print(_("- Size change: {}").format(
+                    sizestr(sizes['added'] + sizes['upgraded'] + sizes['downgraded'] -
                         sizes['removed'])))
 
     def run(self):