From owner-svn-ports-head@freebsd.org Fri Aug 21 18:53:36 2015 Return-Path: Delivered-To: svn-ports-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 200C69BF7F3; Fri, 21 Aug 2015 18:53:36 +0000 (UTC) (envelope-from danfe@freebsd.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:1900:2254:206c::16:87]) by mx1.freebsd.org (Postfix) with ESMTP id 1324CC91; Fri, 21 Aug 2015 18:53:36 +0000 (UTC) (envelope-from danfe@freebsd.org) Received: by freefall.freebsd.org (Postfix, from userid 1033) id 11C621021; Fri, 21 Aug 2015 18:53:36 +0000 (UTC) Date: Fri, 21 Aug 2015 18:53:36 +0000 From: Alexey Dokuchaev To: Dmitry Marakasov Cc: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: Re: svn commit: r394970 - head/devel/love07 Message-ID: <20150821185335.GA86849@FreeBSD.org> References: <201508211600.t7LG05HP050267@repo.freebsd.org> <20150821171523.GA49960@FreeBSD.org> <20150821172406.GA41937@hades.panopticon> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150821172406.GA41937@hades.panopticon> User-Agent: Mutt/1.5.23 (2014-03-12) X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 21 Aug 2015 18:53:36 -0000 On Fri, Aug 21, 2015 at 08:24:06PM +0300, Dmitry Marakasov wrote: > * Alexey Dokuchaev (danfe@FreeBSD.org) wrote: > > (Picking a random commit) I don't quite get it. Pregen'ed files like > > changes.txt and readme.txt can be just always installed into stagedir, > > no? > > So a port is built+installed once, then multiple flavors of packages > > are generated by taking needed files from the stagedir. > > > > That said, `post-install-FOO-on' target should be used when FOO bits are > > not necessarily available (e.g. because they have to be built first, and > > that step could/should be optional). > > This has nothing to do with flavors, these are targets which depend on > whether option is enabled or disabled. May I ask you to quote more accurately? It's hard to follow since I did not say anything about flavors (which, FWIW, were used purely as an example) in the second paragraph (which was essential, yet went unreplied). Apparently I did not make myself clear. I understand the "targets which depend on whether option is enabled or disabled" part; what I don't understand is the necessity of conditional installing of handful of text files whose existence does not depend on any option. After we entered staging era, we started to get rid of .if ${PORTDOCS} checks because we could use the benefit of dumping everything possible [1] into stagedir and then cherry-pick the bits we want to package (it can be one package or several depending on the options). These `post-install -DOCS-on: ' targets kind go against the idea of staging as it looks to me ATM unless I'm missing something obvious here. ./danfe [1] to a extent when some bits actually require to be built via Doxygen or other latex, which means you need to fetch ~1.6G distfile, which means you probably don't want it, or at least prefer to be asked for consent first.