Review Request 113805: Do not change the build types available with cmake
Nicolás Alvarez
nicolas.alvarez at gmail.com
Tue Nov 12 00:18:37 UTC 2013
> On Nov. 11, 2013, 9:02 p.m., Aleix Pol Gonzalez wrote:
> > Maybe I've missed something, but I would like to have it explained somehow. Why is it bad to define such values? How will g++ calls compare?
In normal CMake, -DCMAKE_BUILD_TYPE=Debug builds without optimization and with debugging symbols. But in KDE projects, Debug uses "-O2 -g" and there's a custom type called "Debugfull" which uses "-O0 -g".
This patch removes all the custom stuff so that "Debug" means the same (debug, no opts) whether it's a KDE project or not. To get "-O2 -g" there's RelWithDebInfo.
- Nicolás
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113805/#review43480
-----------------------------------------------------------
On Nov. 11, 2013, 7:16 p.m., Sune Vuorela wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113805/
> -----------------------------------------------------------
>
> (Updated Nov. 11, 2013, 7:16 p.m.)
>
>
> Review request for Build System and KDE Frameworks.
>
>
> Repository: extra-cmake-modules
>
>
> Description
> -------
>
> Do not change the build types available with cmake.
>
>
> Diffs
> -----
>
> kde-modules/KDECompilerSettings.cmake b034751a5be8073f9628971b552faa079c64e8b6
>
> Diff: http://git.reviewboard.kde.org/r/113805/diff/
>
>
> Testing
> -------
>
> Built kdelibs on linux with gcc.
>
>
> Thanks,
>
> Sune Vuorela
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20131112/f3007637/attachment.html>
More information about the Kde-buildsystem
mailing list