Page MenuHomeFreeBSD

New port: devel/p5-MooseX-MungeHas

Authored by kai on Apr 9 2019, 10:40 AM.



MooseX::MungeHas alters the behaviour of the attributes of your Moo, Moose or
Mouse based class.

It manages to support all three because it doesn't attempt to do anything smart
with metathingies; it simply installs a wrapper for has that munges the
attribute specification hash before passing it on to the original has function.


PR: 237109
Submitted by: Geoffroy Desvernay
Approved by: (one of the mentors)

Test Plan
  • poudriere (11.2-, 12.0-RELEASE, 13.0-CURRENTr345355 amd64 + i386) -> OK
  • portlint -> OK
  • Runtime tests -> OK

Test results with "make test" yields"

t/000-report-versions-tiny.t .. ok
t/basic.t ..................... ok
All tests successful.
Files=3, Tests=6, 1 wallclock secs ( 0.03 usr 0.01 sys + 0.52 cusr 0.09 csys = 0.66 CPU)
Result: PASS

Minor fixes to the original patch:

  • Changed MAINTAINER to submitter's address
  • Added _LICENSE_FILE for ART10 and GPLv1+
  • Sorted _DEPENDS according to the PHB
  • Merged pkg-plist into PLIST_FILES
  • Improved pkg-descr a bit by realigning it to the 80 char width and added WWW entry

Diff Detail

No Linters Available
No Unit Test Coverage
Build Status
Buildable 23657
Build 22638: arc lint + arc unit

Event Timeline

This needs to be hooked into devel/Makefile

This revision now requires changes to proceed.Apr 13 2019, 7:22 AM

^ no, this is not doing what you want:

> make -VBUILD_DEPENDS -C devel/p5-MooseX-MungeHas
p5-Moose>=1.10:devel/p5-Moose perl5>=5.28.r1<5.29:lang/perl5.28 perl5>=5.28.r1<5.29:lang/perl5.28

The porters handbook also documents this (for the other way around). Here it's not that harmful...

As it is only one depend, write it twice :)


I take it back, apparently it is fine by the commit message of rP474242

Fully write out BUILD_DEPENDS instead referencing to ${RUN_DEPENDS}

kai marked 2 inline comments as done.Apr 13 2019, 10:41 AM
This revision was not accepted when it landed; it landed in state Needs Review.Apr 15 2019, 9:13 AM
This revision was automatically updated to reflect the committed changes.