svn commit: r313682 - in head/comms/openobex: . files

[Guido Falsi]

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 at FreeBSD.org>