Re: [OE-Core][PATCH] curl-native: add missing ca-certificates.crt file

["Piotr Łobacz" <p.lobacz@welotec.com>]

Dnia poniedziałek, 13 listopada 2023 12:35:23 CET Richard Purdie pisze:
> On Fri, 2023-11-10 at 20:37 +0100, Piotr Łobacz wrote:
> > By default curl is being configured with:
> > 
> > --with-ca-bundle=${sysconfdir}/ssl/certs/ca-certificates.crt
> > 
> > which causes an issue for native build, when calling
> > curl-native command, as certificates file is missing.
> > 
> > This commit fixes the issue.
> > 
> > Signed-off-by: Piotr Łobacz <p.lobacz@welotec.com>
> > ---
> > 
> >  meta/recipes-support/curl/curl_8.4.0.bb | 2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/meta/recipes-support/curl/curl_8.4.0.bb
> > b/meta/recipes-support/curl/curl_8.4.0.bb index 5f97730bf4..165a116139
> > 100644
> > --- a/meta/recipes-support/curl/curl_8.4.0.bb
> > +++ b/meta/recipes-support/curl/curl_8.4.0.bb
> > @@ -130,7 +130,9 @@ PACKAGES =+ "lib${BPN}"
> > 
> >  FILES:lib${BPN} = "${libdir}/lib*.so.*"
> >  RRECOMMENDS:lib${BPN} += "ca-certificates"
> > 
> > +DEPENDS:append:class-native = " ca-certificates-native"
> > 
> > +SYSROOT_DIRS:append =" /etc/ssl/certs"
> > 
> >  FILES:${PN} += "${datadir}/zsh"
> >  
> >  inherit multilib_script
> 
> Adding the dependency I can understand but the SYSROOT_DIRS append
> would need to be in the ca-certificates recipe to be of any use as I
> understand it?

Yeah, my mistake ca-certificates recipe needs this sorry haven't noticed.
> 
> Also, wouldn't a += instead of a append work for SYSROOT_DIRS?

It should I can change it and test.

> 
> Cheers,
> 
> Richard

BR
Piotr