[SDK] Clean up symbol visibility flags and compiler vendor handling

Signed-off-by: Stefan Knoblich <stkn@openisdn.net>

diff --git a/src/mod/sdk/autotools/configure.ac b/src/mod/sdk/autotools/configure.ac
index d4f29f492b10a5ac9fe059dcbc85ee22cf8dbde4..54fe529d9ab8a2600991f77bc790093759c2a677 100644 (file)
--- a/src/mod/sdk/autotools/configure.ac
+++ b/src/mod/sdk/autotools/configure.ac
@@ -22,35 +22,42 @@ m4_ifdef([AM_SILENT_RULES], [AM_SILENT_RULES([yes])])
 # Checks for programs.
 AC_PROG_CC
 AM_PROG_CC_C_O
+AC_PROG_CC_C99
 AC_PROG_LIBTOOL
 
 # pkgconfig
 AC_PATH_PROG([PKG_CONFIG], [pkg-config], ["no"])
-if test "x${PKG_CONFIG}" = "xno"
-then
-       AC_MSG_ERROR([Cannot find pkg-config, make sure it is installed in your PATH])
-fi
+AS_IF([test "x${PKG_CONFIG}" = "xno"],
+       [AC_MSG_ERROR([Cannot find pkg-config, make sure it is installed in your PATH])]
+)
 PKG_PROG_PKG_CONFIG
 
-# Checks for cflags
+
+##
+## Checks for cflags
+##
 AC_MSG_RESULT([${as_nl}<<>> Compiler vendor and features])
 
+AX_COMPILER_VENDOR
+
+AM_CONDITIONAL([CC_VENDOR_GNU], [test "x${ax_cv_c_compiler_vendor}" = "xgnu"])
+AM_CONDITIONAL([CC_VENDOR_SUN], [test "x${ax_cv_c_compiler_vendor}" = "xsun"])
+
+
 ##
 ## Compiler vendor and flag checks
 ##
 HAVE_VISIBILITY="no"
+
 AC_ARG_ENABLE([visibility],
        [AS_HELP_STRING([--disable-visibility], [Disable symbol visibility support (default: enabled, if available)])],
        [case "${enableval}" in
-        yes) enable_visibility="yes" ;;
-        no)  enable_visibility="no" ;;
-        *)   AC_MSG_ERROR([Invalid value ${enableval} for parameter --disable-visibility]) ;;
+        yes|no) enable_visibility="${enableval}" ;;
+        *)      AC_MSG_ERROR([Invalid value ${enableval} for parameter --disable-visibility]) ;;
         esac],
        [enable_visibility="yes"]
 )
 
-AX_COMPILER_VENDOR
-
 case "${ax_cv_c_compiler_vendor}" in
 gnu)
        AC_MSG_CHECKING([whether the compiler supports -fvisibility=hidden])
@@ -63,36 +70,16 @@ gnu)
                                [;]
                        )],
 
-                       [AC_MSG_RESULT([yes])
-                        AC_DEFINE([HAVE_VISIBILITY],[1],[GCC visibility support])
-                        HAVE_VISIBILITY="yes"],
-
-                       [AC_MSG_RESULT([no])
-                        HAVE_VISIBILITY="no"]
+                       [HAVE_VISIBILITY="yes"
+                        AC_DEFINE([HAVE_VISIBILITY], [1], [Symbol visibility support])
+                        AC_SUBST([VISIBILITY_CFLAGS],   [-fvisibility=hidden])
+                        AC_SUBST([VISIBILITY_CXXFLAGS], [-fvisibility=hidden])],
+                       [HAVE_VISIBILITY="no"]
                 )
+                AC_MSG_RESULT([${HAVE_VISIBILITY}])
                 CFLAGS="${save_CFLAGS}"],
                [AC_MSG_RESULT([disabled by user])]
        )
-
-       AS_IF([test "x${HAVE_VISIBILITY}" != "xno"],
-               [save_CFLAGS="${CFLAGS}"
-                CFLAGS="${CFLAGS} -fvisibility-inlines-hidden"
-                AC_MSG_CHECKING([whether the compiler supports -fvisibility-inlines-hidden])
-                AC_COMPILE_IFELSE(
-                       [AC_LANG_PROGRAM(
-                               [;], [;]
-                       )],
-
-                       [AC_MSG_RESULT([yes])
-                        HAVE_VISIBILITY_INLINES_HIDDEN="yes"],
-
-                       [AC_MSG_RESULT([no])
-                        HAVE_VISIBILITY_INLINES_HIDDEN="no"]
-                )
-                CFLAGS="${save_CFLAGS}"],
-               [:]
-       )
-       AC_DEFINE([COMPILER_GCC], [1], [Compiler is GCC])
        ;;
 sun)
        AC_MSG_CHECKING([whether the compiler supports -xldscope=hidden])
