From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from ptx.hi.pengutronix.de ([2001:6f8:1178:2:5054:ff:fec0:8e10]) by metis.ext.pengutronix.de with esmtp (Exim 4.72) (envelope-from ) id 1T8YRb-0007L0-5t for ptxdist@pengutronix.de; Mon, 03 Sep 2012 17:20:07 +0200 Received: from mol by ptx.hi.pengutronix.de with local (Exim 4.72) (envelope-from ) id 1T8YRb-0008He-5G for ptxdist@pengutronix.de; Mon, 03 Sep 2012 17:20:07 +0200 Date: Mon, 3 Sep 2012 17:20:07 +0200 From: Michael Olbrich Message-ID: <20120903152007.GA31757@pengutronix.de> References: <20120821093202.GA16089@pengutronix.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: Subject: Re: [ptxdist] dependencies with collections Reply-To: ptxdist@pengutronix.de List-Id: PTXdist Development Mailing List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: ptxdist-bounces@pengutronix.de Errors-To: ptxdist-bounces@pengutronix.de To: ptxdist@pengutronix.de Hi, On Thu, Aug 30, 2012 at 11:29:25AM +0200, Alexander Dahl wrote: > Am 2012-08-21 11:32, schrieb Michael Olbrich: > >> 1) There's a way to test in the make file if openssl is activated in > >> the collection and I missed it. > > > > ifneq ($(filter openssl,$(PACKAGES)),) > > ... > > endif > > > > I think. I would not recommend this. This will break if you switch > > collections without rebuilding everything. > > Also, you need to depend on openssl to make sure the file is already > > available. > > A question came to my mind: why is "ifdef PTXCONF_FOO" true in the > first place when FOO=m in the ptxconfig and FOO is not set in the > collectionconfig? If collections are used, why wouldn't you wanna check > the collectionconfig if a package is activated instead of the ptxconfig? Well historically the answer is, because it's easier and should make no difference. ptxconfig and collectionconfig are just sourced into the Makefile. 'ifdef PTXCONF_FOO' is to setup variables etc. and 'PACKAGES-$(PTXCONF_FOO) += foo' is to enable the package. In any generic code this is important, because there are platform specific packages that cannot be built, even with 'PTXCONF_FOO=y'. I had a patch at some point, that removed any PTXCONF_FOO=m lines when working with collections. I never applied it because there was no real benefit. Also, creating multiple images from different collectionconfigs in one go is on my todo list. I'm not quite sure how to implement this, but it's quite possible that the concept of one list with all packages to built will go away at some point. There will only be 'can be built' == 'ifdef PTXCONF_FOO' or 'is built' == 'something depends on install/targetinstall' Michael -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | -- ptxdist mailing list ptxdist@pengutronix.de