rpm: Consume group info for sub RPMs (#931)

kellyma2 · web-flow · commit 8c235607a015 · 2025-02-05T00:21:31.000-05:00
In the original subrpm implementation this field appears to have been
overlooked.  This change just injects the correct specfile lines based
on the existing group info from the provider.
diff --git a/pkg/rpm_pfg.bzl b/pkg/rpm_pfg.bzl
@@ -350,6 +350,9 @@ def _process_subrpm(ctx, rpm_name, rpm_info, rpm_ctx, debuginfo_type):
         "Summary: %s" % rpm_info.summary,
     ]
 
+    if rpm_info.group:
+        rpm_lines.append("Group: %s" % rpm_info.group)
+
     if rpm_info.architecture:
         rpm_lines.append("BuildArch: %s" % rpm_info.architecture)
 

Original file line number	Diff line number	Diff line change
`@@ -350,6 +350,9 @@ def _process_subrpm(ctx, rpm_name, rpm_info, rpm_ctx, debuginfo_type):`
`350`	`350`	`"Summary: %s" % rpm_info.summary,`
`351`	`351`	`]`
`352`	`352`
	`353`	`+ if rpm_info.group:`
	`354`	`+ rpm_lines.append("Group: %s" % rpm_info.group)`
	`355`	`+`
`353`	`356`	`if rpm_info.architecture:`
`354`	`357`	`rpm_lines.append("BuildArch: %s" % rpm_info.architecture)`
`355`	`358`