Re: [oe] [meta-multimedia][PATCH] libcamera: add libdrm dependency

Laurent Pinchart <[email protected]>
Newsgroups org.openembedded.lists.openembedded-devel
Message-ID <[email protected]>
On Mon, Jul 13, 2026 at 05:36:35PM +0100, Kieran Bingham wrote:
> +Cc libcamera-devel, Robert,
> 
> Quoting Markus Volk (2026-07-13 17:23:12)
> > Am Mo, 13. Jul 2026 um 11:38:14 +01:00:00 schrieb Kieran Bingham via 
> > lists.openembedded.org 
> > <[email protected]>:
> > > Perhaps rather than add an incorrect dependency, we could backport the
> > > patch for the recipe or bump to libcamera-0.7.2?
> > 
> > That sounds reasonable. I have updated libcamera to v0.7.2 and now 
> > libyuv seems to be a requirement. It was formerly added by the 'virtual'
> > PACKAGECONFIG where I have no idea what this is for but removing this 
> > PACKAGECONFIG  and adding libyuv to DEPENDS fixes the build.
> > Since you know about libcameras dependencies: Is this the right thing 
> > to do or am I missing something about 'virtual'?
> 
> Aha, I wish I'd thought of that for the release notes.
> 
> There was a patch merged that makes virtual built in by default.
> 
> So adding libyuv as a dependency for 0.7.2 probably makes sense. I
> wonder if virtual should instead only be defaulted to build in when
> libyuv is available!

Not a good idea, that would break reproducible builds badly.

> > 0.7.2 builds for me like this:
> > 
> > diff --git 
> > a/meta-multimedia/recipes-multimedia/libcamera/libcamera_0.7.1.bb 
> > b/meta-multimedia/recipes-multimedia/libcamera/libcamera_0.7.2.bb
> > similarity index 97%
> > rename from 
> > meta-multimedia/recipes-multimedia/libcamera/libcamera_0.7.1.bb
> > rename to 
> > meta-multimedia/recipes-multimedia/libcamera/libcamera_0.7.2.bb
> > index 2b351a39be..4b5e735a37 100644
> > --- a/meta-multimedia/recipes-multimedia/libcamera/libcamera_0.7.1.bb
> > +++ b/meta-multimedia/recipes-multimedia/libcamera/libcamera_0.7.2.bb
> > @@ -13,11 +13,11 @@ SRC_URI = " \
> >          
> > git://git.libcamera.org/libcamera/libcamera.git;protocol=https;branch=master;tag=v${PV 
> > <git://git.libcamera.org/libcamera/libcamera.git;protocol=https;branch=master;tag=v$%7BPV>} 
> > \
> >  "
> > 
> > -SRCREV = "183e37362f57ff3ce7493abf0bc6f1b57b931f55"
> > +SRCREV = "191e202178f02430b5942397c70d215cdd2056fa"
> > 
> >  PE = "1"
> > 
> > -DEPENDS = "python3-pyyaml-native python3-jinja2-native 
> > python3-ply-native python3-jinja2-native udev gnutls chrpath-native 
> > libevent libyaml"
> > +DEPENDS = "python3-pyyaml-native python3-jinja2-native 
> > python3-ply-native python3-jinja2-native udev gnutls chrpath-native 
> > libevent libyaml libyuv"
> >  DEPENDS += "${@bb.utils.contains('DISTRO_FEATURES', 'qt', 'qtbase 
> > qtbase-native', '', d)}"
> > 
> >  PACKAGES =+ "${PN}-compliance ${PN}-gst ${PN}-pycamera"
> > @@ -38,7 +38,6 @@ PACKAGECONFIG[raspberrypi] = ",,libpisp"
> >  PACKAGECONFIG[sdl] = 
> > "-Dcam-output-sdl2=enabled,-Dcam-output-sdl2=disabled,virtual/libsdl2"
> >  PACKAGECONFIG[unwind] = 
> > "-Dlibunwind=enabled,-Dlibunwind=disabled,libunwind"
> >  PACKAGECONFIG[vimc] = ",,"
> > -PACKAGECONFIG[virtual] = ",,libyuv libjpeg-turbo"
> > 
> >  ARM_PIPELINES = "imx8-isi,mali-c55,simple,uvcvideo"
> >  # Raspberry Pi requires the meta-raspberrypi layer

-- 
Regards,

Laurent Pinchart
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.