On Tue, Jun 18, 2013 at 1:35 PM, Stuart Henderson <[email protected]> wrote:
> On 2013/06/18 14:13, Roman Kravchuk wrote:
>> Thanks for comments.
>>
>> Updated port in attached.
>
> Ah, I just noticed stuntestcode; I don't think this should be installed
> in the package, but we can use it for do-test. Diff to your port, also new
> tgz attached. (Since there are now fewer programs to install, I also
> switched to a consistent style in do-install for programs and manpages).
>
> Any OKs to import this?
>
>
> diff --git Makefile Makefile
> index 14a207c..c13a7d3 100644
> --- Makefile
> +++ Makefile
> @@ -26,15 +26,14 @@ MAKE_FLAGS =
> BOOST_INCLUDE='-I${LOCALBASE}/include' \
>
> USE_GMAKE = Yes
>
> -NO_TEST = Yes
> -
> WRKDIST = ${WRKDIR}/stunserver
>
> do-install:
> -.for s in stunserver stunclient stuntestcode
> - ${INSTALL_PROGRAM} ${WRKSRC}/$s ${PREFIX}/sbin
> -.endfor
> + cd ${WRKSRC}; ${INSTALL_PROGRAM} stunserver stunclient ${PREFIX}/sbin
> cd ${WRKSRC}/resources/; ${INSTALL_MAN} stunclient.1 stunserver.1 \
> ${PREFIX}/man/man1
>
> +do-test:
> + ${WRKSRC}/stuntestcode
> +
> .include <bsd.port.mk>
> diff --git pkg/PLIST pkg/PLIST
> index 27c7ebb..43fb56e 100644
> --- pkg/PLIST
> +++ pkg/PLIST
> @@ -5,6 +5,5 @@
> @man man/man1/stunserver.1
> @bin sbin/stunclient
> @bin sbin/stunserver
> -@bin sbin/stuntestcode
> share/doc/pkg-readmes/${FULLPKGNAME}
> @rcscript ${RCDIR}/stunserver
Why "stuntman-${VERSION:S/_/./g}" for PKGNAME?
