wireshark: fix wrong CFLAGS_FOR_BUILD assumption

Tested valid CFLAGS can't be pushed into CFLAGS_FOR_BUILD, they're
different compilers.

Fixes:
http://autobuild.buildroot.net/results/2b1/2b1a0654b649618422d65edf370b82476ba84491/

Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
This commit is contained in:
Gustavo Zacarias 2013-12-04 14:37:27 -03:00 committed by Peter Korsgaard
parent 408743d29d
commit fd6b999fe5

View File

@ -0,0 +1,40 @@
AC_WIRESHARK_GCC_CFLAGS_CHECK checks for valid CFLAGS against CC and pushes
them into CFLAGS and CFLAGS_FOR_BUILD.
This is wrong since CC_FOR_BUILD might be wildly different and older than CC
leading to build errors.
Upstream status: reported in tracker
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9507
Patch status: Interim to fix:
http://autobuild.buildroot.net/results/2b1/2b1a0654b649618422d65edf370b82476ba84491/
Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
diff -Nura wireshark-1.10.3.orig/acinclude.m4 wireshark-1.10.3/acinclude.m4
--- wireshark-1.10.3.orig/acinclude.m4 2013-12-04 14:06:44.986074426 -0300
+++ wireshark-1.10.3/acinclude.m4 2013-12-04 14:25:36.496356791 -0300
@@ -1664,11 +1664,6 @@
# the saved value plus just the new option.
#
CFLAGS="$CFLAGS_saved $GCC_OPTION"
- #
- # Add it to the flags we use when building
- # build tools.
- #
- CFLAGS_FOR_BUILD="$CFLAGS_FOR_BUILD $GCC_OPTION"
if test "$2" != C ; then
#
# Add it to the C++ flags as well.
@@ -1687,11 +1682,6 @@
# the saved value plus just the new option.
#
CFLAGS="$CFLAGS_saved $GCC_OPTION"
- #
- # Add it to the flags we use when building
- # build tools.
- #
- CFLAGS_FOR_BUILD="$CFLAGS_FOR_BUILD $GCC_OPTION"
if test "$2" != C ; then
#
# Add it to the C++ flags as well.