[MPlayer-dev-eng] -z is not a portable linker option

[Reimar Döffinger]

Hi,
On Wed, Apr 20, 2005 at 11:53:39PM -0400, Rich Felker wrote:
> I've reversed it for now. It should be detected by configure if this
> nonsense is needed for anything, but I don't understand why it should
> be needed anyway...

More like nice-to-have esp. together with SELinux. Any comments on the
attached patch? It makes gcc spit out a lot of warnings during
compiling, so probably not quite perfect yet :-(

Greetings,
Reimar Döffinger
-------------- next part --------------
Index: configure
===================================================================
RCS file: /cvsroot/mplayer/main/configure,v
retrieving revision 1.979
diff -u -r1.979 configure
--- configure	17 Apr 2005 13:04:28 -0000	1.979
+++ configure	21 Apr 2005 17:46:47 -0000
@@ -6552,6 +6575,17 @@
   echores "only used for x86"
 fi
 
+echocheck "compiler support for noexecstack"
+cat > $TMPC <<EOF
+int main(void) { return 0; }
+EOF
+if cc_check -Wl,-z,noexecstack ; then
+  CFLAGS="-Wl,-z,noexecstack $CFLAGS"
+  echores "yes"
+else
+  echores "no"
+fi
+
 echocheck "ftello()"
 # if we don't have ftello use the osdep/ compatibility module
 cat > $TMPC << EOF