-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/9394/
-----------------------------------------------------------

Review request for qpid, Andrew Stitcher, Kenneth Giusti, Keith Wall, and 
Rafael Schloming.


Description
-------

test gcc version and set flags based on that.  Code stolen from the large 
qpid/cpp/src/CMakeLists.txt file.  I poked around the gcc docs and determined 
that they all showed up at the same time in 4.3.0.

Please comment if you find the warning message too scary or too pip-squeaky. 

I tried to use CMake's CHECK_C_COMPILER_FLAG() to test each flag and add it 
separately (oh, so elegant), but it didn't work at all, and the function 
definition changtes a lot between 2.6 and 2.8.  So I opted for the simpler 
solution you see here.


This addresses bug PROTON-221.
    https://issues.apache.org/jira/browse/PROTON-221


Diffs
-----

  http://svn.apache.org/repos/asf/qpid/proton/trunk/proton-c/CMakeLists.txt 
1444335 

Diff: https://reviews.apache.org/r/9394/diff/


Testing
-------

Just on rhel6.  I clearly need a test rhel5 VM to augment my Windows and 
Solaris VMs.


Thanks,

Cliff Jansen

Reply via email to