[libcamera-devel] [PATCH] libcamera: Use meson summary() function to summarize configuration

Niklas Söderlund niklas.soderlund at ragnatech.se
Wed Jan 20 16:11:40 CET 2021


Hi Laurent,

Thanks for your work.

On 2021-01-20 09:40:48 +0200, Laurent Pinchart wrote:
> Now that Debian backports have been updated to meson v0.56, all major
> distributions provide meson >= v0.53 in their latest LTS version. We can
> replace the manual message()-based configuration summary with the
> summmary() function.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

Reviewed-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>

> ---
>  meson.build | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/meson.build b/meson.build
> index c47eb420f4db..743cb1d299f7 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -1,7 +1,7 @@
>  # SPDX-License-Identifier: CC0-1.0
>  
>  project('libcamera', 'c', 'cpp',
> -    meson_version : '>= 0.51',
> +    meson_version : '>= 0.53',
>      version : '0.0.0',
>      default_options : [
>          'werror=true',
> @@ -155,4 +155,6 @@ py_mod = import('python')
>  py_mod.find_installation('python3', modules: py_modules)
>  
>  ## Summarise Configurations
> -message('Enabled pipelines: ' + ', '.join(get_option('pipelines')))
> +summary({
> +            'Enabled pipelines': get_option('pipelines'),
> +        }, section : 'Configuration')
> -- 
> Regards,
> 
> Laurent Pinchart
> 
> _______________________________________________
> libcamera-devel mailing list
> libcamera-devel at lists.libcamera.org
> https://lists.libcamera.org/listinfo/libcamera-devel

-- 
Regards,
Niklas Söderlund


More information about the libcamera-devel mailing list