From: keescook <keescook@users.sourceforge.net>
Date: Wed, 30 Apr 2008 06:34:29 +0000 (+0000)
Subject: and check for relro now too, since using gcc doesn't mean we have a GNU ld
X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=fb66491029e5855b66d23df764e5df19445297f8;p=inkscape.git

and check for relro now too, since using gcc doesn't mean we have a GNU ld
---

diff --git a/configure.ac b/configure.ac
index 091ec25ab..822508ef7 100644
--- a/configure.ac
+++ b/configure.ac
@@ -50,8 +50,6 @@ if test "$GCC" = "yes"; then
 
   # Enable all default warnings, and also format and format security warnings
   CFLAGS="-Wall -Wformat -Wformat-security $CFLAGS"
-  # Have linker produce read-only relocations, if it knows how
-  CFLAGS="-Wl,-z,relro $CFLAGS"
   # What is just plain "-W" ?
   # Fortify source requires -O2 or higher, which is handled separately.
   CFLAGS="-W -D_FORTIFY_SOURCE=2 $CFLAGS"
@@ -67,6 +65,17 @@ if test "$GCC" = "yes"; then
     CFLAGS="$ink_svd_CFLAGS"
   fi
 
+  # Have linker produce read-only relocations, if it knows how
+  AC_MSG_CHECKING([gcc support for -Wl,-z,relro])
+  ink_svd_CFLAGS="$CFLAGS"
+  CFLAGS="-Wl,-z,relro $CFLAGS"
+  AC_COMPILE_IFELSE(AC_LANG_PROGRAM([]), [gcc_opt_ok=yes], [gcc_opt_ok=no])
+  AC_MSG_RESULT([$gcc_opt_ok])
+  if test "x$gcc_opt_ok" != "xyes"; then
+    CFLAGS="$ink_svd_CFLAGS"
+  fi
+
+
   AC_LANG_POP
 fi
 
@@ -777,8 +786,6 @@ if test "$GXX" = "yes"; then
 
   # Enable all default warnings, and also format and format security warnings
   CXXFLAGS="-Wall -Wformat -Wformat-security $CXXFLAGS"
-  # Have linker produce read-only relocations, if it knows how
-  CXXFLAGS="-Wl,-z,relro $CXXFLAGS"
   # What is just plain "-W" ?
   # Fortify source requires -O2 or higher, which is handled separately.
   CXXFLAGS="-W -D_FORTIFY_SOURCE=2 $CXXFLAGS"
@@ -790,7 +797,17 @@ if test "$GXX" = "yes"; then
   AC_MSG_CHECKING([g++ support for -Werror=format-security])
   ink_svd_CXXFLAGS="$CXXFLAGS"
   CXXFLAGS="-Werror=format-security $CXXFLAGS"
-  AC_COMPILE_IFELSE(AC_LANG_PROGRAM([]), [gxx_opt_ok=yes], [gcc_opt_ok=no])
+  AC_COMPILE_IFELSE(AC_LANG_PROGRAM([]), [gxx_opt_ok=yes], [gxx_opt_ok=no])
+  AC_MSG_RESULT([$gxx_opt_ok])
+  if test "x$gxx_opt_ok" != "xyes"; then
+    CXXFLAGS="$ink_svd_CXXFLAGS"
+  fi
+
+  # Have linker produce read-only relocations, if it knows how
+  AC_MSG_CHECKING([g++ support for -Wl,-z,relro])
+  ink_svd_CXXFLAGS="$CXXFLAGS"
+  CXXFLAGS="-Wl,-z,relro $CXXFLAGS"
+  AC_COMPILE_IFELSE(AC_LANG_PROGRAM([]), [gxx_opt_ok=yes], [gxx_opt_ok=no])
   AC_MSG_RESULT([$gxx_opt_ok])
   if test "x$gxx_opt_ok" != "xyes"; then
     CXXFLAGS="$ink_svd_CXXFLAGS"