[Rpm-maint] [rpm-software-management/rpm] Fix brp-strip-static-archive parallelism (#690)

Florian Festi notifications at github.com
Fri May 3 11:24:59 UTC 2019


The change made in fc2c986 can break for large values of %_smp_build_ncpus as
this many processes are able to overflow the following pipe.

Thanks to Denys Vlasenko for testing this.

This change solves this problem by running a whole processing pileline for each
parallel (file) process. This has also the benefit of running at least some
stip commands in parallel.

The -n param fro xargs was increased to 32 to further reduce the over head of
spawing the helpers as they are now needed for each run of the file command.
You can view, comment on, or merge this pull request online at:

  https://github.com/rpm-software-management/rpm/pull/690

-- Commit Summary --

  * Fix brp-strip-static-archive parallelism

-- File Changes --

    M scripts/brp-strip-static-archive (10)

-- Patch Links --

https://github.com/rpm-software-management/rpm/pull/690.patch
https://github.com/rpm-software-management/rpm/pull/690.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/690
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20190503/287f0c0b/attachment.html>


More information about the Rpm-maint mailing list