On Fri, 2018-06-08 at 09:49 +0100, Daniel P. Berrangé wrote: [...] > + perl-generators: > + default: > + Fedora: perl-generators Sorting is off - this should be further down, after perl-accessors. Empty entries are only necessary when you have to remove a value that was added by a more generic entry; in this case, you can leave it out and only have the Fedora entry. [...] > packages: > + - perl-generators > - perl-Archive-Tar > - perl-CPAN-Changes > - perl-Module-Build Sorting is again off. [...] > packages: > + - perl-generators > - perl-Archive-Tar > - perl-Config-Record > - perl-Digest Here too. Regardless of the nits pointed out above, this commit will fix the build issue on Fedora but not on CentOS 7, where neither perl-generators nor perl-interpreter exist: you need to also tweak libvirt-perl's spec so that those packages are only required on Fedora. Since that needs to be fixed in a different repository, though, once you have addressed all of the above you can consider this patch Reviewed-by: Andrea Bolognani <abologna@xxxxxxxxxx> -- Andrea Bolognani / Red Hat / Virtualization -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
