compile errors when building against protobuf headers

2009-07-27 Thread achap
anyone else experience compile errors when building against the descriptor.h and wire_format.h files? error messages below: include/google/protobuf/descriptor.h:322: error: comma at end of enumerator list include/google/protobuf/descriptor.h:341: error: comma at end of enumerator list

Re: compile errors when building against protobuf headers

2009-07-27 Thread achap
forgot to mention... I'm using protobuf version 2.1.0 On Jul 27, 6:25 pm, achap achappe...@gmail.com wrote: anyone else experience compile errors when building against the descriptor.h and wire_format.h files? error messages below: include/google/protobuf/descriptor.h:322: error: comma

Re: compile errors when building against protobuf headers

2009-07-27 Thread Kenton Varda
What compiler flags are you using? It looks like you have an excessively pedantic warning mode enabled, with -Werror to make warnings into errors, and it is applying even to system headers (usually warnings in system headers aren't reported). On Mon, Jul 27, 2009 at 6:27 PM, achap

Re: compile errors when building against protobuf headers

2009-07-27 Thread achap
I am not using -Werror, but I am using -W -pedantic, which apparently is the problem. what is the likely hood that this will work with the pedantic warning enabled? My build system forces pendant... I can override it, but I'd rather not if I don't have too. Thanks. On Jul 27, 6:39 pm,

Re: compile errors when building against protobuf headers

2009-07-27 Thread Kenton Varda
Again, GCC will ignore warnings as long as it recognizes the protobuf headers as system headers. You can explicitly mark an include directory as containing system headers by using -isystem instead of -I. On Mon, Jul 27, 2009 at 7:02 PM, achap achappe...@gmail.com wrote: I am not using

Re: compile errors when building against protobuf headers

2009-07-27 Thread achap
oops... yes I forgot about the -isystem option... I will use this for the protobuf headers. I am still a little surprised that protobuf didn't pass the pedantic test. I've use lots of third party headers and haven't ran into this problem before. thanks for the quick response. On Jul 27,