User Agent: Mozilla/5.0 (Ubuntu; X11; Linux i686; rv:9.0.1) Gecko/20100101 Firefox/9.0.1 Build ID: 20111221215619 Steps to reproduce: Compiling firefox nightly code with debug mode enabled (ac_add_options --enable-debug on mozconfig) gcc-4.3.real (Ubuntu 4.3.5-3ubuntu1) 4.3.5 Ubuntu 10.10 Actual results: /mozilla-central/other-licenses/snappy/src/snappy.cc:504: error: extra ‘;’ mozilla-central/other-licenses/snappy/src/snappy.cc:590: error: extra ‘;’ Expected results: Finish the compilation process.
Created attachment 585728 [details] [diff] [review] Patch to this bug.
Comment on attachment 585728 [details] [diff] [review] Patch to this bug. Snappy is a library we pull from google, this issue needs to be reported to them: https://code.google.com/p/snappy/issues/list
Ok. I reported the error on the snappy site: https://code.google.com/p/snappy/issues/detail?id=58
The answer obtained on snappy issue: -- Hi, This is not an error in Snappy; it is not illegal C++, but it probably stops Firefox compile anyway due to -Werror. (I haven't checked that they actually compile with -Werror.) However, these warnings were already reported in bug 57 , and the warning was fixed in r56. -- https://code.google.com/p/snappy/source/detail?r=56
Re-summarizing the bug.
r56 seems to only fix one of the instances touched in Rogerio's patch.
Kyle tells me that we don't compile with -Werror... Are you using special compile flags? Or maybe WARNINGS_AS_ERRORS in your mozconfig?
Rogerio, do you have -Werror in your CFLAGS/CXXFLAGS in your environment? As far as I can tell we shouldn't be building Snappy with -Werror by default.
The content of my mozconfig: ac_add_options --enable-debug Don't have WARNINGS_AS_ERRORS. Environment vars CFLAGS/CXXFLAGS don't have -Werror.
I checked out snappy code on mozilla-central/other-licenses/snappy/src (svn checkout http://snappy.googlecode.com/svn/trunk/ src) The error doesn't occur on compiling process.
Ok, if pulling an updated upstream rev fixes it I don't care to worry about why it's broken.
Please make sure you follow the steps outlined in other-licenses/snappy/README too.
Created attachment 586069 [details] [diff] [review] Update snappy to r56
Comment on attachment 586069 [details] [diff] [review] Update snappy to r56 Looks fine, let's add the revision number and instructions to update it to README also.
This was probably broken because we compile with -pedantic, FWIW. (I've hit the same problem with Breakpad code.)
4 months ago