Rafael Sadowski <raf...@sizeofvoid.org> wrote:

>- update to 1.3.1
>- switch to MIT LICENSE
>- install LICENSE and man page

OK czarkoff@ with several tweaks below.

>index: Makefile
>===================================================================
>RCS file: /cvs/ports/graphics/cadubi/Makefile,v
>retrieving revision 1.14
>diff -u -p -u -p -r1.14 Makefile
>--- Makefile   12 May 2016 14:00:13 -0000      1.14
>+++ Makefile   30 Sep 2016 14:40:22 -0000
>@@ -1,15 +1,18 @@
> # $OpenBSD: Makefile,v 1.14 2016/05/12 14:00:13 espie Exp $
> 
>-DISTNAME=     cadubi-1.3
>-REVISION =    1
>+COMMENT=      ASCII drawing utility
>+V=            1.3.1
>+DISTNAME=     cadubi-${V}
>+GH_ACCOUNT =  statico
>+GH_PROJECT =  cadubi
>+GH_TAGNAME =  v${V}
>+

You don't really need DISTNAME and V:

  $ grep -E '^DISTNAME.+GH' /usr/ports/infrastructure/mk/bsd.port.mk
  DISTNAME ?=     ${GH_PROJECT}-${GH_TAGNAME:C/^v//}

So all you need is:

| GH_ACCOUNT =  statico
| GH_PROJECT =  cadubi
| GH_TAGNAME =  v1.3.1

> CATEGORIES=   graphics editors
> 
>-# perl artistic
>-PERMIT_PACKAGE_CDROM= Yes
>+HOMEPAGE=     https://github.com/statico/cadubi
> 
>-COMMENT=      ASCII drawing utility
>-HOMEPAGE=     http://langworth.com/cadubi.html
>-MASTER_SITES= http://langworth.com/downloads/
>+# MIT
>+PERMIT_PACKAGE_CDROM= Yes
> 
> NO_BUILD=     Yes
> NO_TEST=      Yes
>@@ -18,6 +21,8 @@ PKG_ARCH=    *
> do-install:
>  ${INSTALL_SCRIPT} ${WRKSRC}/cadubi ${PREFIX}/bin
>  @mkdir ${PREFIX}/share/doc/cadubi

This should be "${INSTALL_DATA_DIR} ${PREFIX}/share/doc/cadubi"

>-      ${INSTALL_DATA} ${WRKSRC}/help.txt ${PREFIX}/share/doc/cadubi
>+      ${INSTALL_DATA} ${WRKSRC}/{help.txt,LICENSE} \
>+              ${PREFIX}/share/doc/cadubi
>+      ${INSTALL_MAN} ${WRKSRC}/cadubi.1 ${PREFIX}/man/man1
> 
> .include <bsd.port.mk>

Reply via email to