diff options
| author | Luboš Luňák <l.lunak@collabora.com> | 2021-03-23 15:52:31 +0100 |
|---|---|---|
| committer | Luboš Luňák <l.lunak@collabora.com> | 2021-03-24 21:10:20 +0100 |
| commit | 0f02651d693b131060595313db31c7b4b8dad528 (patch) | |
| tree | cb27ca46ef527cb7acf9373e5838e818eddda319 | |
| parent | b9eca45f56bbb8bf0acf25469e9f5a9918584e4e (diff) | |
use --enable-ooenv only if --enable-debug/dbgutil
I'm not sure what the original purpose of the ooenv script was,
but now it contains only debugging settings such as malloc debugging,
which has a noticeable overhead (8.2s->10.1s in my random case).
At least openSUSE appears to not actually package the script, but
it still doesn't make sense to use this script in non-debug builds.
Change-Id: I4518bb1680a543ed520399c11c83dd6dc5539f71
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/112999
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
| -rw-r--r-- | configure.ac | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/configure.ac b/configure.ac index 7cc376d9b242..9a96e56ab8e1 100644 --- a/configure.ac +++ b/configure.ac @@ -1493,8 +1493,8 @@ libo_FUZZ_ARG_ENABLE(compiler-plugins-analyzer-pch, relevant in the --disable-compiler-plugins case.])) libo_FUZZ_ARG_ENABLE(ooenv, - AS_HELP_STRING([--disable-ooenv], - [Disable ooenv for the instdir installation.])) + AS_HELP_STRING([--enable-ooenv], + [Enable ooenv for the instdir installation.])) AC_ARG_ENABLE(lto, AS_HELP_STRING([--enable-lto], @@ -5427,6 +5427,13 @@ AC_SUBST(ASSERT_ALWAYS_ABORT) # =================================================================== if test $_os != "WINNT" -a $_os != "Darwin"; then AC_MSG_CHECKING([whether to use ooenv for the instdir installation]) + if test -z "$enable_ooenv"; then + if test -n "$ENABLE_DEBUG$ENABLE_DBGUTIL"; then + enable_ooenv=yes + else + enable_ooenv=no + fi + fi if test "$enable_ooenv" = "no"; then AC_MSG_RESULT([no]) else |