MantisBT - CMake
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0012119CMakeModulespublic2011-04-21 12:132012-07-09 06:52
ReporterMatthias Kretz 
Assigned ToRolf Eike Beer 
PrioritynormalSeverityfeatureReproducibilityN/A
StatusclosedResolutionfixed 
PlatformOSLinuxOS Version
Product VersionCMake-2-8 
Target VersionCMake 2.8.8Fixed in VersionCMake 2.8.8 
Summary0012119: improve C/C++ compiler flag checks for Open64 and C/C++ incompatible GCC flags
Descriptioncheck_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.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Files? CheckCCompilerFlag.cmake (1,917) 2011-04-21 12:13
https://public.kitware.com/Bug/file/3829/CheckCCompilerFlag.cmake
? CheckCXXCompilerFlag.cmake (1,934) 2011-04-21 12:14
https://public.kitware.com/Bug/file/3830/CheckCXXCompilerFlag.cmake
Issue History
Date ModifiedUsernameFieldChange
2011-04-21 12:13Matthias KretzNew Issue
2011-04-21 12:13Matthias KretzFile Added: CheckCCompilerFlag.cmake
2011-04-21 12:14Matthias KretzFile Added: CheckCXXCompilerFlag.cmake
2012-02-11 11:58Rolf Eike BeerNote Added: 0028556
2012-02-11 15:32Matthias KretzNote Added: 0028559
2012-02-12 04:48Rolf Eike BeerNote Added: 0028561
2012-02-12 04:48Rolf Eike BeerAssigned To => Rolf Eike Beer
2012-02-12 04:48Rolf Eike BeerStatusnew => resolved
2012-02-12 04:48Rolf Eike BeerResolutionopen => fixed
2012-02-12 04:48Rolf Eike BeerCategoryCMake => Modules
2012-02-12 04:48Rolf Eike BeerFixed in Version => CMake 2.8.8
2012-02-12 04:48Rolf Eike BeerTarget Version => CMake 2.8.8
2012-07-09 06:52David ColeNote Added: 0029971
2012-07-09 06:52David ColeStatusresolved => closed

Notes
(0028556)
Rolf Eike Beer   
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   
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   
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   
2012-07-09 06:52   
Closing resolved issues that have not been updated in more than 4 months.