[libcamera-devel] [PATCH] meson: options: Add a option to control compilation of qcam
Laurent Pinchart
laurent.pinchart at ideasonboard.com
Tue Jun 16 20:11:59 CEST 2020
On Tue, Jun 16, 2020 at 02:03:15PM -0400, Nicolas Dufresne wrote:
> Le mardi 16 juin 2020 à 17:27 +0200, Niklas Söderlund a écrit :
> > Add a option to control compilation of the qcam test application. The
> > default behavior is to compile qcam, no change in behavior without user
> > intervention.
> >
> > Signed-off-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>
> > ---
> > meson_options.txt | 5 +++++
> > src/meson.build | 5 ++++-
> > 2 files changed, 9 insertions(+), 1 deletion(-)
> >
> > diff --git a/meson_options.txt b/meson_options.txt
> > index badace151bb62bc9..9a4edf2a5fde8bcb 100644
> > --- a/meson_options.txt
> > +++ b/meson_options.txt
> > @@ -27,3 +27,8 @@ option('v4l2',
> > type : 'boolean',
> > value : false,
> > description : 'Compile the V4L2 compatibility layer')
> > +
> > +option('qcam',
> > + type : 'boolean',
> > + value : true,
> > + description : 'Compile the qcam test application')
> > diff --git a/src/meson.build b/src/meson.build
> > index d69b4c1ea9788f83..3a816fb610a68905 100644
> > --- a/src/meson.build
> > +++ b/src/meson.build
> > @@ -19,7 +19,10 @@ endif
> > subdir('libcamera')
> > subdir('ipa')
> > subdir('cam')
> > -subdir('qcam')
> > +
> > +if get_option('qcam')
> > + subdir('qcam')
> > +endif
>
> It's much cleaner if you check that option inside the subdir
> meson.build and call subdir_done() instead.
>
> (ignore if your required meson version is too ancient)
Actually, now that I've sent an Rb tag, I wonder if we should make the
option a feature instead of a boolean, and use
qt5_dep = dependency('qt5',
method : 'pkg-config',
modules : ['Core', 'Gui', 'Widgets'],
required : get_option('qcam'))
like we do for gstreamer. We wouldn't need to use subdir_done().
> >
> > if get_option('v4l2')
> > subdir('v4l2')
>
> And then this could be cleaned up.
subdir_done() was introduced in 0.46 and we require 0.47 or newer, so
this should indeed be cleaned up.
--
Regards,
Laurent Pinchart
More information about the libcamera-devel
mailing list