#314 Drop some now-unneeded manual handling of arch-specific packages
Merged by siosm. Opened by adamwill.
adamwill/workstation-ostree-config arch-simplify  into  main

Download 314.patch

This lets the recently-merged improved arch-specific package
support handle all the arch-specific packages which were
previously excluded in comps-sync-exclude-list.yml and listed
manually in fedora-common-ostree.yaml. As the diff shows,
the sync script now correctly includes the same packages for
the same arches in fedora-common-ostree-pkgs.yaml, plus a few
appropriate additions of things that should be there but had
been left out, on ppc64le and aarch64.

We also drop the packages-armhfp and packages-ppc64 lists,
as we no longer build for either of those arches. This allows
us to move ostree-grub2 into the non-arch-specific list, since
it's no longer left out on any arch we care about.

Signed-off-by: Adam Williamson awilliam@redhat.com

Build succeeded.

A further potential improvement here would be to drop the listing out of bootloader packages in fedora-common-ostree.yaml and instead use the comps group cloud-bootloader-tools, which would give a very similar result. The difference would be that it would also include the packages grub2-tools and grub2-tools-extra; I don't know if you want those.

To actually achieve this we'd probably have to add cloud-bootloader-tools to the list of groups that we iterate over in comps-sync.py, this one:

for gid in ws_environ.group_ids:

that mechanism is honestly a little bit obscure anyway (it took me a few minutes to grok that what's really going on is we're using workstation-product-environment minus the gnome-desktop group as a sort of proxy for "the core environment").

Metadata Update from @siosm:
- Request assigned

Thanks for working on this. Will take a look asap.

LGTM. Thanks

rebased onto 6a689cc38098cd5fb5eecdaa31974f887d4590ef

Build succeeded.

Pull-Request has been merged by siosm

Metadata