diff options
author | Christian Lohmaier <lohmaier+LibreOffice@googlemail.com> | 2024-06-04 14:32:08 +0200 |
---|---|---|
committer | Christian Lohmaier <lohmaier+LibreOffice@googlemail.com> | 2024-06-04 16:43:56 +0200 |
commit | da4259bd7fef35185418cad7a695268add6341a2 (patch) | |
tree | 525bb9cb265f7c02157e94941fd9bcacf9ab0f5b | |
parent | 21d7936b320e084a08ef70211a3a51106503ab98 (diff) |
windows packaging: reduce parallelism further to three
Some jenkins daily builders still run into issues from time to time, and
even at a reduced parallelism of three it doesn't affect overall
buildtime when building with just two or three languages and also not
when building with all languages. There might be a number of languages
where this has a penalty, but probably not a significant one.
Change-Id: I778b5fe21fe8f9bb1fcca1da3c4c0eeeda6c29d8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/168403
Tested-by: Jenkins
Reviewed-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>
-rw-r--r-- | solenv/bin/job-limiter.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/solenv/bin/job-limiter.cpp b/solenv/bin/job-limiter.cpp index 9111ead35127..ee1bd8e6edab 100644 --- a/solenv/bin/job-limiter.cpp +++ b/solenv/bin/job-limiter.cpp @@ -79,12 +79,12 @@ int wmain(int argc, wchar_t* argv[]) // system. So when it creates a new one to use, and then releases one it would hit the max // limit otherwise. This only happens when nothing else is waiting for a slot anymore, // i.e. when there are already fewer jobs than imposed by the limiter. - // A limit of four (main installer and 3 controlled by this limiter) was chosen because that + // A limit of three (main installer + 2 controlled by this limiter) was chosen because that // won't affect overall build time (creating the main installer with multiple languages // takes much longer than all the helppackages and the single sdk package combined, even // when those are limited to three jobs), and seems to be low enough to avoid the random // cscript/WiLangId.vbs failures. - semaphore_handle = CreateSemaphoreW(NULL, 3, 4, semaphorename); + semaphore_handle = CreateSemaphoreW(NULL, 2, 3, semaphorename); // keep this process alive for other jobs to grab the semaphore, otherwise it is gone too // quickly and everything creates their own semaphore that immediately has enough slots, // completely bypassing the point of having a limiter... |