Re: www/p5-Catalyst-Plugin-Session-PerUser : Fix the test

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
>