On Mon, Dec 26, 2016 at 11:51:32AM -0600, attila wrote:
> 
> attila <att...@stalphonsos.com> writes:
> 
> > attila <att...@stalphonsos.com> writes:
> >
> >> Hi ports@,
> >>
> >> Attached is a port for the Data::SExpression Perl module.  It is a
> >> dependency of the Perl interface I recently contributed to mail/mu
> >> upstream, and which will be coming down the pike in some new mu
> >> release fairly soon.
> >>
> >> $ cat pkg/DESCR
> >> Data::SExpression is a Perl module that parses LISP-like
> >> S-expressions into Perl data structures.
> >>
> >> Passes make test on i386.

Also on amd64, if it is in the "cpan" directory.


> >> Comments, feedback most welcome.

There is no "cpan" category, need to pick one.  Perhaps devel, or
textproc?

Don't start off the COMMENT with a capital letter unless it is
semantically important (like the COMMENT is the expansion of an acronym
or somesuch).  

You should add yourself as maintainer if you are interested in that.

Should add a test dependency on devel/p5-Test-Memory-Cycle to avoid
skipping tests.

Some of the newlines in the Makefile can go away.

Other than that, it looks OK to me.

l8rZ,
-- 
andrew - http://afresh1.com

Full-time system administration is a delicate balance 
    between proactiveness and laziness.
                      --  jhorwitz from use.perl.org

Reply via email to