Re: F35 Change: Memory Constraints macros for RPM (System-Wide Change proposal)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 8/10/21 8:53 PM, Ankur Sinha wrote:
On Thu, Aug 05, 2021 09:01:14 +0200, Miroslav Suchý wrote:
Dne 05. 08. 21 v 2:42 Michel Alexandre Salim via devel napsal(a):
This is now implemented on Rawhide; Fedora updates are in testing:

Where is it documented?

I suggest one of these

https://rpm-packaging-guide.github.io/
https://docs.fedoraproject.org/en-US/packaging-guidelines/

That would be great. I was trying to put %limit_build at the top of my
spec and kept getting errors. Then I looked at the mcrouter spec and
realised this needs to go into the %build section XD

Yes, the macro is a strange as it mixes rpm macro language and shell language, relying on some rather subtle aspects of how spec parsing and macros work and has to be placed in a shell section. If you ask me, it's asking for trouble.

Setting RPM_BUILD_NCPUS environment should achieve the same without requiring the twisty %global override which looks like it may break _smp_build_ncpus use in later sections (but I may be missing something there)

	- Panu -
_______________________________________________
devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx
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@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Users]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]

  Powered by Linux