Re: Fedora 33 System-Wide Change proposal: Use %make_build and %make_install macros

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

 



On 20. 06. 20 14:47, Andy Mender wrote:
On Sat, 20 Jun 2020 at 00:38, Miro Hrončok <mhroncok@xxxxxxxxxx <mailto:mhroncok@xxxxxxxxxx>> wrote:

    On 19. 06. 20 23:11, Ben Cotton wrote:
     > All make invocations in spec files that don't use the install target will be
     > modified to use the %make_build macro

    Many Python packages build Sphinx documentation with variant of "make html".
    Such invocation will always be just 1 job. Hence there is no benefit of
    parallel
    make. Replacing it with %make_build will only make it harder to read.

    Can we exclude such cases? Or do we want all make invocations to be mecronized,
    even if there is no benefit?


The way I understand it is that %make_build would be a replacement for the "make all" target with N jobs, which is called by default when you just run "make".

The proposal says all make invocations.

That would be the all-in-1 scenario when you want to build the main binaries, docs and examples. If there is only 1 target (html) and you run "make" with N jobs, there is no downside, because only that single target gets picked up.

No downside when building, but downside when reading the spec file.

I really like the proposal and kind of sort of agree with Tomasz that if a build requires -j1 explicitly and fails with -jN, it means the target dependency tree was not defined properly and should be fixed.

That is not the case here.

--
Miro Hrončok
--
Phone: +420777974800
IRC: mhroncok
_______________________________________________
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




[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