Fix CFLAGS_NOOPT sed.

View: New views
1 Messages — Rating Filter:   Alert me  

Fix CFLAGS_NOOPT sed.

by Ryan Hill-3 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

configure currently does this:

  CFLAGS_NOOPT=`echo $CFLAGS | sed -e s/-O2// | sed -e s/-O//`

This breaks the build when -O1, -O3, or -Os is used, and doesn't handle the
case where -O? appears more than once.


--
fonts,                             Character is what you are in the dark.
gcc-porting,
wxwidgets @ gentoo     EFFD 380E 047A 4B51 D2BD C64F 8AA8 8346 F9A4 0662


diff --git a/configure.dynamic.in b/configure.dynamic.in
index 6998825..21aeb3f 100644
--- a/configure.dynamic.in
+++ b/configure.dynamic.in
@@ -1023,7 +1023,7 @@ if test "$gww_archlist" != "" ; then
   LDFLAGS="$LDFLAGS $gww_archlist"
 fi
 AC_SUBST(CFLAGS_NOOPT)
-CFLAGS_NOOPT=`echo $CFLAGS | sed -e s/-O2// | sed -e s/-O//`
+CFLAGS_NOOPT=`echo $CFLAGS | sed -e s/-O.//g | sed -e s/-O//g`
 
 cp $ac_confdir/Makefile.dynamic.in $ac_confdir/Makefile.in
 cp $ac_confdir/Unicode/Makefile.dynamic.in $ac_confdir/Unicode/Makefile.in
diff --git a/configure.in b/configure.in
index 6998825..21aeb3f 100644
--- a/configure.in
+++ b/configure.in
@@ -1023,7 +1023,7 @@ if test "$gww_archlist" != "" ; then
   LDFLAGS="$LDFLAGS $gww_archlist"
 fi
 AC_SUBST(CFLAGS_NOOPT)
-CFLAGS_NOOPT=`echo $CFLAGS | sed -e s/-O2// | sed -e s/-O//`
+CFLAGS_NOOPT=`echo $CFLAGS | sed -e s/-O.//g | sed -e s/-O//g`
 
 cp $ac_confdir/Makefile.dynamic.in $ac_confdir/Makefile.in
 cp $ac_confdir/Unicode/Makefile.dynamic.in $ac_confdir/Unicode/Makefile.in



------------------------------------------------------------------------------
Come build with us! The BlackBerry(R) Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay
ahead of the curve. Join us from November 9 - 12, 2009. Register now!
http://p.sf.net/sfu/devconference
_______________________________________________
Fontforge-devel mailing list
Fontforge-devel@...
https://lists.sourceforge.net/lists/listinfo/fontforge-devel

signature.asc (205 bytes) Download Attachment