configure.ac |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 69dd4ce13a34b046b4fe649548e5bb119b5cef6b
Author:     Sam James <s...@gentoo.org>
AuthorDate: Fri Jan 13 15:09:59 2023 +0000
Commit:     Stephan Bergmann <sberg...@redhat.com>
CommitDate: Tue Jan 31 08:09:12 2023 +0000

    configure.ac: Fix build with --with-system-zxing and < Clang 16
    
    Clang 15 and older default to -std=c++14, not -std=c++17 (unlike Clang 16 
onwards).
    
    This doesn't show up with GCC because GCC 11 onwards defaults to -std=c++17.
    
    The new version of libzxing requires C++ 17, per its release notes. Adapt
    the configure check accordingly to pass -std=c++17 when checking for its
    headers and save accordingly if successful.
    
    (Do this via ${CXXFLAGS_CXX11} which has the appropriate switch
    for our compiler, despite its name.)
    
    Bug: https://bugs.gentoo.org/890324
    Change-Id: Iaaa8fdc05eea0e26416b605bfda8f2d831f8729c
    Signed-off-by: Sam James <s...@gentoo.org>
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145479
    Tested-by: Jenkins
    Reviewed-by: Stephan Bergmann <sberg...@redhat.com>
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/146372

diff --git a/configure.ac b/configure.ac
index e626c925f49b..e40051e2c616 100644
--- a/configure.ac
+++ b/configure.ac
@@ -11178,7 +11178,7 @@ else
                 continue
             fi
             dnl TODO: White space in $i would cause problems:
-            CXXFLAGS="$save_CXXFLAGS -I$i/ZXing"
+            CXXFLAGS="$save_CXXFLAGS ${CXXFLAGS_CXX11} -I$i/ZXing"
             AC_CHECK_HEADER(MultiFormatWriter.h, [ZXING_CFLAGS=-I$i/ZXing; 
break],
                 [unset ac_cv_header_MultiFormatWriter_h], [#include 
<stdexcept>])
         done

Reply via email to