Date: Tue, 2 Sep 2014 03:54:12 +0000 From: Alexey Dokuchaev <danfe@FreeBSD.org> To: Adam Weinberger <adamw@adamw.org> Cc: svn-ports-head@freebsd.org, svn-ports-all@freebsd.org, Adam Weinberger <adamw@FreeBSD.org>, ports-committers@freebsd.org Subject: Re: svn commit: r366936 - head/devel/avr-libc Message-ID: <20140902035412.GA66317@FreeBSD.org> In-Reply-To: <A2778CE6-D08B-4A56-86B1-9034DDA702CF@adamw.org> References: <201409011926.s81JQ4IV011890@svn.freebsd.org> <20140902031509.GB51270@FreeBSD.org> <A2778CE6-D08B-4A56-86B1-9034DDA702CF@adamw.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, Sep 01, 2014 at 11:35:52PM -0400, Adam Weinberger wrote: > On 1 Sep, 2014, at 23:15, Alexey Dokuchaev <danfe@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. If maintainer insists on having generated docs in the official packages, they can always do something like this: .if defined(BATCH) || defined(PACKAGE_BUILDING) OPTIONS_DEFAULT+= DOXYGEN .endif Then they would be happy and people who install from the ports tree (the old way) would not be upset by default option as well. > 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. Ditto. I cannot emphasize more on how annoying this manual checking is. ./danfe
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20140902035412.GA66317>