MantisBT - CMake
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0012394CMakeModulespublic2011-08-08 11:562012-02-06 06:07
ReporterGiuseppe Bilotta 
Assigned ToBrad King 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformLinux x86_64OSDebianOS Versionunstable
Product Version 
Target VersionCMake 2.8.6Fixed in VersionCMake 2.8.6 
Summary0012394: Check compiler flags for Clang
DescriptionThe CHECK_C_COMPILER_FLAG and CHECK_CXX_COMPILER_FLAG macros do not properly detect the warning message with which clang complains about unknown flags
Steps To ReproduceA test such as
CHECK_CXX_COMPILER_FLAG("-Wagoiuhaoigfh" FLAG_IS_KNOWN)
will report Success instead of Fail
Additional Information
TagsNo tags attached.
Relationships
related to 0009516closed Brad King CHECK_COMPILER_FLAG and CHECK_CXX_COMPILER_FLAG won't detect unrecognized options with GCC 
Attached Filespatch 0001-Fix-CHECK_-C-CXX-_COMPILER_FLAG-for-Clang.patch (2,011) 2011-08-08 11:57
https://public.kitware.com/Bug/file/4000/0001-Fix-CHECK_-C-CXX-_COMPILER_FLAG-for-Clang.patch
Issue History
Date ModifiedUsernameFieldChange
2011-08-08 11:56Giuseppe BilottaNew Issue
2011-08-08 11:57Giuseppe BilottaFile Added: 0001-Fix-CHECK_-C-CXX-_COMPILER_FLAG-for-Clang.patch
2011-08-08 11:57Giuseppe BilottaNote Added: 0027180
2011-08-08 12:54Brad KingAssigned To => Brad King
2011-08-08 12:54Brad KingStatusnew => assigned
2011-08-08 12:54Brad KingRelationship addedrelated to 0009516
2011-08-08 12:54Brad KingNote Added: 0027181
2011-08-08 13:00Brad KingNote Added: 0027182
2011-08-08 13:00Brad KingStatusassigned => resolved
2011-08-08 13:00Brad KingResolutionopen => fixed
2011-09-12 17:16David ColeFixed in Version => CMake 2.8.6
2011-09-12 17:16David ColeTarget Version => CMake 2.8.6
2012-02-06 06:07David ColeNote Added: 0028505
2012-02-06 06:07David ColeStatusresolved => closed

Notes
(0027180)
Giuseppe Bilotta   
2011-08-08 11:57   
The attached file is a git patch that fixes the issues.
(0027181)
Brad King   
2011-08-08 12:54   
It looks like that patch was adapted from

  http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=53fb07ef [^]

Thanks.
(0027182)
Brad King   
2011-08-08 13:00   
Fixed, thanks:

  http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=ed6be501 [^]
(0028505)
David Cole   
2012-02-06 06:07   
Closing resolved issues that have not been updated in more than 4 months.