buildroot
Public
Source
committed d813f707411xxxxxxxxxxDon't do horrible magic with CFLAGS and CXXFLAGSThe purpose of the lines being removed is to suppress the-g<something> option from the CFLAGS or CXXFLAGS, when --enable-debugis not used. This is stupid because it is up to the user to pass theseflags or not.But it is not only stupid, but utterly broken: if your CFLAGS containsa flags like -mfloat-gprs=double, then this option gets truncated to-mfloat simply because the option contained the -g string!Completely ridiculous piece of code, so get rid of it.Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>Index: libhid-0.2.16/m4/md_conf_debugging.m4===================================================================--- libhid-0.2.16.orig/m4/md_conf_debugging.m4 2004-11-08 18:16:25.000000000 +0100+++ libhid-0.2.16/m4/md_conf_debugging.m4 2012-12-12 20:24:50.000000000 +0100@@ -1,7 +1,7 @@ AC_DEFUN([MD_CONF_DEBUGGING], [- CFLAGS=$(echo $CFLAGS | sed -e 's,-g[[^[:space:]]]*,,g')- CXXFLAGS=$(echo $CXXFLAGS | sed -e 's,-g[[^[:space:]]]*,,g')+# CFLAGS=$(echo $CFLAGS | sed -e 's,-g[[^[:space:]]]*,,g')+# CXXFLAGS=$(echo $CXXFLAGS | sed -e 's,-g[[^[:space:]]]*,,g') MD_CHECK_ARG_ENABLE(debug, [ DEBUG_FLAG=-g