Review Request 114885: Remove custom build types
David Faure
faure at kde.org
Tue Jan 7 19:06:45 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/114885/#review46996
-----------------------------------------------------------
Ship it!
Nice, this also fixes some inconsistencies between compilers.
I tested in a pure qt5+cmake test (no ECM) that:
* RelWithDebInfo already sets -DQT_NO_DEBUG -O2 -g -DNDEBUG. So this was indeed unnecessary by now.
* Release sets -DQT_NO_DEBUG -O3 -DNDEBUG, which is even better (-O3 vs -O2) than what ECM was doing.
* Debug sets -g. No -O2, which isn't expected in debug mode.
- David Faure
On Jan. 7, 2014, 4:30 p.m., Alex Merry wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/114885/
> -----------------------------------------------------------
>
> (Updated Jan. 7, 2014, 4:30 p.m.)
>
>
> Review request for Build System, KDE Frameworks, David Faure, Kevin Ottens, and Stephen Kelly.
>
>
> Repository: extra-cmake-modules
>
>
> Description
> -------
>
> This is a cleaned-up version of https://git.reviewboard.kde.org/r/113805/ , with documentation fixes. The discussion there appeared to end up being largely in favour of this move.
>
> Obviously, this can only go in once TP1 is done.
>
>
> Remove custom build types
>
> KDECompilerSettings.cmake no longer alters CMake's built-in build types
> or adds its own. The "debug" build type therefore simply sets -g with
> no additional flags (rather than -O2 and, depending on the compiler,
> some no-inline/no-reorder flags as previously), the "release" build
> types no longer set -DQT_NO_DEBUG and the "debugfull", "profile" and
> "coverage" build types no longer exist.
>
> QT_NO_DEBUG is set by Qt's CMake scripts depending on the build type of
> Qt itself. "debugfull" mostly set -g3, allowing macro expansion when
> debugging; users can set this flag using environment variables if they
> wish. "RelWithDebugInfo" should be used instead of "profile" (according
> to dfaure); -fprofile-arcs and -ftest-coverage are easy enough to add to
> $CXX_FLAGS if they are required (formerly set by "profile" and
> "coverage").
>
>
> Diffs
> -----
>
> kde-modules/KDECompilerSettings.cmake 72824e166d03dcc2d089814dc121f08ba998974a
>
> Diff: https://git.reviewboard.kde.org/r/114885/diff/
>
>
> Testing
> -------
>
> Built kcoreaddons on linux with gcc. -DCMAKE_BUILD_TYPE=debugfull works, but does not set -g. -DCMAKE_BUILD_TYPE=debug does set -g.
>
>
> Thanks,
>
> Alex Merry
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20140107/712fb4cc/attachment.html>
More information about the Kde-buildsystem
mailing list