All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v2 1/1] linux: Make dtc install step more reliable
Date: Fri, 23 Nov 2018 22:29:30 +0100	[thread overview]
Message-ID: <20181123222930.41f98733@windsurf> (raw)
In-Reply-To: <20181113155016.24022-1-anaumann@ultratronik.de>

Hello,

On Tue, 13 Nov 2018 16:50:16 +0100, Andreas Naumann wrote:
> Checking for the existence of the dtc binary built by the
> non-dependent dtc package may cause instable behaviour when giving more
> freedom on the order of how the packages are built (parallelization).
> 
> In addidion, when moving to per-package host/target method, the check
> would always trigger in the isolated host, leading to linux-dtc always
> being installed as dtc.
> This in turn may lead to undesired overwriting of the real host-dtc binary
> when finally assembling the global host dir.
> 
> Thus rework the linux-dtc install condition to be defined by configuration
> rather than compile time order.
> 
> Signed-off-by: Andreas Naumann <anaumann@ultratronik.de>
> ---
> Changes v1 -> v2: 
>   - simplified by direct usage of make conditional almost as suggested by
>     Arnout (test revealed that an additinal comma was needed for correct logic)
> ---
>  linux/linux.mk | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)

I hesitated a bit, but in the end decided to apply this to master.
Indeed, with the current code, it's really a matter of luck whether
host-dtc gets built before/after linux that determines if the dtc ->
linux-dtc symlink will be created or not. With this patch, regardless
of whether host-dtc is built before or after linux, we'll have the same
result. To me, this was a sufficient motivation to apply this patch to
master.

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

  reply	other threads:[~2018-11-23 21:29 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-11-13 15:50 [Buildroot] [PATCH v2 1/1] linux: Make dtc install step more reliable Andreas Naumann
2018-11-23 21:29 ` Thomas Petazzoni [this message]
2018-11-26  7:50   ` Andreas Naumann
2018-11-26  7:58     ` Thomas Petazzoni
2018-11-26  8:14       ` Andreas Naumann
2018-11-26 17:48 ` Peter Korsgaard

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=20181123222930.41f98733@windsurf \
    --to=thomas.petazzoni@bootlin.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.