On Mon, 13 May 2019 08:23:41 +0000 wen heping <wenheping2...@hotmail.com> wrote:
> Here is an revised patch, based on suggestions of afresh1@(Thanks !). > removed PKG_ARCH=*, add TEST_DEPENDS=devel/p5-Readonly. > > OK ? Comments ? > > wen > ________________________________ > 发件人: Andrew Hewus Fresh <and...@afresh1.com> > 发送时间: 2019年5月11日 0:30 > 收件人: wen heping > 抄送: Charlene Wendling; ports@openbsd.org > 主题: Re: 回复: [NEW] devel/p5-Ref-Util and devel/p5-Ref-Util-XS > (again) > > On Thu, May 09, 2019 at 06:36:01AM +0000, wen heping wrote: > > Hi, > > > > The new patch add openbsd tag in PLIST and add WANTLIB in > > Makefile. > > > > Comments? > > p5-Ref-Util-XS could use TEST_DEPENDS=devel/p5-Readonly, as an XS > module is not a candidate for PKG_ARCH=* so that needs to be removed, > and the COMMENT should not start with a capital letter and could > probably use a space before the open paren. > > For p5-Ref-Util, it also still needs the > TEST_DEPENDS=devel/p5-Readonly and the COMMENT should not start with > a capital letter. > > > With that, OK afresh1@ p5-Ref-Util-XS builds and pass tests on macppc. p5-Ref-Util has still inconsistent spacing in *_DEPENDS, but i'm nitpicking ;) OK cwen@ > > > > > wen > > ________________________________ > > 发件人: Charlene Wendling <juliana...@posteo.jp> > > 发送时间: 2019年5月8日 17:05 > > 收件人: wen heping > > 抄送: ports@openbsd.org > > 主题: Re: [NEW] devel/p5-Ref-Util and devel/p5-Ref-Util-XS (again) > > > > On Wed, 8 May 2019 00:58:25 +0000 > > wen heping <wenheping2...@hotmail.com> wrote: > > > > > Hi: > > > > > > Here is a patch for create new port devel/p5-Ref-Util and > > > devel/p5-Ref-Util-XS , both are needed by the future update of > > > www/p5-Dancer2. It build and pass all the tests on my amd64 > > > system on OpenBSD-current. > > > > > > I submitted it in March, this time I fix some bugs which > > > pointed by afresh1@. > > > > As far as Ref::Util::XS is concerned, it's still the old version > > you're sending, without Andrew's suggestions: > > > > https://marc.info/?l=openbsd-ports&m=155157380201369&w=2 > > > > Ref::Util would be happier with: > > > > TEST_DEPENDS= devel/p5-Readonly > > > > > OK? Comments ? > > > > > > Regards, > > > wen > > > > Charlène. > > > > -- > andrew - http://afresh1.com > > Computer Science: solving today's problems tomorrow.