On Wed, Oct 30, 2019 at 01:37:38AM -0600, Anthony J. Bentley wrote: > Stuart Henderson writes: > > > net/p5-Net-WebSocket-Server > > > > - cc'ing afresh1 for comment, but rather than listing devel/p5-Test-Pod and > > devel/p5-Test-Pod-Coverage in TEST_DEPENDS the usual approach in cpan ports > > seems to be "MAKE_ENV= TEST_POD=Yes" which adds the deps automatically > > Done.
OK afresh1@ > > > net/p5-Protocol-WebSocket > > > > - drop BUILD_DEPENDS, use CONFIGURE_STYLE = modbuild tiny > > Done. OK afresh1@ > > > security/p5-Crypt-LE > > > > - drop p5-JSON dep; it's unused; Crypt::LE diverts all JSON via > > JSON::MaybeXS which depends on Cpanel::JSON::XS and uses it preferentially Tests for this require https access to 'acme-staging.api.letsencrypt.org:443', not sure if that means we need to set "manual testing" or whatever, but tests fail for me with PORTS_PRIVSEP. OK afresh1@ if that's not a problem, or if it is fixed. > > > > - maybe worth adding a HOMEPAGE? the cpan meta just has https://do-know.com > > and there's also https://zerossl.com/, but one of these might be better .. > > https://do-know.com/guides/blog-Get-free-SSL-Certificates-with-Perl.html > > https://github.com/do-know/Crypt-LE > > Went with GitHub. > > > > security/p5-Crypt-PKCS10 OK afresh1@ > > - outdated - please bump to 2.001 (straightforward update, no big changes) > > > > - s/PERMIT_PACKAGE_CDROM/PERMIT_PACKAGE/ > > > > - definitely missing a dep on converters/p5-Convert-ASN1 > > > > - also needs either security/p5-Crypt-OpenSSL-DSA or > > security/p5-Crypt-OpenSSL-RSA or Crypt::PK::ECC (not ported) > > You must be looking at an older port, the tarball I sent had all the > changes listed here already. > > Crypt::PK::ECC is in ports, included in security/p5-CryptX. > > New tarball attached, relative to /usr/ports. > > -- > Anthony J. Bentley -- andrew - http://afresh1.com Life's unfair - but root password helps!
