Hi Alan,
It also failed when I did cmake .. though I now can't remember if I
remembered to clear the cache after I first did
cmake -DCMAKE_BUILD_TYPE=Release ..
:->
So it might well just relate to the Release build, looking at the code
that'd make more sense. Thanks for fixing I'll give it another whirl
Frase
On 18/02/14 16:11, Alan Conway wrote:
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
