View Issue Details [ Jump to Notes ] | [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0012633 | CMake | CMake | public | 2011-12-18 14:11 | 2012-09-03 16:01 | ||||
Reporter | hrogge | ||||||||
Assigned To | Brad King | ||||||||
Priority | normal | Severity | major | Reproducibility | always | ||||
Status | closed | Resolution | fixed | ||||||
Platform | OS | OS Version | |||||||
Product Version | CMake 2.8.6 | ||||||||
Target Version | CMake 2.8.8 | Fixed in Version | CMake 2.8.8 | ||||||
Summary | 0012633: GCC fails some flags with "is valid for C++/ObjC++ but not for C" | ||||||||
Description | GCC has some flags that are valid for C++, but not for C. The different warning message breaks the CHECK_C_COMPILER_FLAG macro. Suggestion: add 'FAIL_REGEX "but not for C" # GNU' to CheckCCompilerFlag.cmake | ||||||||
Steps To Reproduce | compile "int main(void) { return 0 };" on gcc 4.4.5 with -Wenum-compare: /usr/bin/gcc -Wenum-compare -o test.o -c test.c cc1: warning: command line option "-Wenum-compare" is valid for C++/ObjC++ but not for C | ||||||||
Additional Information | Combined with ADD_DEFINITIONS(-Werror) this can break the build process. The CheckCCompilerFlag does not recognize that -Wenum-compare is not available, -Werror stops the build process. | ||||||||
Tags | No tags attached. | ||||||||
Attached Files | |||||||||
Relationships | |
Relationships |
Notes | |
(0028009) Brad King (manager) 2011-12-19 10:55 |
The regex added in http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=b7f3f7cd [^] doesn't match? |
(0028015) Sean McBride (reporter) 2011-12-19 23:21 |
Not exactly related, but I'm very much reminded of bug 0004756. |
(0028029) hrogge (reporter) 2011-12-21 14:50 |
Ahh, I just noticed I was using 2.8.5 on one of the machines. The error does only happen in 2.8.6 when you use "-Werror" together with the other option, because the start of the output will change from "warning" to "error". But testing two options at once was more an experiment to learn what was going on than a good usage. Sorry for making that much noise. Maybe it would be a good idea to completely remove the "warning:" part from the regex (and replace the "# Apple gcc" comment with "# gcc", because it can also happens on Linux). |
(0028035) Brad King (manager) 2011-12-22 09:42 |
Good ideas, thanks: http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=646cf919 [^] |
(0030868) David Cole (manager) 2012-09-03 16:01 |
Closing resolved issues that have not been updated in more than 4 months. |
Notes |
Issue History | |||
Date Modified | Username | Field | Change |
2011-12-18 14:11 | hrogge | New Issue | |
2011-12-19 10:55 | Brad King | Note Added: 0028009 | |
2011-12-19 23:21 | Sean McBride | Note Added: 0028015 | |
2011-12-21 14:50 | hrogge | Note Added: 0028029 | |
2011-12-22 09:42 | Brad King | Note Added: 0028035 | |
2011-12-22 09:42 | Brad King | Status | new => resolved |
2011-12-22 09:42 | Brad King | Resolution | open => fixed |
2011-12-22 09:42 | Brad King | Assigned To | => Brad King |
2012-04-19 15:42 | David Cole | Fixed in Version | => CMake 2.8.8 |
2012-04-19 15:42 | David Cole | Target Version | => CMake 2.8.8 |
2012-09-03 16:01 | David Cole | Note Added: 0030868 | |
2012-09-03 16:01 | David Cole | Status | resolved => closed |
Issue History |
Copyright © 2000 - 2018 MantisBT Team |