[Buildroot] [PATCH v2] package/pkg-cmake: use MinSizeRel for size optimized build

[Samuel Martin]

Hi Jörg,

On Sun, Jan 17, 2016 at 8:25 PM, Jörg Krause
<joerg.krause at embedded.rocks> wrote:
> MinSizeRel sets the gcc compiler optimization level to '-Os' whereas Release
> sets it to '-O3'.
>
> Signed-off-by: Jörg Krause <joerg.krause at embedded.rocks>
> ---
> Changes v2:
>  - fix typo
> ---
>  package/pkg-cmake.mk | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/package/pkg-cmake.mk b/package/pkg-cmake.mk
> index 81dcfcc..43da079 100644
> --- a/package/pkg-cmake.mk
> +++ b/package/pkg-cmake.mk
> @@ -87,7 +87,7 @@ define $(2)_CONFIGURE_CMDS
>         PATH=$$(BR_PATH) \
>         $$($$(PKG)_CONF_ENV) $$(HOST_DIR)/usr/bin/cmake $$($$(PKG)_SRCDIR) \
>                 -DCMAKE_TOOLCHAIN_FILE="$$(HOST_DIR)/usr/share/buildroot/toolchainfile.cmake" \
> -               -DCMAKE_BUILD_TYPE=$$(if $$(BR2_ENABLE_DEBUG),Debug,Release) \
> +               -DCMAKE_BUILD_TYPE=$$(if $$(BR2_ENABLE_DEBUG),Debug,$$(if $$(BR2_OPTIMIZE_S),MinSizeRel,Release)) \

This looks good! :-)
Could you update the doc [1] as well?

[1] https://git.busybox.net/buildroot/tree/docs/manual/adding-packages-cmake.txt#n103

Regards,

-- 
Samuel