mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Michael Olbrich <m.olbrich@pengutronix.de>
To: ptxdist@pengutronix.de
Subject: Re: [ptxdist] dependencies with collections
Date: Mon, 3 Sep 2012 17:20:07 +0200	[thread overview]
Message-ID: <20120903152007.GA31757@pengutronix.de> (raw)
In-Reply-To: <f1d8d5ed09009ece9d5e8ebc232b18d7@idefix.lespocky.dyndns.org>

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

      reply	other threads:[~2012-09-03 15:20 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-16 13:06 Alexander Dahl
2012-08-21  9:32 ` Michael Olbrich
2012-08-29 12:40   ` Alexander Dahl
2012-08-30  9:29   ` Alexander Dahl
2012-09-03 15:20     ` Michael Olbrich [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120903152007.GA31757@pengutronix.de \
    --to=m.olbrich@pengutronix.de \
    --cc=ptxdist@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox