unix/configure.ac is broken with CFLAGS or CXXFLAGS "-ggdb"

[GoogleCodeExporter]

Could we remove these lines from configure.ac in unix/ ? It's trying to be
a bit too smart... And when you have -ggdb in CFLAGS or CXXFLAGS the sed
strips the -g out of -ggdb and leave you with just "gdb" which isn't a
valid compiler flag... ;-)

imho, setting debugging flags should be left for user anyway

-# Remove -g and -O2 flags manually.
-
-if test "x$CFLAGS" != "x"; then
-       CFLAGS="`echo \"$CFLAGS\" | sed -e 's/-g//'`"
-       CFLAGS="`echo \"$CFLAGS\" | sed -e 's/-O2//'`"
-fi
-
-if test "x$CXXFLAGS" != "x"; then
-       CXXFLAGS="`echo \"$CXXFLAGS\" | sed -e 's/-g//'`"
-       CXXFLAGS="`echo \"$CXXFLAGS\" | sed -e 's/-O2//'`"
-fi
-

Original issue reported on code.google.com by ssuomi...@unk.fi on 14 Mar 2010 at 10:25

[GoogleCodeExporter]

like this,

$ echo CXXFLAGS="-ggdb"
CXXFLAGS=-ggdb
$ echo CXXFLAGS="-ggdb" | sed -e 's/-g//'
CXXFLAGS=gdb
$ 

Original comment by ssuomi...@unk.fi on 14 Mar 2010 at 10:37

[GoogleCodeExporter]

Actually, this is the wrong place for these things. This project is primarily 
for the
GTK+ version and bugs that affect it. zones is the one who's maintaining the 
unix
port. You'll have better luck re-posting this in the public development forum 
in the
boards at snes9x.com.

Original comment by bear...@gmail.com on 15 Mar 2010 at 9:31

• Changed state: Invalid