On Tue, 2014-02-18 at 11:04 -0500, Alan Conway wrote: > On Sat, 2014-02-15 at 16:55 +0000, Fraser Adams wrote: > > Hello all, > > I've just done an svn update to r1568639 and tried to build qpid cpp > > > > I did > > cmake -DCMAKE_BUILD_TYPE=Release .. > > and also, just to check > > cmake .. > > > > both fail for me using gcc 4.6.1 > > > > I got as far as below - seems to be a problem with ha_test_max_queues.cpp > > > > > > Scanning dependencies of target ha_test_max_queues > > [ 80%] Building CXX object > > src/tests/CMakeFiles/ha_test_max_queues.dir/ha_test_max_queues.cpp.o > > /home/fadams/qpid/qpid-trunk/qpid/cpp/src/tests/ha_test_max_queues.cpp:32:5: > > > > error: unused parameter ‘argc’ [-Werror=unused-parameter] > > cc1plus: all warnings being treated as errors > > > > make[2]: *** > > [src/tests/CMakeFiles/ha_test_max_queues.dir/ha_test_max_queues.cpp.o] > > Error 1 > > make[1]: *** [src/tests/CMakeFiles/ha_test_max_queues.dir/all] Error 2 > > make: *** [all] Error 2 > > My bad! That would be a release build right? Stupid assert(). Fixing > now...
Fixed. ------------------------------------------------------------------------ r1569400 | aconway | 2014-02-18 11:09:23 -0500 (Tue, 18 Feb 2014) | 4 lines NO-JIRA: Fix build error in ha_test_max_queues for release builds. argc was only being tested in an assert which was compiled out for release builds. ------------------------------------------------------------------------ Of course "stupid assert" is short for "stupid developer who should know better than to use assert like that..." --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
