View Issue Details [ Jump to Notes ] | [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0012119 | CMake | Modules | public | 2011-04-21 12:13 | 2012-07-09 06:52 | ||||
Reporter | Matthias Kretz | ||||||||
Assigned To | Rolf Eike Beer | ||||||||
Priority | normal | Severity | feature | Reproducibility | N/A | ||||
Status | closed | Resolution | fixed | ||||||
Platform | OS | Linux | OS Version | ||||||
Product Version | CMake-2-8 | ||||||||
Target Version | CMake 2.8.8 | Fixed in Version | CMake 2.8.8 | ||||||
Summary | 0012119: improve C/C++ compiler flag checks for Open64 and C/C++ incompatible GCC flags | ||||||||
Description | check_cxx_compiler_flag and check_c_compiler_flag fail to work for Open64 and fail to recognize a specific answer from GCC. Improved version attached. | ||||||||
Tags | No tags attached. | ||||||||
Attached Files | CheckCCompilerFlag.cmake [^] (1,917 bytes) 2011-04-21 12:13 CheckCXXCompilerFlag.cmake [^] (1,934 bytes) 2011-04-21 12:14 | ||||||||
Relationships | |
Relationships |
Notes | |
(0028556) Rolf Eike Beer (developer) 2012-02-11 11:58 |
This should be fixed in current CMake (2.8.7 or next branch) already. Can you retest? |
(0028559) Matthias Kretz (reporter) 2012-02-11 15:32 |
I just looked at the two files in gitweb and it looks like 1/4 is done. I.e. flags to gcc that are not valid for C are detected. Open64's "WARNING: unknown flag:" and the check for g++, whether the flag is invalid for C++, are still missing. If I'm overlooking something let me know and I will test it. |
(0028561) Rolf Eike Beer (developer) 2012-02-12 04:48 |
Fix pushed to next: http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=9b4e4c92aeae15f58438e3bb9846242daafcfc9c [^] I've slightly modified the g++ line to match the one used for gcc. Please attach a patch next time, this makes processing much easier. |
(0029971) David Cole (manager) 2012-07-09 06:52 |
Closing resolved issues that have not been updated in more than 4 months. |
Notes |
Issue History | |||
Date Modified | Username | Field | Change |
2011-04-21 12:13 | Matthias Kretz | New Issue | |
2011-04-21 12:13 | Matthias Kretz | File Added: CheckCCompilerFlag.cmake | |
2011-04-21 12:14 | Matthias Kretz | File Added: CheckCXXCompilerFlag.cmake | |
2012-02-11 11:58 | Rolf Eike Beer | Note Added: 0028556 | |
2012-02-11 15:32 | Matthias Kretz | Note Added: 0028559 | |
2012-02-12 04:48 | Rolf Eike Beer | Note Added: 0028561 | |
2012-02-12 04:48 | Rolf Eike Beer | Assigned To | => Rolf Eike Beer |
2012-02-12 04:48 | Rolf Eike Beer | Status | new => resolved |
2012-02-12 04:48 | Rolf Eike Beer | Resolution | open => fixed |
2012-02-12 04:48 | Rolf Eike Beer | Category | CMake => Modules |
2012-02-12 04:48 | Rolf Eike Beer | Fixed in Version | => CMake 2.8.8 |
2012-02-12 04:48 | Rolf Eike Beer | Target Version | => CMake 2.8.8 |
2012-07-09 06:52 | David Cole | Note Added: 0029971 | |
2012-07-09 06:52 | David Cole | Status | resolved => closed |
Issue History |
Copyright © 2000 - 2018 MantisBT Team |