MantisBT - CMake
View Issue Details
0012301CMakeCMakepublic2011-06-22 17:362012-01-02 15:56
Stephen Kelly 
David Cole 
normalminoralways
closedfixed 
CMake 2.8.4 
CMake 2.8.5CMake 2.8.5 
0012301: CMAKE_BUILD_TYPE not set on MinGW
CMAKE_BUILD_TYPE is not set when using MinGW, which causes problems with loading Qt plugins unless the user manually sets the CMAKE_BUILD_TYPE.

CMAKE_BUILD_TYPE should be preset to debug or release, as it is on most other platforms.

Context here:

http://thread.gmane.org/gmane.comp.programming.tools.cmake.user/36994/focus=37032 [^]
No tags attached.
Issue History
2011-06-22 17:36Stephen KellyNew Issue
2011-06-23 12:15Clinton StimpsonNote Added: 0026958
2011-07-29 19:15Clinton StimpsonNote Added: 0027129
2011-08-01 11:30David ColeAssigned To => David Cole
2011-08-01 11:30David ColeStatusnew => assigned
2011-08-01 11:31David ColeNote Added: 0027143
2011-08-01 11:31David ColeStatusassigned => resolved
2011-08-01 11:31David ColeFixed in Version => CMake 2.8.5
2011-08-01 11:31David ColeResolutionopen => fixed
2011-08-01 11:31David ColeTarget Version => CMake 2.8.5
2011-08-01 11:31David ColeDescription Updatedbug_revision_view_page.php?rev_id=396#r396
2012-01-02 15:56David ColeNote Added: 0028117
2012-01-02 15:56David ColeStatusresolved => closed

Notes
(0026958)
Clinton Stimpson   
2011-06-23 12:15   
This commit is a better fix for the problem.
http://cmake.org/gitweb?p=cmake.git;a=commit;h=5a2b208617d13808287b3c785183fed9c99576f1 [^]

If there is no other compelling reason for the build type to have a default for MinGW, then I think we can close this bug.
(0027129)
Clinton Stimpson   
2011-07-29 19:15   
Isn't an empty CMAKE_BUILD_TYPE valid for MinGW, like it is with Unix Makefiles, unlike with NMake and the MSVC compiler?
(0027143)
David Cole   
2011-08-01 11:31   
I agree with Clinton here. I think his fix regarding the definition of QT_NO_DEBUG is the appropriate fix and we can leave the value of CMAKE_BUILD_TYPE empty by default for MinGW. As it is with gcc and Unix Makefiles on other platforms, too.

Feel free to re-open this issue if you think it needs more discussion.
(0028117)
David Cole   
2012-01-02 15:56   
Closing resolved issues that have not been updated in more than 4 months.