MantisBT - CMake
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0014606CMakeCMakepublic2013-11-28 06:332014-06-02 08:37
ReporterStephan Tolksdorf 
Assigned ToBrad King 
PrioritynormalSeveritytextReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product VersionCMake 2.8.12 
Target VersionCMake 3.0Fixed in VersionCMake 3.0 
Summary0014606: Obsolete description for CMAKE_(C|CXX|Fortran)_FLAGS_RELEASE
DescriptionBy default, the descriptions/helpstrings for CMAKE_C_FLAGS_RELEASE, CMAKE_CXX_FLAGS_RELEASE and CMAKE_Fortran_FLAGS_RELEASE all contain "(/MD /Ob1 /Oi /Ot /Oy /Gs will produce slightly less optimized but smaller files)". This comment seems obsolete and probably should be removed, as the flags are compiler dependent and a user could use the MINSIZEREL configuration to optimize for size.

I also noticed the inconsistent capitalization in the RELWITHDEBINFO descriptions.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2013-11-28 06:33Stephan TolksdorfNew Issue
2013-12-02 10:27Brad KingNote Added: 0034672
2013-12-02 10:29Brad KingNote Added: 0034673
2013-12-02 10:29Brad KingAssigned To => Brad King
2013-12-02 10:29Brad KingStatusnew => resolved
2013-12-02 10:29Brad KingResolutionopen => fixed
2013-12-02 10:29Brad KingFixed in Version => CMake 3.0
2013-12-02 10:29Brad KingTarget Version => CMake 3.0
2013-12-02 10:35Stephan TolksdorfNote Added: 0034674
2013-12-02 10:35Stephan TolksdorfStatusresolved => feedback
2013-12-02 10:35Stephan TolksdorfResolutionfixed => reopened
2013-12-02 10:43Brad KingNote Added: 0034676
2013-12-02 10:43Brad KingStatusfeedback => resolved
2013-12-02 10:43Brad KingResolutionreopened => fixed
2014-06-02 08:37Robert MaynardNote Added: 0036047
2014-06-02 08:37Robert MaynardStatusresolved => closed

Notes
(0034672)
Brad King   
2013-12-02 10:27   
Fixed, thanks:

Fix CMAKE_(C|CXX|Fortran)_RELEASE helpstrings
http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=424c6fcc [^]
(0034673)
Brad King   
2013-12-02 10:29   
The RELWITHDEBINFO/RelWithDebInfo capitalization is as intended. The configuration names are case insensitive. The generated projects use CamelCase but the variables are UPPERCASE. It is historical.
(0034674)
Stephan Tolksdorf   
2013-12-02 10:35   
Thanks for fixing this.

I meant the capitalization in the description "Flags used by the compiler during Release with Debug Info builds", which is inconsistent than the one used in the other descriptions (note the upper case 'Release', 'Debug' and 'Info' here).
(0034676)
Brad King   
2013-12-02 10:43   
Okay, thanks for clarifying. Also fixed:

Make CMAKE_(C|CXX|Fortran)_<CONFIG> help strings more consistent
http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=2b77b532 [^]
(0036047)
Robert Maynard   
2014-06-02 08:37   
Closing resolved issues that have not been updated in more than 4 months.