Re: [gentoo-dev] [PATCH 4/4] cmake.eclass: Default CMAKE_BUILD_TYPE=RelWithDebInfo in EAPI 8

[Guilherme Amadio]

Hi,

On Thu, Aug 19, 2021 at 07:45:16PM +0200, Andreas Sturmlechner wrote:
> Signed-off-by: Michał Górny <mgorny@g.o>
> Signed-off-by: Andreas Sturmlechner <asturm@g.o>
> ---
>  eclass/cmake.eclass | 11 ++++++++---
>  1 file changed, 8 insertions(+), 3 deletions(-)
>
> diff --git a/eclass/cmake.eclass b/eclass/cmake.eclass
> index 8befd9e5a9f..3021a3a2b1e 100644
> --- a/eclass/cmake.eclass
> +++ b/eclass/cmake.eclass
> @@ -42,14 +42,19 @@ _CMAKE_ECLASS=1
>  # Eclass can use different cmake binary than the one provided in by system.
>  : ${CMAKE_BINARY:=cmake}
>
> +[[ ${EAPI} == 7 ]] && : ${CMAKE_BUILD_TYPE:=Gentoo}
>  # @ECLASS-VARIABLE: CMAKE_BUILD_TYPE
>  # @DESCRIPTION:
>  # Set to override default CMAKE_BUILD_TYPE. Only useful for packages
>  # known to make use of "if (CMAKE_BUILD_TYPE MATCHES xxx)".
>  # If about to be set - needs to be set before invoking cmake_src_configure.
> -# You usually do *NOT* want nor need to set it as it pulls CMake default
> -# build-type specific compiler flags overriding make.conf.
> -: ${CMAKE_BUILD_TYPE:=Gentoo}
> +#
> +# The default is RelWithDebInfo as that is least likely to append undesirable
> +# flags. However, you may still need to sed CMake files or choose a different
> +# build type to achieve desirable results.
> +#
> +# In EAPI 7, the default was non-standard build type of Gentoo.
> +: ${CMAKE_BUILD_TYPE:=RelWithDebInfo}

Although I think this is better than the "Gentoo" build type we had, I
think that even better is to just leave this blank and set a default on
a per-ebuild basis, as I commented on bug 802786[1]. Repeating my
comment from there, build type flags always come after CMAKE_<LANG>_FLAGS,
which then override make.conf settings. Moreover, setting a build type
can also have funny behavior with multi-config generators (possible with
ninja on Linux too now) because it has no meaning in that context. For
instance, this may override make.conf flags with "-O2 -g" in most cases,
preventing people from changing the optimization level of any CMake-based
package, or disabling compilation of debug info (which is then stripped
anyway). This is probably why we had the "Gentoo" build in the first place.

For ebuilds that must have a CMAKE_BUILD_TYPE set, we should probably
use Release and override CMAKE_<LANG>_FLAGS_RELEASE with settings from
make.conf, or make CMAKE_<LANG>_FLAGS contain settings from make.conf
and enforce CMAKE_<LANG>_FLAGS_<BUILD_TYPE> is empty for all build types.

Cheers,
-Guilherme

1. https://bugs.gentoo.org/802786

>
>  # @ECLASS-VARIABLE: CMAKE_IN_SOURCE_BUILD
>  # @DEFAULT_UNSET
> --
> 2.33.0
>