diff options
| author | Peter Wu <peter@lekensteyn.nl> | 2015-11-26 11:26:13 +0100 |
|---|---|---|
| committer | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2015-11-26 13:02:42 +0000 |
| commit | 470c745a7a0a5b930bdfb0edd8874436cb9beea0 (patch) | |
| tree | 8dbb5e741f8143705a5d8f55f769055756d5f647 /wireshark-gtk.desktop | |
| parent | a198f47c06723fabef174cb3983b26ead24f5bc9 (diff) | |
autotools: do not apply CFLAGS in reverse order
Fixes false "-Wvariadic-macros" because "-Wpedantic" came after
"-Wno-variadic-macros". While at it, avoid (unintentionally?) adding
-fPIE to all C++ programs (via CXXFLAGS).
Availability of flags is checked by appending a flag (in case the user
has something like CFLAGS=-Wno-error).
This removes a -fPIE check for CXXFLAGS and removes 5
-Woverlength-strings and 9 -Wvariadic-macros warnings from make with
Clang 3.7.0. configure and compile times were equal. (A diff between the
configure outputs showed no other changes.)
This reverts commit cf0d762d7304aa569ea25faf999c74bbe94f9023 and applies
a different approach.
Ping-Bug: 10791
Change-Id: Ic7b4137e2d98b06bc7625091be9bc7dd69182586
Reviewed-on: https://code.wireshark.org/review/12175
Petri-Dish: Peter Wu <peter@lekensteyn.nl>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
Diffstat (limited to 'wireshark-gtk.desktop')
0 files changed, 0 insertions, 0 deletions