Re: [OE-core] [PATCH v2] libssh2: Upgrade to v1.11.0

[Alexandre Belloni <alexandre.belloni@bootlin.com>]

On 14/06/2023 21:29:56+0100, Sudip Mukherjee wrote:
> On Wed, 14 Jun 2023 at 16:47, Ross Burton <Ross.Burton@arm.com> wrote:
> >
> > On 14 Jun 2023, at 16:13, Sudip Mukherjee <sudipm.mukherjee@gmail.com> wrote:
> > >
> > > Hi Ross,
> > >
> > > On Wed, 14 Jun 2023 at 14:53, Ross Burton <ross.burton@arm.com> wrote:
> > >>
> > >> On 13 Jun 2023, at 21:52, Sudip Mukherjee via lists.openembedded.org <sudipm.mukherjee=gmail.com@lists.openembedded.org> wrote:
> > >>> # only one of openssl and gcrypt could be set
> 
> <snip>
> 
> > >>> mkdir -p ${D}${PTEST_PATH}/docs
> > >>> cp -r ${S}/docs/* ${D}${PTEST_PATH}/docs/
> > >>> }
> > >>
> > >> As you’ve clearly been looking at the tests, why does the ptest support need the documentation?
> > >
> > > There is a test for manpage syntax "mansyntax.sh" which needs the
> > > generated man pages in the docs folder.
> > > compile_ptest_base will not fail, but actual tests will fail.
> >
> > This is where I start to say that we should be trimming the tests and only running useful tests for integration purposes, not the complete upstream test suite.
> 
> Sure, and as confirmed over irc I will send a followup patch removing that test.
> 
> -- 
> Regards
> Sudip

Note that test_sshd.test is failing, full log here:
https://autobuilder.yocto.io/pub/non-release/20230614-11/testresults/qemux86-64-ptest/core-image-ptest-libssh2/log.do_testimage.50695.20230614084246

-- 
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com