@@ -105,28 +92,24 @@ sun)
                                [;]
                        )],
 
-                       [AC_MSG_RESULT([yes])
-                        AC_DEFINE([HAVE_VISIBILITY],[1],[SUNCC visibility support])
-                        HAVE_VISIBILITY="yes"],
-
-                       [AC_MSG_RESULT([no])
-                        HAVE_VISIBILITY="no"]
+                       [HAVE_VISIBILITY="yes"
+                        AC_DEFINE([HAVE_VISIBILITY], [1], [Symbol visibility support])
+                        AC_SUBST([VISIBILITY_CFLAGS],   [-xldscope=hidden])
+                        AC_SUBST([VISIBILITY_CXXFLAGS], [-xldscope=hidden])],
+                       [HAVE_VISIBILITY="no"]
                 )
+                AC_MSG_RESULT([${HAVE_VISIBILITY}])
                 CFLAGS="${save_CFLAGS}"],
                [AC_MSG_RESULT([disabled by user])]
        )
-       AC_DEFINE([COMPILER_SUNCC], [1], [Compiler is SunCC])
        ;;
 *)
        AC_MSG_WARN([No visibility checks for this compiler defined])
        ;;
 esac
 
-AM_CONDITIONAL([COMPILER_GCC],   [test "x${ax_cv_c_compiler_vendor}" = "xgnu"])
-AM_CONDITIONAL([COMPILER_SUNCC], [test "x${ax_cv_c_compiler_vendor}" = "xsun"])
+AM_CONDITIONAL([CC_HAS_VISIBILITY], [test "x${HAVE_VISIBILITY}" = "xyes"])
 
-AM_CONDITIONAL([HAVE_VISIBILITY], [test "x${HAVE_VISIBILITY}" = "xyes"])
-AM_CONDITIONAL([HAVE_VISIBILITY_INLINES_HIDDEN], [test "x${HAVE_VISIBILITY_INLINES_HIDDEN}" = "xyes"])
 
 ##
 ## pkgconfig based freeswitch detection code
@@ -210,6 +193,7 @@ PKG_CHECK_MODULES([freeswitch], [freeswitch],
        [AC_MSG_ERROR([FreeSWITCH not found])]
 )
 
+
 ##
 ## Add your other dependency checks here
 ##
@@ -221,6 +205,10 @@ AC_MSG_RESULT([${as_nl}<<>> Other dependencies])
 
 # Checks for library functions.
 
+
+##
+## Done, create output files and print summary
+##
 AC_MSG_RESULT([${as_nl}<<>> Create output files])
 AC_CONFIG_FILES([Makefile src/Makefile])
 AC_OUTPUT

diff --git a/src/mod/sdk/autotools/src/Makefile.am b/src/mod/sdk/autotools/src/Makefile.am
index d2bad5bc80a59b8f33d9cc489f3f39f909f389f3..a8a95b8d15e04e96d358ca02043856355d983079 100644 (file)
--- a/src/mod/sdk/autotools/src/Makefile.am
+++ b/src/mod/sdk/autotools/src/Makefile.am
@@ -10,33 +10,27 @@ AM_CPPFLAGS= -I. -I$(includedir)
 AM_LDFLAGS = -L. -L$(libdir) -avoid-version -module -no-undefined -shared
 
 ###
-# GCC specific flags
+# GCC (and compatible) specific flags
 #
-if COMPILER_GCC
+if CC_VENDOR_GNU
 AM_CFLAGS += -Wall
-# symbol visibility support
-if HAVE_VISIBILITY
-AM_CFLAGS  += -fvisibility=hidden
-AM_CXXFLAGS+= -fvisibility=hidden
-AM_CPPFLAGS+= -DSWITCH_API_VISIBILITY=1
-endif
-if HAVE_VISIBILITY_INLINES_HIDDEN
-AM_CXXFLAGS += -fvisibility-inlines-hidden
-endif
 endif
 
 ###
 # SunCC specific flags
 #
-if COMPILER_SUNCC
+if CC_VENDOR_SUN
 AM_CFLAGS +=
+endif
+
+###
 # symbol visibility support
-if HAVE_VISIBILITY
-AM_CFLAGS   += -xldscope=hidden
-AM_CXXFLAGS += -xldscope=hidden
+#
+if CC_HAS_VISIBILITY
+AM_CFLAGS   += $(VISIBILITY_CFLAGS)
+AM_CXXFLAGS += $(VISIBILITY_CXXFLAGS)
 AM_CPPFLAGS += -DSWITCH_API_VISIBILITY=1
 endif
-endif
 
 
 ###