Huge thanks to our Platinum Members Endace and LiveAction,
and our Silver Member Veeam, for supporting the Wireshark Foundation and project.

Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 21195: /trunk/ /trunk/: configure.in

From: Sebastien Tandel <sebastien@xxxxxxxxx>
Date: Mon, 26 Mar 2007 10:37:39 +0200
Note that  '==' is not portable on BSD unix.


Jeff Morriss wrote:
> Thanks & sorry for the trouble.  Unfortunately something was stripping 
> the []'s out of the test in "configure.in" so the resulting "configure" 
> just had "if $? == 0 ;" which didn't work.  I just checked in a new 
> version which should work for everyone.
>
> ronnie sahlberg wrote:
>   
>> I just checked in a different test for configure.in in trunk that works for me.
>>
>> Please review.
>>
>>
>>
>>
>> On 3/26/07, Jeff Morriss <jeff.morriss@xxxxxxxxxxx> wrote:
>>     
>>> Hmmm, that's weird, it works for me.
>>>
>>> What version of GCC are you using?
>>>
>>> What does:
>>>
>>> echo yes | gcc -E -Wfoodeclaration-after-statement -
>>>
>>> produce for you?  For me I get only:
>>>
>>>       
>>>> cc1: error: unrecognized command line option
>>>>         
>>> "-Wfoodeclaration-after-statement"
>>>       
>>>> # 1 "<stdin>"
>>>>         
>>> (but no "yes").
>>>
>>> Oh, OK, it looks like older versions of GCC (I just tried 3.2) don't
>>> error out when presented an invalid command line argument.
>>>
>>> Hmmm, at least it should still compile on those GCC versions, but now I
>>> have to think of a different test...
>>>
>>> ronnie sahlberg 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
>>>>>
>>>>>           
>>>> _______________________________________________
>>>> Wireshark-dev mailing list
>>>> Wireshark-dev@xxxxxxxxxxxxx
>>>> http://www.wireshark.org/mailman/listinfo/wireshark-dev
>>>>
>>>>
>>>>         
>>> _______________________________________________
>>> Wireshark-dev mailing list
>>> Wireshark-dev@xxxxxxxxxxxxx
>>> http://www.wireshark.org/mailman/listinfo/wireshark-dev
>>>
>>>       
>> _______________________________________________
>> Wireshark-dev mailing list
>> Wireshark-dev@xxxxxxxxxxxxx
>> http://www.wireshark.org/mailman/listinfo/wireshark-dev
>>
>>
>>     
> _______________________________________________
> Wireshark-dev mailing list
> Wireshark-dev@xxxxxxxxxxxxx
> http://www.wireshark.org/mailman/listinfo/wireshark-dev
>