svn commit: r367212 - in head: astro/planets devel/aifad ftp/ocaml-ocurl

[John Marino]

On 9/3/2014 17:25, Antoine Brodin wrote:
> Author: antoine
> Date: Wed Sep  3 15:25:16 2014
> New Revision: 367212
> URL: http://svnweb.freebsd.org/changeset/ports/367212
> QAT: https://qat.redports.org/buildarchive/r367212/
> 
> Log:
>   Fix packaging
> 
> Added:
>   head/astro/planets/pkg-plist   (contents, props changed)
>   head/devel/aifad/pkg-plist   (contents, props changed)
> Modified:
>   head/astro/planets/Makefile
>   head/devel/aifad/Makefile
>   head/ftp/ocaml-ocurl/Makefile
>   head/ftp/ocaml-ocurl/pkg-plist
> 
> Modified: head/astro/planets/Makefile
> ==============================================================================
> --- head/astro/planets/Makefile	Wed Sep  3 15:14:22 2014	(r367211)
> +++ head/astro/planets/Makefile	Wed Sep  3 15:25:16 2014	(r367212)
> @@ -20,15 +20,11 @@ ALL_TARGET=	${PORTNAME}
>  
>  OPTIONS_DEFINE=	DOCS
>  
> -PORTDOCS=	KEYBINDINGS.txt getting_started.html
> -PLIST_FILES=	bin/planets man/man1/planets.1.gz
> -
>  do-install:
>  	${INSTALL_PROGRAM} ${WRKSRC}/${PORTNAME} ${STAGEDIR}${PREFIX}/bin
>  	${INSTALL_MAN} ${WRKSRC}/planets.1 ${STAGEDIR}${MANPREFIX}/man/man1
>  	@${MKDIR} ${STAGEDIR}${DOCSDIR}
> -.for doc in ${PORTDOCS}
> -	${INSTALL_DATA} ${WRKSRC}/${doc} ${STAGEDIR}${DOCSDIR}
> -.endfor
> +	${INSTALL_DATA} ${WRKSRC}/KEYBINDINGS.txt ${STAGEDIR}${DOCSDIR}
> +	${INSTALL_DATA} ${WRKSRC}/getting_started.html ${STAGEDIR}${DOCSDIR}
>  
>  .include <bsd.port.mk>
> 
> Added: head/astro/planets/pkg-plist
> ==============================================================================
> --- /dev/null	00:00:00 1970	(empty, because file is newly added)
> +++ head/astro/planets/pkg-plist	Wed Sep  3 15:25:16 2014	(r367212)
> @@ -0,0 +1,5 @@
> +bin/planets
> +man/man1/planets.1.gz
> +%%PORTDOCS%%%%DOCSDIR%%/KEYBINDINGS.txt
> +%%PORTDOCS%%%%DOCSDIR%%/getting_started.html
> +%%PORTDOCS%%@dirrm %%DOCSDIR%%




Can you explain what was broken?
You keep saying you are "fixing packaging" but I don't see anything what
was wrong.  Especially this one that introduces pkg-plist again when
we've been told to remove it if it's <= 5 files.