Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 21195: /trunk/ /trunk/: configure.in
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
>