svn commit: r394782 - head/games/dangen

Mathieu Arnold mat at FreeBSD.org
Wed Aug 19 16:38:17 UTC 2015


+--On 19 août 2015 18:55:43 +0300 Dmitry Marakasov <amdmi3 at amdmi3.ru>
wrote:
| * Mathieu Arnold (mat at FreeBSD.org) wrote:
|> +--On 19 août 2015 13:58:07 +0000 Dmitry Marakasov <amdmi3 at FreeBSD.org>
|> wrote:
|> |  
|> | -.include <bsd.port.options.mk>
|> | -
|> |  post-install:
|> | -.if ${PORT_OPTIONS:MDOCS}
|> |  	${MKDIR} ${STAGEDIR}${DOCSDIR}
|> |  	${INSTALL_DATA} ${WRKSRC}/README ${STAGEDIR}${DOCSDIR}/
|> | -.endif
|> 
|> This bit was not part of the commit message, which is bad :-)
| 
| I thought of it as too minor to mention.
| 
|> Having said that, you can now use the post-install-DOCS-on target in
|> there, it will only run when the DOCS option is enabled.
| 
| Oh, thanks for reminding! I guess I'll adopt more of that in my
| ports. Could you suggest a meaningful commit message for such
| changes? "Use conditional targets"?

Use option helpers, or use option specific target, see what I did in
r394778.

-- 
Mathieu Arnold
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 949 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/svn-ports-all/attachments/20150819/0726d76c/attachment.bin>


More information about the svn-ports-all mailing list