From b27571d6882b8436db9d5284a876b54569ea7a32 Mon Sep 17 00:00:00 2001 From: Stephan Bergmann Date: Tue, 11 Aug 2020 12:39:08 +0200 Subject: [PATCH] Enable LTO again It had reportedly been disabled due to failing CppunitTest_sw_apitests in LO 6.4. My recent upstream check with --enable-lto on master towards LO 7.1 found CppunitTest_sw_apitests to work fine, and only found one other failing test in CppunitTest,xmlsecurity_signing, but which is new for LO 7.1 and not yet present in LO 7.0 (see "Database document apparently needs to be closed before it is disposed" and "external/cppunit: Run tests in deterministic order". From a successful scratch build of this commit, it looks like CppunitTest_sw_apitests already works fine in LO 7.0, and that there are no other issue with LTO in LO 7.0. --- libreoffice.spec | 4 ---- 1 file changed, 4 deletions(-) diff --git a/libreoffice.spec b/libreoffice.spec index 5e60a6d..0ea2d85 100644 --- a/libreoffice.spec +++ b/libreoffice.spec @@ -999,10 +999,6 @@ mv .git .git-rpm # path to external tarballs EXTSRCDIR=`dirname %{SOURCE0}` -# This package fails its testsuite when LTO is enabled. -# see: https://bugs.documentfoundation.org/show_bug.cgi?id=126442 -%define _lto_cflags %{nil} - #use the RPM_OPT_FLAGS but remove the LibreOffice overridden ones for i in $RPM_OPT_FLAGS; do case "$i" in