[libcamera-devel] [PATCH 1/2] media-libs/libcamera: Android support is a feature not a boolean

[Niklas Söderlund]

Hi Tomasz,

On 2020-11-10 19:06:41 +0900, Tomasz Figa wrote:
> Hi Niklas,
> 
> 
> On Mon, Nov 9, 2020 at 10:17 AM Niklas Söderlund
> <niklas.soderlund at ragnatech.se> wrote:
> >
> > Upstream switched the android option from a boolean to a meson feature,
> > update ebuild to reflect this.
> >
> > Signed-off-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>
> > ---
> >  media-libs/libcamera/libcamera-9999.ebuild | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/media-libs/libcamera/libcamera-9999.ebuild b/media-libs/libcamera/libcamera-9999.ebuild
> > index 4273adbc9e1e0f33..57ff00337309f30c 100644
> > --- a/media-libs/libcamera/libcamera-9999.ebuild
> > +++ b/media-libs/libcamera/libcamera-9999.ebuild
> > @@ -35,7 +35,7 @@ src_configure() {
> >         local emesonargs=(
> >                 $(meson_use doc documentation)
> >                 $(meson_use test)
> > -               -Dandroid="true"
> > +               -Dandroid=enabled
> >                 -Dpipelines="$(pipeline_list "${pipelines[@]}")"
> >         )
> >         meson_src_configure
> > --
> > 2.25.1
> >
> 
> Not sure why this wasn't the case for you, but this has been fixed in
> Chromium OS for quite a long time already:
> https://chromium-review.googlesource.com/c/chromiumos/overlays/chromiumos-overlay/+/2418720

Cool, did not know that. I'm still working on from the manifest for 
R87-13476 to have as much of common base as possible that's likely why I 
did not already have this fix.

> 
> Best regards,
> Tomasz

-- 
Regards,
Niklas Söderlund