From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCHv5 1/2] pkg-generic: detect incorrectly used package
Date: Tue, 29 Dec 2015 19:36:29 +0100 [thread overview]
Message-ID: <20151229193629.47b6dc3f@free-electrons.com> (raw)
In-Reply-To: <1444382106-16019-2-git-send-email-thomas.petazzoni@free-electrons.com>
Hello,
On Fri, 9 Oct 2015 11:15:05 +0200, Thomas Petazzoni wrote:
> In Buildroot, the selection of a package from a Config.in level and
> from a Makefile level are completely disconnected. This can lead to
> issues where the build of a package is triggered at the Makefile level
> due to the package being listed in another package <pkg>_DEPENDENCIES
> variable, even if that package is not enabled in the configuration.
>
> This has for example been the case recently with python-can having
> 'python' in its <pkg>_DEPENDENCIES, while python-can could be enabled
> when Python 3.x is used, in which case the 'python' package should not
> be built.
>
> To detect such issues more easily, this patch adds a check in the
> package infrastructure. When the build process of a package is being
> triggered, we verify that the package is enabled in the
> configuration. We do this check in the "configure" step, since this
> step is the first common step between the normal download case and the
> "local site method" / "package override" case.
>
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> ---
> Changes since v4:
> - Use $($(PKG)_KCONFIG_VAR) and $($(PKG)_TYPE) as suggested by Arnout.
> - Split the "potential culprit" mechanism into a separate commit, as
> suggested by Yann.
I've applied, after taking into account the comment from Yann (and
fixing a minor issue with it: $(1) cannot be used, it should have been
$($(PKG)_NAME)).
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
next prev parent reply other threads:[~2015-12-29 18:36 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-09 9:15 [Buildroot] [PATCHv5 0/2] Detect incorrect dependencies Thomas Petazzoni
2015-10-09 9:15 ` [Buildroot] [PATCHv5 1/2] pkg-generic: detect incorrectly used package Thomas Petazzoni
2015-12-29 18:19 ` Yann E. MORIN
2015-12-29 18:36 ` Thomas Petazzoni [this message]
2015-10-09 9:15 ` [Buildroot] [PATCHv5 2/2] pkg-generic: improve incorrectly used package detection Thomas Petazzoni
2015-12-29 18:06 ` Yann E. MORIN
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=20151229193629.47b6dc3f@free-electrons.com \
--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