On Mon, 2014-07-28 at 19:17 +0100, Fraser Adams wrote: > On 28/07/14 14:29, Alan Conway wrote: > > > > Based on discussion so far I am inclined to > > - make the default build type RelWithDebInfo > > - udpate cpp/INSTALL discussion of build types (it does mention > > Release/Debug already, I'll add mention of the other types. > > > > We don't have unanimity over Release vs. RelWithDebInfo as the preferred > > default, but RelWithDebInfo appears to have more support. Don't forget > > this is only the default, people can choose whatever they prefer (and > > the INSTALL will be updated so they'll know what the choices are) > > > I still don't really like symbols being included by default, but I > appear to be outnumbered. If that is going to be the default could I ask > that some comment/echo be added to the CMake to say what build type is > being used and to warn users that debug symbols are being included?
Done ------------------------------------------------------------------------ r1614472 | aconway | 2014-07-29 15:59:19 -0400 (Tue, 29 Jul 2014) | 2 lines QPID-5941: Set sensible default build type: default is RelWithDebInfo. ------------------------------------------------------------------------ --------------------------------------------------------------------- To unsubscribe, e-mail: users-unsubscr...@qpid.apache.org For additional commands, e-mail: users-h...@qpid.apache.org
