On Fri, Nov 29, 2024 at 11:39 AM Daniel P. Berrangé <berra...@redhat.com> wrote: > > On Fri, Nov 29, 2024 at 04:16:51PM +0000, Richard W.M. Jones wrote: > > On Fri, Nov 29, 2024 at 04:02:43PM +0000, Daniel P. Berrangé wrote: > > > On Fri, Nov 29, 2024 at 03:48:02PM +0000, Richard W.M. Jones wrote: > > > > On Fri, Nov 29, 2024 at 04:36:14PM +0100, Florian Weimer wrote: > > > > > * Daniel P. Berrangé: > > > > > > > > > > > I think I'd prefer to stay away from redhat-rpm-config. Not because > > > > > > it > > > > > > is a problem in Fedora, but because it will cause us pain downstream > > > > > > to deal with RHEL development process bureaucracy to commit changes > > > > > > into a package we don't directly own as virt maintainers. > > > > > > > > > > How do you plan to install the new package into the buildroot, so that > > > > > it's there when the SRPM is generated? > > > > > > > > > > (Maybe dynamic build dependencies can be used to solve this without > > > > > buildroot preseeding, but seems a bit convoluted.) > > > > > > > > For better or worse redhat-rpm-config has Requires on a whole bunch of > > > > these *-srpm-macros packages: > > > > > > > > https://src.fedoraproject.org/rpms/redhat-rpm-config/blob/rawhide/f/redhat-rpm-config.spec#_94 > > > > > > > > AIUI that is how the buildroot gets populated. > > > > > > Isn't this only needed for historical compat ? I was anticipating that > > > apps would directly "BuildRequires: qemu-srpm-macros", *not* get it > > > indirectly via redhat-rpm-config > > > > I think there's an ordering problem, for example, you can't expand any > > qemu macros until qemu-srpm-macros is installed, but using > > BuildRequires includes it too late. IIRC anyway. > > Oh, I see, so the ordering problem is something that affects '-srpm-macros' > packages but not '-rpm-macros' packages. > > So if it is something that affects parseability of a spec file it would > be in -srpm-macros, but if expanding it only matters at build time, then > -rpm-macros is sufficient ? >
Yes. And the latter can be set up as a "Requires: (<foo>-rpm-macros if rpm-build)" for a more natural build dependency to pull in at build-time. -- 真実はいつも一つ！/ Always, there's only one truth! -- _______________________________________________ devel mailing list -- devel@lists.fedoraproject.org To unsubscribe send an email to devel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
