Hello Eric, On Wed, Sep 21, 2022 at 05:02:54PM +0000, Eric Wong wrote: > Uwe Kleine-König wrote: > > $ prove t/pop3d.t > > t/pop3d.t .. skipped: certs/ missing for t/pop3d.t, run /usr/bin/perl ./create-certs.perl in certs/ > > > t/pop3d.t (Wstat: 0 Tests: 2 Failed: 0) > > Parse errors: Bad plan. You planned 0 tests but ran 2. > > > It looks better after I called the create-certs script, but IMHO it > > should work as is and just skip the two tests as advertised? > > Oops, yes, fix at bottom. > > > After calling the create-certs script I get: > > > > t/pop3d.t .. 1/? Not enough arguments for PublicInbox::Daemon::run at blib/script/public-inbox-pop3d line 8, near "'pop3://0.0.0.0:110')" > > Execution of blib/script/public-inbox-pop3d aborted due to compilation errors. > > You need to use `prove -b' to add blib/{lib,arch} to the search > path. Omitting -b means you're using the system-wide installed > version. > > `prove -l' also works for this project for users who don't run > `make' first (uses ./lib instead of blib/...) since this project > has no XS. Ah, that makes sense! > > Then it hangs. I'm not fluent enough in Perl to fix that quickly myself > > and my today's time to look into public-inbox is running out, so I > > thought I let you know. > > No worries, I should be online more the rest of the year :> > > --------8<------- > Subject: [PATCH] t/pop3d: skip all tests if no certs are found > > This test could be written with optional OpenSSL dependencies, but > it's probably not worth it since IO::Socket::SSL seems pretty > common. > > Reported-by: Uwe Kleine-König > Link: https://public-inbox.org/meta/20220921154741.siubptwcv4463w5l@pengutronix.de/ Works great, thanks, Tested-by: Uwe Kleine-König Thanks a lot for your quick and helpful reply, Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | https://www.pengutronix.de/ |