svn commit: r367002 - head/devel/cmake

[John Marino]

On 9/2/2014 17:37, Adam Weinberger wrote:
> On 2 Sep, 2014, at 11:33, John Marino <freebsd.contact at marino.st> wrote:
> 
>> On 9/2/2014 17:27, Andrej Zverev wrote:
>>> +1
>>>
>>>
>>> On Tue, Sep 2, 2014 at 6:20 PM, Adam Weinberger <adamw at adamw.org
>>> <mailto:adamw at adamw.org>> wrote:
>>>
>>>    On 2 Sep, 2014, at 9:39, Raphael Kubo da Costa <rakuco at FreeBSD.org>
>>>    wrote:
>>>
>>>> Author: rakuco
>>>> Date: Tue Sep  2 13:39:32 2014
>>>> New Revision: 367002
>>>> URL: http://svnweb.freebsd.org/changeset/ports/367002
>>>> QAT: https://qat.redports.org/buildarchive/r367002/
>>>>
>>>> Modified:
>>>> head/devel/cmake/Makefile
>>>>
>>>> Modified: head/devel/cmake/Makefile
>>>>
>>>    ==============================================================================
>>>> --- head/devel/cmake/Makefile Tue Sep  2 13:25:18 2014       
>>>    (r367001)
>>>> +++ head/devel/cmake/Makefile Tue Sep  2 13:39:32 2014       
>>>    (r367002)
>>>> @@ -2,10 +2,10 @@
>>>> # $FreeBSD$
>>>> +BUILD_DEPENDS=       sphinx-build:${PORTSDIR}/textproc/py-sphinx
>>>
>>>    I’m sorry, but that is an absolutely ludicrous number of
>>>    dependencies. Can you please make an OPTION to disable this? I don’t
>>>    want to install a dozen dependencies just to get some manpages I’ll
>>>    never read.
>>>
>>
>> I understand that installing man pages is mandatory, that it should not
>> be OPTION controlled.  A lot of ports use sphinx so I don't know what
>> the big deal about sphinx in particular is, but lets say it's something
>> else far worse.   man pages could be pregenerated and installed from
>> $FILEDIR right?  So there are alternatives, but unless I'm wrong about
>> the policy using OPTIONS is not one of them (but I'm wrong a lot, so
>> let's see what the answer is).
> 
> How do you mean mandatory?
> 
> OPTIONS_DEFINE+=	MANPAGES
> OPTIONS_DEFAULT+=	MANPAGES
> MANPAGES_CONFIGURE_ON=	--sphinx-man
> 

By mandatory, I mean that a port that does that is violating policy.
Documentation is optional (DOCS) but manpages are not.  That's what I've
understood.  I've see ports that tried the above and I've removed code
of that equivalent.

John