diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2020-09-10 22:28:28 +0200 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2020-09-11 09:08:57 +0200 |
| commit | 51f4691e7f4f3bceac1d5ec9cbbf37de7e71e471 (patch) | |
| tree | 82a32f62b1cec15c5da9c8dcb60cfb2cd2101e7f /solenv/gbuild/CppunitTest.mk | |
| parent | b3e4ba69b70a30bbcc32ac59ce90e2eda2bc76de (diff) | |
Use { ... } for grouping that does not need a sub-shell
...in case that makes ever so small a difference, esp. on Windows (where
spawning processes is known to be expensinve; but note that at least Bash seems
to not spawn a sub-shell anyway when what is enclosed in ( ... ) is just a
simple single process, which is commonly the case for these cppunittester
invocations)
Change-Id: Ie9200270743754e02fbfdb2b17239e9873031dab
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/102407
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'solenv/gbuild/CppunitTest.mk')
| -rw-r--r-- | solenv/gbuild/CppunitTest.mk | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/solenv/gbuild/CppunitTest.mk b/solenv/gbuild/CppunitTest.mk index a2243dad3e31..a287fc513aa3 100644 --- a/solenv/gbuild/CppunitTest.mk +++ b/solenv/gbuild/CppunitTest.mk @@ -123,7 +123,7 @@ else $(if $(gb_CppunitTest__interactive),, \ $(if $(value gb_CppunitTest_postprocess), \ rm -fr $@.core && mkdir $@.core && cd $@.core &&)) \ - ( \ + { \ $(if $(gb_CppunitTest_localized),for l in $(WITH_LANG_LIST) ; do LO_TEST_LOCALE="$$l" ) \ $(if $(gb_CppunitTest_PREGDBTRACE),$(gb_CppunitTest_PREGDBTRACE) &&) \ $(if $(gb_CppunitTest__vcl_no_svp), \ @@ -141,7 +141,7 @@ else $(if $(gb_CppunitTest_POSTGDBTRACE), \ ; RET=$$? && $(gb_CppunitTest_POSTGDBTRACE) && (exit $$RET)) \ $(if $(gb_CppunitTest_localized),|| exit $$?; done) \ - ) \ + ; } \ $(if $(gb_CppunitTest__interactive),, \ > $@.log 2>&1 \ || ($(if $(value gb_CppunitTest_postprocess), \ |