On Mon, 2024-06-10 at 18:52 +0200, Vít Ondruch wrote:
> Dne 10. 06. 24 v 17:35 Adam Williamson napsal(a):
> > On Mon, 2024-06-10 at 16:38 +0200, Vít Ondruch wrote:
> > > Dne 10. 06. 24 v 16:24 Daniel P. Berrangé napsal(a):
> > > > On Mon, Jun 10, 2024 at 04:18:14PM +0200, Miroslav Suchý wrote:
> > > > > Lately, I noticed that several SPEC files in Fedora use this syntax:
> > > > > 
> > > > > Source:        macros.vlc
> > > > > 
> > > > > And this file defines macros that are loaded by rpmbuild during 
> > > > > buildtime and are used in the SPEC file.
> > > > > 
> > > > > This makes parsing of the SPEC file harder, because any parser have 
> > > > > to have
> > > > > this maro file in current directory - just reading SPEC file is not 
> > > > > enough.
> > > 
> > > There is also:
> > > 
> > > ~~~
> > > 
> > > %{load:%{S:1}}
> > > 
> > > ~~~
> > > 
> > > 
> > > Which actually loads the file.
> > > 
> > > 
> > > > > I mentioned vlc, but it is used in many other packages: valkey, zig,
> > > > > typelib-srpm-macros, ansible-packaging, rakudo, sip and many many 
> > > > > other.
> > > > > 
> > > > > Why are packagers doing this? I am not saying this is bad, it just 
> > > > > surprised
> > > > > me. I am used to put all macros at the top of the SPEC file and this 
> > > > > is new
> > > > > to me. What is the benefit?
> > > > I don't know if it is the case for vlc, but the common benefit would be
> > > > where the same macros are to be used in both this local spec, and the
> > > > specs of other dependent RPMs.
> > > 
> > > That is precisely the reason I have pioneered this approach and using it
> > > for Ruby.
> > It might be nice to package the macros, in that case.
> 
> 
> They are packaged of course, either in rubygem-devel or in ruby-devel

But then why would spec files be listing them as a source, which is
where this thread started, instead of buildrequiring the appropriate
package?
-- 
Adam Williamson (he/him/his)
Fedora QA
Fedora Chat: @adamwill:fedora.im | Mastodon: @ad...@fosstodon.org
https://www.happyassassin.net



--
_______________________________________________
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

Reply via email to