[libcamera-devel] [PATCH v1 2/6] qcam: Simplify dependency handling for libtiff

[paul.elder at ideasonboard.com]

On Thu, Oct 20, 2022 at 02:15:33AM +0300, Laurent Pinchart via libcamera-devel wrote:
> There's no need to conditionally add tiff_dep to the list of qcam
> dependency, as a not found dependency() is ignored when listed in the
> executable dependencies argument.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

Reviewed-by: Paul Elder <paul.elder at ideasonboard.com>

> ---
>  src/qcam/meson.build | 14 ++++++--------
>  1 file changed, 6 insertions(+), 8 deletions(-)
> 
> diff --git a/src/qcam/meson.build b/src/qcam/meson.build
> index 9f5759ff0786..d5916d0df532 100644
> --- a/src/qcam/meson.build
> +++ b/src/qcam/meson.build
> @@ -36,18 +36,11 @@ qcam_resources = files([
>      'assets/feathericons/feathericons.qrc',
>  ])
>  
> -qcam_deps = [
> -    libatomic,
> -    libcamera_public,
> -    qt5_dep,
> -]
> -
>  qt5_cpp_args = ['-DQT_NO_KEYWORDS']
>  
>  tiff_dep = dependency('libtiff-4', required : false)
>  if tiff_dep.found()
>      qt5_cpp_args += ['-DHAVE_TIFF']
> -    qcam_deps += [tiff_dep]
>      qcam_sources += files([
>          '../cam/dng_writer.cpp',
>      ])
> @@ -81,5 +74,10 @@ resources = qt5.preprocess(moc_headers: qcam_moc_headers,
>  
>  qcam  = executable('qcam', qcam_sources, resources,
>                     install : true,
> -                   dependencies : qcam_deps,
> +                   dependencies : [
> +                       libatomic,
> +                       libcamera_public,
> +                       qt5_dep,
> +                       tiff_dep,
> +                   ],
>                     cpp_args : qt5_cpp_args)