On 2019/07/17 08:19, wen heping wrote: > Hi, ports@: > > Here is a patch for www/p5-Catalyst-Plugin-Session-PerUser : > 1) Fix the test by adding missing TEST_DEPENDS > www/p5-Catalyst-Plugin-Session-PerUser > 2) sort RUN_DEPENDS > 3) I am not sure whether or REVISION shoube bumped, since only > TEST_DEPENDS changed. > 4) I did not add POD_TEST to MAKE_ENV, because pod tests failed. I think > no need to fix it. > > It build well and now passed all tests on my amd64-head system. > > Comments? OK?
committed. I checked for REVISION bump - not needed for this - no change to package contents, and I did "make package" before / "make repackage" after applying the diff to ensure it didn't trip PLIST_DB. > PERMIT_PACKAGE= Yes > > -RUN_DEPENDS= www/p5-Catalyst-Plugin-Session \ > - security/p5-Catalyst-Plugin-Authentication \ > - devel/p5-Hash-Merge \ > +RUN_DEPENDS= devel/p5-Hash-Merge \ > devel/p5-Moose \ > devel/p5-MRO-Compat \ > + devel/p5-Object-Signature \ > devel/p5-namespace-autoclean \ > - devel/p5-Object-Signature > + security/p5-Catalyst-Plugin-Authentication \ > + www/p5-Catalyst-Plugin-Session > > -TEST_DEPENDS=devel/p5-Test-WWW-Mechanize-Catalyst > +TEST_DEPENDS= devel/p5-Test-WWW-Mechanize-Catalyst \ > + www/p5-Catalyst-Plugin-Session-State-Cookie > > CONFIGURE_STYLE = modinst >
