pkglistgen: tool: decouple summary dict creation

The summary dictionary used to create summary-{scope}.txt was
previously returned by solve_project() via write_all_groups().

After c46dd3e, solve_project()
doesn't return anything anymore, as write_all_groups() is being
invoked just after solve_project().

Thus, the "summary" dictionary would always be None.

Since the summary dict creation is generic enough, move it
to its own function, and call it afterwards after project
solving.

This commit also drops the return statement both on write_all_groups()
and write_productcompose() (where it would always return an empty
dictionary), and moves the write_productcompose() call after
solve_project() as well to match the change with write_all_groups().

Signed-off-by: Eugenio Paolantonio <eugenio.paolantonio@suse.com>

pkglistgen/tool.py

@@ -138,7 +138,6 @@ def write_group_stubs(self):
 
     def write_productcompose(self):
         self._check_supplements()
-        summary = dict()
         archs = ['*'] + self.all_architectures
         # a single file covering all builds via multibuild flavors
         with open(os.path.join(self.productcompose_dir, 'default.productcompose'), 'a') as opfh:
@@ -178,17 +177,13 @@ def write_productcompose(self):
                     for included_arch in self.all_architectures:
                         opfh.write('  - ' + f + '_' + included_arch + '\n')
 
-        return summary
-
     def write_all_groups(self):
         self._check_supplements()
-        summary = dict()
         archs = ['*'] + self.all_architectures
         for name in self.groups:
             group = self.groups[name]
             if not group.solved:
                 continue
-            summary[name] = group.summary()
             fn = f'{group.name}.group'
             with open(os.path.join(self.output_dir, fn), 'w') as fh:
                 comment = group.comment
@@ -200,6 +195,15 @@ def write_all_groups(self):
                     x = re.sub(r'\s*<!-- reason:', ' <!-- reason:', x)
                     fh.write(x)
 
+    def make_summary(self):
+        self._check_supplements()
+        summary = dict()
+        for name in self.groups:
+            group = self.groups[name]
+            if not group.solved:
+                continue
+            summary[name] = group.summary()
+
         return summary
 
     def solve_module(self, groupname, includes, excludes, use_recommends):
@@ -613,8 +617,6 @@ def solve_project(
                         module.solved_packages[arch].pop(p, None)
 
         self._collect_unsorted_packages(modules, self.groups.get('unsorted'))
-        if not self.skip_productcompose:
-            self.write_productcompose()
 
     def strip_medium_from_staging(self, path):
         # staging projects don't need source and debug medium - and the glibc source
@@ -798,15 +800,21 @@ def update_and_solve_target(
             self.load_all_groups()
             self.write_group_stubs()
         else:
-            summary = self.solve_project(
+            self.solve_project(
                 ignore_unresolvable=str2bool(target_config.get('pkglistgen-ignore-unresolvable')),
                 ignore_recommended=str2bool(target_config.get('pkglistgen-ignore-recommended')),
                 locale=target_config.get('pkglistgen-locale'),
                 locales_from=target_config.get('pkglistgen-locales-from')
             )
+
+            if not self.skip_productcompose:
+                self.write_productcompose()
+
             if product_dir:
                 self.write_all_groups()
 
+            summary = self.make_summary()
+
         if stop_after_solve:
             return