Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 21195: /trunk/ /trunk/: configure.in
The problem here is the line
if test xyes != x && echo yes | gcc -E -Wdeclaration-after-statement -
2>&1 | grep yes
/dev/null ; then
at lest the versions of gcc that do not provide that -W do not check
the -W arguments when compiling with -E
thus making the test pointless.
if test xyes != x && echo yes | gcc -E -Wi-just-made-this-warning-up -
2>&1 | grep yes
/dev/null ; then
also successfully passes through older gcc and enters the then-arm.
On 3/26/07, ronnie sahlberg <ronniesahlberg@xxxxxxxxx> wrote:
The checks in configure.in
for this doesnt work properly for
GCC versions which do not support this -W directive.
Eventhough it is not supported by GCC
AC_MSG_CHECKING(to see if we can add '-Wdeclaration-after-statement'...
still adds it to the compile flags.
I will see if i can figure out why the test fails.
Me and automake/configure are not on friendly terms.
On 3/26/07, morriss@xxxxxxxxxxxxx <morriss@xxxxxxxxxxxxx> wrote:
> http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=21195
>
> User: morriss
> Date: 2007/03/26 12:32 AM
>
> Log:
> If we're using gcc, try to use -Wdeclaration-after-statement to catch
more
> non-portable commits. I'm not sure if this is the Right Way to test to
see
> if the compiler can handle a specific option but it's simple and efficient
> enough.
>
> Directory: /trunk/
> Changes Path Action
> +8 -0 configure.in Modified
>
> _______________________________________________
> Wireshark-commits mailing list
> Wireshark-commits@xxxxxxxxxxxxx
> http://www.wireshark.org/mailman/listinfo/wireshark-commits
>
