[Libreoffice-commits] core.git: configure.ac

Luboš Luňák (via logerrit) logerrit at kemper.freedesktop.org
Wed Mar 24 20:11:13 UTC 2021


 configure.ac |   11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

New commits:
commit 0f02651d693b131060595313db31c7b4b8dad528
Author:     Luboš Luňák <l.lunak at collabora.com>
AuthorDate: Tue Mar 23 15:52:31 2021 +0100
Commit:     Luboš Luňák <l.lunak at collabora.com>
CommitDate: Wed Mar 24 21:10:20 2021 +0100

    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 at collabora.com>

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


More information about the Libreoffice-commits mailing list