Date: Sun, 10 Mar 2013 17:24:38 +0100 From: Guido Falsi <madpilot@FreeBSD.org> To: Max Brazhnikov <makc@issp.ac.ru> Cc: svn-ports-head@freebsd.org, svn-ports-all@freebsd.org, ports-committers@freebsd.org Subject: Re: svn commit: r313682 - in head/comms/openobex: . files Message-ID: <513CB3C6.7070408@FreeBSD.org> In-Reply-To: <513CA44C.3060402@FreeBSD.org> References: <201303090039.r290dh5G031351@svn.freebsd.org> <9879549.96OiXCOr5y@mercury.ph.man.ac.uk> <513CA44C.3060402@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 03/10/13 16:18, Guido Falsi wrote:
> On 03/09/13 20:48, Max Brazhnikov wrote:
>> On Sat, 09 Mar 2013 00:39:43 +0000Sat Mar 9 00:39:43 2013 Guido Falsi
>> wrote:
>>> - Add DOXYGEN option, disabled by default to keep the port slim
>>
>> I've disabled all options in doxygen recently, it's pretty light now.
>
> I noticed. I can enable this one by default without problems.
>
>>
>>> +.if ${PORT_OPTIONS:MDOXYGEN}
>>> +BUILD_DEPENDS+= doxygen:${PORTSDIR}/devel/doxygen
>>> +PORTDOCS= *
>>> .else
>>
>> PORTDOCS should only be use with DOCS option, otherwise packaging will
>> fail if
>> DOXYGEN is set and DOCS is unset.
>
> But the doxygen files are being put in DOCSDIR by this port.
>
> I also thought I could make the OPTION be DOCS instead of DOXYGEN,
> which, for this port, seems to be the same, would this be acceptable?
>
> This would also make the option enabled by default.
Please disregard the above comment. I was misunderstanding how doxygen
works. I'll fix the plist and Makefile not to use PORTDOCS. I'll also
see to make DOXYGEN the default.
Thanks for reporting!
--
Guido Falsi <madpilot@FreeBSD.org>
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?513CB3C6.7070408>