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

2007-03-26 Thread Sebastien Tandel
Me and automake/configure are not on friendly terms. Is there someone who is on friendly terms with them? ;) ___ Wireshark-dev mailing list Wireshark-dev@wireshark.org http://www.wireshark.org/mailman/listinfo/wireshark-dev

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

2007-03-26 Thread Jeff Morriss
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

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

2007-03-26 Thread Sebastien Tandel
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

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

2007-03-26 Thread Guy Harris
Sebastien Tandel wrote: Note that '==' is not portable on BSD unix. For the test/[ command, the standard string-comparison operator is =, rather than ==. (There's also -eq for a numerical comparison.) ___ Wireshark-dev mailing list

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

2007-03-25 Thread Stephen Fisher
We probably want to add this to wiretap/configure.in also.. On Mon, Mar 26, 2007 at 12:32:15AM +, [EMAIL PROTECTED] wrote: http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=revrevision=21195 User: morriss Date: 2007/03/26 12:32 AM Log: If we're using gcc, try to use

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

2007-03-25 Thread Jeff Morriss
Oops, forgot to do that, thanks for the reminder. (It's done now.) Stephen Fisher wrote: We probably want to add this to wiretap/configure.in also.. On Mon, Mar 26, 2007 at 12:32:15AM +, [EMAIL PROTECTED] wrote: http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=revrevision=21195

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

2007-03-25 Thread ronnie sahlberg
[EMAIL PROTECTED] trunk]$ echo yes | gcc -E -Wfoodeclaration-after-statement - # 1 yes [EMAIL PROTECTED] trunk]$ gcc -v Reading specs from /usr/lib/gcc-lib/i386-redhat-linux/2.96/specs gcc version 2.96 2731 (Red Hat Linux 7.1 2.96-81) [EMAIL PROTECTED] trunk]$ It appears it only looks at the

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

2007-03-25 Thread ronnie sahlberg
Dont worry. Ill rewrite it so that it tries to compile a real file. On 3/26/07, ronnie sahlberg [EMAIL PROTECTED] wrote: [EMAIL PROTECTED] trunk]$ echo yes | gcc -E -Wfoodeclaration-after-statement - # 1 yes [EMAIL PROTECTED] trunk]$ gcc -v Reading specs from

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

2007-03-25 Thread Stephen Fisher
This add-on macro claims to do what you need: ax_cflags_gcc_option Check whether GCC supports the given option. http://autoconf-archive.cryp.to/ax_cflags_gcc_option.html On Mon, Mar 26, 2007 at 06:32:32AM +, ronnie sahlberg wrote: Dont worry. Ill rewrite it so that it tries to

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

2007-03-25 Thread ronnie sahlberg
I just checked in a different test for configure.in in trunk that works for me. Please review. On 3/26/07, Jeff Morriss [EMAIL PROTECTED] 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 -