Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] Fwd:  [PATCH 1/1] ola: new package
Date: Thu, 15 Aug 2013 17:11:55 +0200	[thread overview]
Message-ID: <20130815171155.6d9be3c5@skate> (raw)
In-Reply-To: <CALy4uEWtQiLAfbe8VRQ1iJX1TQxbZ=HSTXN+De7DKumsH2sWrA@mail.gmail.com>

Dear David Skok,

On Thu, 15 Aug 2013 10:30:12 -0400, David Skok wrote:

> I checked with maintainer and karate and spi plugins build by mistake even
> with --disable to configure.  Will have to either live with that or take
> selection out and user assumes that it will be built.

Ok, then for now, remove the options for those plugins that are always
built, and only add options for plugins that can actually be
disabled/enabled.

>  The milinst plugin
> is new and I added it to the list as well as ordered all alphabetically.
>  Also found error with a plug in requiring python dependency in original
> patch and added python binding dependency comment for said plugin.
> 
> A general buildroot question...
> When plugin selections added after first build, ./configure is not re-run.
>  As such dependencies are built but not new plugins.  Is there something I
> need to put in package patch to make this work?

No, because we don't try to make this work. When an user changes the
Buildroot configuration, he has either to rebuild everything, or to
know what are the impacts of the configuration change and rebuild the
corresponding packages.

The problem is that it is almost impossible to know what is the impact
of an arbitrary change in the configuration, so we don't try to handle
this, it's part of the KISS principle.

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

      reply	other threads:[~2013-08-15 15:11 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-12 18:05 [Buildroot] [PATCH 1/1] ola: new package Dave Skok
2013-08-13 22:32 ` Thomas Petazzoni
2013-08-14  0:51   ` Blanco
     [not found]     ` <CALy4uEU8nED=gPosezp_5dK79BQ8Q18ZtogJhPComrO+kHkJZA@mail.gmail.com>
2013-08-15 14:30       ` [Buildroot] Fwd: " David Skok
2013-08-15 15:11         ` Thomas Petazzoni [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=20130815171155.6d9be3c5@skate \
    --to=thomas.petazzoni@free-electrons.com \
    --cc=buildroot@busybox.net \
    /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