On Wed, 1 May 2019 20:49:52 -0700
Andrew Hewus Fresh <and...@afresh1.com> wrote:

> Bump
> 
> On Sat, Mar 09, 2019 at 05:49:58PM -0700, Andrew Hewus Fresh wrote:
> > I'm the MAINTAINER on this one, but could use another set of eyes.
> > This normally comes with a "package-stash-conflicts" binary that is
> > pretty useless, so I thought I would avoid installing that, which
> > means we don't need Package::Stash::Conflicts and don't need
> > Dist::CheckConflicts as a RUN_DEPENDS.

Agreed, all removals you're doing here make sense.

> > It actually builds without Dist-CheckConflicts, but the Makefile
> > throws warnings, that could be a TEST_DEPENDS though as tests fail
> > without it.
> > 
> > Comments, OK?

I've met no issues with it. OK cwen@


> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/devel/p5-Package-Stash/Makefile,v
> retrieving revision 1.11
> diff -u -p -r1.11 Makefile
> --- Makefile  20 Mar 2016 19:56:31 -0000      1.11
> +++ Makefile  10 Mar 2019 00:46:30 -0000
> @@ -4,7 +4,7 @@ COMMENT =             routines for manipulating sta
>  
>  MODULES =            cpan
>  PKG_ARCH =           *
> -DISTNAME =           Package-Stash-0.37
> +DISTNAME =           Package-Stash-0.38
>  CATEGORIES =         devel
>  
>  MAINTAINER =         Andrew Fresh <afre...@openbsd.org>
> @@ -12,11 +12,14 @@ MAINTAINER =              Andrew Fresh
> <afresh1@open
>  # perl
>  PERMIT_PACKAGE_CDROM =       Yes
>  
> -RUN_DEPENDS =        devel/p5-Dist-CheckConflicts>=0.02 \
> -             devel/p5-Module-Implementation>=0.06 \
> -             devel/p5-Package-Stash-XS
> -             
> -TEST_DEPENDS =       devel/p5-Test-Fatal>=0 \
> -             devel/p5-Test-Requires>=0
> +BUILD_DEPENDS =      devel/p5-Dist-CheckConflicts>=0.02
> +
> +RUN_DEPENDS =        devel/p5-Module-Implementation>=0.06 \
> +             devel/p5-Package-Stash-XS>=0.26
> +
> +TEST_DEPENDS =       devel/p5-Test-Fatal \
> +             devel/p5-Test-Requires
> +
> +FIX_EXTRACT_PERMISSIONS =    Yes
>  
>  .include <bsd.port.mk>
> Index: distinfo
> ===================================================================
> RCS file: /cvs/ports/devel/p5-Package-Stash/distinfo,v
> retrieving revision 1.3
> diff -u -p -r1.3 distinfo
> --- distinfo  9 Jan 2015 15:51:02 -0000       1.3
> +++ distinfo  10 Mar 2019 00:46:30 -0000
> @@ -1,2 +1,2 @@
> -SHA256 (Package-Stash-0.37.tar.gz) =
> BqsFOI+RMM03fA4dPjuv7tbvah4iEEVxqeHXv6x4eyw= -SIZE
> (Package-Stash-0.37.tar.gz) = 40309 +SHA256
> (Package-Stash-0.38.tar.gz) =
> xY7ohE3y3aOOO/Zv30Q0OarvrvGjOUDt8gVfCv0iOn8= +SIZE
> (Package-Stash-0.38.tar.gz) = 50136 Index: pkg/PLIST
> ===================================================================
> RCS file: /cvs/ports/devel/p5-Package-Stash/pkg/PLIST,v retrieving
> revision 1.4 diff -u -p -r1.4 PLIST
> --- pkg/PLIST 9 Jan 2015 15:51:02 -0000       1.4
> +++ pkg/PLIST 10 Mar 2019 00:46:30 -0000
> @@ -1,10 +1,10 @@
>  @comment $OpenBSD: PLIST,v 1.4 2015/01/09 15:51:02 afresh1 Exp $
> -bin/package-stash-conflicts
> +@comment bin/package-stash-conflicts
>  ${P5SITE}/Package/
>  ${P5SITE}/Package/Stash/
>  ${P5SITE}/Package/Stash.pm
> -${P5SITE}/Package/Stash/Conflicts.pm
> +@comment ${P5SITE}/Package/Stash/Conflicts.pm
>  ${P5SITE}/Package/Stash/PP.pm
> -@man man/man1/package-stash-conflicts.1
> +@comment @man man/man1/package-stash-conflicts.1
>  @man man/man3p/Package::Stash.3p
>  @man man/man3p/Package::Stash::PP.3p
> 

Reply via email to