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.
next prev 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