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

jan Iversen jani at libreoffice.org
Tue Nov 21 08:20:51 UTC 2017


 configure.ac |    7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

New commits:
commit d015a6e4e3ae72c38bfb0516c2229daf2254e3d0
Author: jan Iversen <jani at libreoffice.org>
Date:   Mon Nov 20 16:07:09 2017 +0100

    iOS, removed visibility=hidden
    
    Seems visibility=hidden gives debug problem, with
    templated constructors (at least the debugger did not accept
    breakpoint correctly).
    
    Change-Id: I094dc68ffd06516b31ebc649649fad923b903d75

diff --git a/configure.ac b/configure.ac
index 0d2b8aca01a6..e634474cffb8 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2877,14 +2877,11 @@ if test $_os = iOS; then
     if test "$ENABLE_LTO" = TRUE; then
         lto=-flto
     fi
-    # Just add -fvisibility=hidden to CC and CXX directly so that the 3rd-party libs also
-    # get compiled with it, to avoid ld warnings when linking all that together into one
-    # executable.
 
     stdlib="-stdlib=libc++"
 
-    CC="`xcrun -find clang` -arch $host_cpu -fvisibility=hidden -isysroot $sysroot $lto $versionmin"
-    CXX="`xcrun -find clang++` -arch $host_cpu -fvisibility=hidden $stdlib -isysroot $sysroot $lto $versionmin"
+    CC="`xcrun -find clang` -arch $host_cpu -isysroot $sysroot $lto $versionmin"
+    CXX="`xcrun -find clang++` -arch $host_cpu $stdlib -isysroot $sysroot $lto $versionmin"
 
     INSTALL_NAME_TOOL=`xcrun -find install_name_tool`
     AR=`xcrun -find ar`


More information about the Libreoffice-commits mailing list