From: Orcan O. <oge...@gm...> - 2010-11-19 06:45:53
|
On Thu, Nov 18, 2010 at 3:37 AM, Tim E. Real wrote: > I concluded that we cannot override the CMAKE_BUILD_TYPE if it is empty, > in order to set it to something more reasonable than plain un-opt, > like Release. > > If no CMAKE_BUILD_TYPE is given on the command line, > cmake either uses the cached value, or 'empty' (plain un-opt build). > And yet the only way a user can reset a cached CMAKE_BUILD_TYPE > to 'empty' is with "-DCMAKE_BUILD_TYPE=". > So we cannot interfere with this. > Hmm, why don't we set one default? e.g. "release" for releases, "debug" for svn checkouts etc. > So I print out the build type now to make the user aware of what they're > building, and updated the README. > Be sure to turn on that Release type manually ! > > What do you think? > Nicely done. Brief and to the point. Thanks, Orcan |