Hi all,
On Wed, Aug 11, 2021 at 10:56:39AM +0300, Panu Matilainen wrote:
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)
I'll revise, thanks! The documentation is forthcoming, personal life was
getting in the way over the past few weeks. Hopefully using
RPM_BUILD_NCPUS will obviate having to declare this in `%build`
--
Michel Alexandre Salim
profile:
https://keyoxide.org/michel@michel-slm.name