Date: Mon, 23 Jan 2023 12:35:21 +0100
From: Jacopo Mondi <jacopo.mondi@ideasonboard.com>
To: Kevin Daudt <kdaudt@alpinelinux.org>
Cc: Jacopo Mondi <jacopo.mondi@ideasonboard.com>,
	alpine-aports@lists.alpinelinux.org
Subject: Re: [PATCH 1/2] community/libcamera: remove boost from dependencies
Message-ID: <20230123113521.ibtgnryx4drvl6kd@uno.localdomain>
References: <20230114163844.10487-1-jacopo.mondi@ideasonboard.com>
 <Y8ZAcVIYWi66ml2s@bbea3f19b1bb>
 <20230123111724.pz2ixbmrjvr5i3ua@uno.localdomain>
 <Y85tDhK716t+KntA@7a0aac3667e2>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
In-Reply-To: <Y85tDhK716t+KntA@7a0aac3667e2>

Hi Kevin

On Mon, Jan 23, 2023 at 11:18:38AM +0000, Kevin Daudt wrote:
> On Mon, Jan 23, 2023 at 12:17:24PM +0100, Jacopo Mondi wrote:
> > Hi Kevin
> >
> > On Tue, Jan 17, 2023 at 06:30:09AM +0000, Kevin Daudt wrote:
> > > On Sat, Jan 14, 2023 at 05:38:43PM +0100, Jacopo Mondi wrote:
> > > > Boost is not required anymore to build libcamera since the dependency
> > > > has been dropped in commit c1597f989654 ("ipa: raspberrypi: Use
> > > > YamlParser to replace dependency on boost").
> > > >
> > > > Signed-off-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com>
> > > > ---
> > > >  community/libcamera/APKBUILD | 1 -
> > > >  1 file changed, 1 deletion(-)
> > > >
> > > > diff --git a/community/libcamera/APKBUILD b/community/libcamera/APKBUILD
> > > > index 1399bcda0f74..1bf0015ec127 100644
> > > > --- a/community/libcamera/APKBUILD
> > > > +++ b/community/libcamera/APKBUILD
> > > > @@ -8,7 +8,6 @@ url="https://libcamera.org/"
> > > >  arch="all"
> > > >  license="LGPL-2.1-or-later AND GPL-2.0-or-later"
> > > >  depends_dev="
> > > > -	boost-dev
> > > >  	eudev-dev
> > > >  	glib-dev
> > > >  	gnutls-dev
> > > > --
> > > > 2.39.0
> > > >
> > >
> > > FYI, you need to bump the pkgrel, otherwise the package does not get
> > > rebuilt.
> >
> > Ah thanks, will fix and resend soon!
>
> FYI, the MR[0] has already been merged.
>
> [0]: https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/43267

I noticed as soon as I've re-fetch the repo. Sorry I was not following
gitlab and thanks for fixing the not-required sdl2_image dependency
which indeed seems a leftover.

Cheers