public inbox for isar-users@googlegroups.com
 help / color / mirror / Atom feed
From: Anton Mikanovich <amikan@ilbers.de>
To: "Moessbauer, Felix" <felix.moessbauer@siemens.com>,
	"isar-users@googlegroups.com" <isar-users@googlegroups.com>
Cc: "Bezdeka, Florian" <florian.bezdeka@siemens.com>,
	"Kiszka, Jan" <jan.kiszka@siemens.com>,
	"Schild, Henning" <henning.schild@siemens.com>
Subject: Re: Conditional weak assignments behave unexpectedly
Date: Thu, 27 Apr 2023 09:58:00 +0300	[thread overview]
Message-ID: <1e4d6904-c554-4977-7fa4-a4bd516dfd6d@ilbers.de> (raw)
In-Reply-To: <989e781863ed449603493e574cce32581f69b292.camel@siemens.com>

23/04/2023 10:48, 'Moessbauer, Felix' via isar-users wrote:
> Dear developers,
>
> I just stumbled upon the following pattern, which needs further
> investigation:
>
> - use the qemuamd64 machine on debian-bullseye
> - define a custom WKS_FILE in local.conf
>
> While the qemuamd64.conf file contains the following line:
> WKS_FILE:debian-bullseye ?= "sdimage-efi-btrfs"
> I expected that I'm able to override that unconditionally in the
> local.conf using: WKS_FILE = "foo.wks"
>
> However, the final value of the WKS_FILE variable is "sdimage-efi-
> btrfs". In the bitbake documentation, I cannot find any example of
> combining weak assignments and overrides. However, the implementation
> seems to evaluate the weak assignment on the override first. Then, the
> WKS_FILE part from the local conf is evaluated and finally the
> WKS_FILE:debian-bullseye overrides what was in WKS_FILE. This suspicion
> is also backed by the following test:
>
> local.conf: WKS_FILE:debian-bullseye = "foo.wks"
> Now, the final value of WKS_FILE is "foo.wks".
>
> I'm unsure if that has ever been like that and was just unnoticed, or
> if the behavior change was introduced in bitbake 2.0.
> Anyways, we must not use overrides in combination with weak
> assignments. And we also need to fix that on the qemuamd64 machine
> config immediately (even if we loose some tests by doing that).
> Otherwise it is impossible to override the WKS_FILE of the qemuamd64
> machine.
>
> Best regards,
> Felix
>
Hello Felix,

As Henning already mentioned its all about CI usage.
We are already working on a solution that will move all that corner 
cases from
the layer to CI itself, so we can remove overrides.
This solution requires some testing and then I hope I'll send it to 
maillist.


  parent reply	other threads:[~2023-04-27  6:58 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-04-23  7:48 Moessbauer, Felix
2023-04-23  9:11 ` Moessbauer, Felix
2023-04-24 13:28   ` Henning Schild
2023-04-27  6:58 ` Anton Mikanovich [this message]
2023-04-28 19:12 ` Anton Mikanovich

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=1e4d6904-c554-4977-7fa4-a4bd516dfd6d@ilbers.de \
    --to=amikan@ilbers.de \
    --cc=felix.moessbauer@siemens.com \
    --cc=florian.bezdeka@siemens.com \
    --cc=henning.schild@siemens.com \
    --cc=isar-users@googlegroups.com \
    --cc=jan.kiszka@siemens.com \
    /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