Re: [concordance-devel] Build failure with latest CVS
On Sun, March 30, 2008 11:41 pm, Phil Dibowitz wrote: And again, you missed what I said about declarations not at the top. This is why we use -ansi -pedantic-errors: [EMAIL PROTECTED] tmp]$ gcc -std=c99 foo.c [EMAIL PROTECTED] tmp]$ gcc -std=c99 -pedantic-errors foo.c [EMAIL PROTECTED] tmp]$ gcc -ansi -pedantic-errors foo.c foo.c: In function 'main': foo.c:9: error: ISO C90 forbids mixed declarations and code [EMAIL PROTECTED] tmp]$ It's the ONLY combination that seems to catch that error. Yes, but: a) There's no reason to care about this, because like I said, I believe we can fix the Windows build issues by building concordance.c as C++ instead of C. b) Those gcc options cause gcc (some versions, e.g. 4.1.x, 4.3.x) to fail to build concordance. Do we really want to prevent compilation on Linux just so it works on Windows? - Check out the new SourceForge.net Marketplace. It's the best place to buy or sell services for just about anything Open Source. http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace ___ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel
Re: [concordance-devel] Build failure with latest CVS
Stephen Warren wrote: * It certainly won't remove the warning about strdup not being prototyped (that's due to -ansi), which is a warning for me, but I believe an error on gcc-4.3 (hence why I submitted the patch for libconcord to add a bunch of headers to help packaging libconcord on Fedora 9 which uses gcc-4.3) This warning is now gone. As soon as I can reproduce the other error your getting, I'll deal with it as well. -- Phil Dibowitz [EMAIL PROTECTED] Open Source software and tech docsInsanity Palace of Metallica http://www.phildev.net/ http://www.ipom.com/ Never write it in C if you can do it in 'awk'; Never do it in 'awk' if 'sed' can handle it; Never use 'sed' when 'tr' can do the job; Never invoke 'tr' when 'cat' is sufficient; Avoid using 'cat' whenever possible -- Taylor's Laws of Programming signature.asc Description: OpenPGP digital signature - Check out the new SourceForge.net Marketplace. It's the best place to buy or sell services for just about anything Open Source. http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace___ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel
Re: [concordance-devel] Build failure with latest CVS
Stephen Warren wrote: Phil Dibowitz wrote: Stephen Warren wrote: * It certainly won't remove the warning about strdup not being prototyped (that's due to -ansi), which is a warning for me, but I believe an error on gcc-4.3 (hence why I submitted the patch for libconcord to add a bunch of headers to help packaging libconcord on Fedora 9 which uses gcc-4.3) This warning is now gone. As soon as I can reproduce the other error your getting, I'll deal with it as well. It appears there is a gcc option -Wno-variadic-macros. I'm cool with that - though I would like to confirm the current source does in fact build in windows before I nuke the warning I don't suppose that Debian somehow sets this as a default (does gcc -dumpspecs show anything relevant?), or do you perhaps have a CFLAGS environment variable set that includes this or something? [EMAIL PROTECTED] tmp]$ gcc -dumpspecs | grep vari [EMAIL PROTECTED] tmp]$ I have no CFLAGS or CPPFLAGS set... It's so weird I don't get the same behavior... -- Phil Dibowitz [EMAIL PROTECTED] Open Source software and tech docsInsanity Palace of Metallica http://www.phildev.net/ http://www.ipom.com/ Never write it in C if you can do it in 'awk'; Never do it in 'awk' if 'sed' can handle it; Never use 'sed' when 'tr' can do the job; Never invoke 'tr' when 'cat' is sufficient; Avoid using 'cat' whenever possible -- Taylor's Laws of Programming signature.asc Description: OpenPGP digital signature - Check out the new SourceForge.net Marketplace. It's the best place to buy or sell services for just about anything Open Source. http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace___ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel
Re: [concordance-devel] Build failure with latest CVS
Stephen Warren wrote: Phil Dibowitz wrote: Stephen Warren wrote: * It certainly won't remove the warning about strdup not being prototyped (that's due to -ansi), which is a warning for me, but I believe an error on gcc-4.3 (hence why I submitted the patch for libconcord to add a bunch of headers to help packaging libconcord on Fedora 9 which uses gcc-4.3) This warning is now gone. As soon as I can reproduce the other error your getting, I'll deal with it as well. It appears there is a gcc option -Wno-variadic-macros. Committed. Still, weird. -- Phil Dibowitz [EMAIL PROTECTED] Open Source software and tech docsInsanity Palace of Metallica http://www.phildev.net/ http://www.ipom.com/ Never write it in C if you can do it in 'awk'; Never do it in 'awk' if 'sed' can handle it; Never use 'sed' when 'tr' can do the job; Never invoke 'tr' when 'cat' is sufficient; Avoid using 'cat' whenever possible -- Taylor's Laws of Programming signature.asc Description: OpenPGP digital signature - Check out the new SourceForge.net Marketplace. It's the best place to buy or sell services for just about anything Open Source. http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace___ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel