svn commit: r366936 - head/devel/avr-libc

[Adam Weinberger]

On 1 Sep, 2014, at 23:15, Alexey Dokuchaev <danfe at FreeBSD.org> wrote:

> On Mon, Sep 01, 2014 at 07:26:04PM +0000, Adam Weinberger wrote:
>> New Revision: 366936
>> URL: http://svnweb.freebsd.org/changeset/ports/366936
>> 
>> Log:
>>  The doxygen build was failing on 8 and 9 for non-obvious reasons.
>>  As a stopgap (and as a favour to anyone building from ports ;-), change
>>  the DOCS option to DOXYGEN, and default it to off.
> 
> Thanks for bringing more sanity to ports!  Doxygen should be default to off
> everywhere indeed.

The problem with the doxygen issue is that it only negatively affects people who build from ports. It’s generally a build-time dependency only. For people installing from pkg, it’s often a couple extra files, no big deal. Defaulting doxygen to off means that those ports produce packages with no (or less) documentation.

The crucially important thing, from my perspective, is that any port that requires doxygen HAS TO say so up front! The fact that some ports hide doxygen behind the DOCS option means that I have to inspect every single port’s Makefile before I install it.

In fact, I think that portmgr should include making sure that doxygen dependencies are protected behind a DOXYGEN option instead of DOCS in the blanket just-fix-it. Even blanket permission to add “(requires doxygen)” to those ports’ DOCS_DESC would be a great start.

I’ve said it before, it is important to maintain a consistent and predictable end-user experience, and that means we need to get rid of these sorts of gotchas.

# Adam


-- 
Adam Weinberger
adamw at adamw.org
http://www.adamw.org