On 13 Jun at 20:45, Andrew Hewus Fresh <and...@afresh1.com> wrote:
> On Sun, Jun 13, 2021 at 03:40:22PM +0200, Mischa wrote:
> > This module implement JSON-RPC 2.0 protocol in transport-independent
> > way.
> > 
> > Also it support non-blocking client remote procedure call and both
> > blocking and non-blocking server method execution. This can be very
> > useful in case server methods will need to do some RPC or other
> > slow things like network I/O, which can be done in parallel with
> > executing other server methods in any event-based environment.
> > 
> > meta::cpan: https://metacpan.org/pod/JSON::RPC2
> > 
> > JSON RPC is for example needed to use the eAPI from Arista.
> > 
> > Any comments / suggestions welcome!
> 
> Most of the other JSON perl modules are in converters, not devel.  This
> one should probably go there as well.

That was my first instict as well, until I checked some of the other
JSON RPC packages and they are all in devel, so was a little
conflicted. Using converters now as it does make more sense considering
the other JSON Perl modules.

> The COMMENT starts with a capital letter, it shouldn't.
> See item #16: http://www.openbsd.org/faq/ports/guide.html#PortsChecklist

Got it!

> Usually perl ports don't need to set BUILD_DEPENDS=${RUN_DEPENDS},
> and in this case that seems to be true.  It doesn't hurt anything, just
> makes bulk builds take slightly longer than than need to.

Argh... Artifact from me working around the errors I got.
Removed.

> Something was telling the truth though, as we do need a RUN_DEPENDS on
> p5-JSON-Maybe-XS instead of p5-JSON-XS because as it is, `make test`
> fails if that isn't installed.

Reverted to what it was orginally.

> Other than those nits, it looks good to me.

Great! :)

> > Mischa
> > 
> > PS: Thank you Andrew for your guidance so far!
> 
> You're welcome!
> 
> I did commit a fix so portgen should properly set the CONFIGURE_STYLE if
> p5-Module-Build-Tiny isn't installed and needs to be.  Thanks for
> reporting that!

Nice!

New tgz attached.

Mischa

> 
> 
> l8rZ,
> -- 
> andrew - http://afresh1.com
> 
> Real programmers don't document.
>           If it was hard to write, it should be hard to understand.
> 

Attachment: p5-JSON-RPC2.tgz
Description: application/tar-gz

Reply via email to