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