diff options
| author | Quentin Schulz <quentin.schulz@theobroma-systems.com> | 2023-10-05 15:39:39 +0200 |
|---|---|---|
| committer | Steve Sakoman <steve@sakoman.com> | 2023-10-25 04:51:01 -1000 |
| commit | 7d7dc5e2b4e864ca293957955120ee1f0e5384dd (patch) | |
| tree | aa81d89b875638ea019a7c7ea803f95443213d20 /meta/classes-recipe | |
| parent | 3f2a31ddc64e86f708b866f3c96fb0b7dbad8f12 (diff) | |
| download | poky-7d7dc5e2b4e864ca293957955120ee1f0e5384dd.tar.gz | |
uboot-extlinux-config.bbclass: fix missed override syntax migration
uboot-extlinux-config allows to specify multiple "labels" (entries in a
menu, à-la grub) and each of them have their own values for some fields.
Each "base" variable, e.g. UBOOT_EXTLINUX_FDT can be overridden for each
label. This is done via the OVERRIDES mechanism based on the label name,
e.g. UBOOT_EXTLINUX_FDT:linux if linux is a label.
However, OVERRIDES doesn't contain the label globally because it's only
necessary in one task. Therefore, the OVERRIDES itself is modified
within the task. This means that the sigdata will not be told the
dependency on UBOOT_EXTLINUX_FDT:linux, because it cannot know about it.
For this reason, we need to explicitly specify which variables this task
depends on via vardeps varflag for the task.
This was done in the past, but we missed updating it during the override
syntax migration so the cache wouldn't get invalidated if someone
modifies UBOOT_EXTLINUX_FDT:linux from a configuration file or a
bbappend for example.
Let's fix this by migrating it to the new syntax.
(From OE-Core rev: 585544b99c62d3e4edebd7c4e9225f70069a5176)
Signed-off-by: Quentin Schulz <quentin.schulz@theobroma-systems.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit b4dd9d873508128adbbf5ff6cf0a3df3d2ffbcf6)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
Diffstat (limited to 'meta/classes-recipe')
| -rw-r--r-- | meta/classes-recipe/uboot-extlinux-config.bbclass | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/meta/classes-recipe/uboot-extlinux-config.bbclass b/meta/classes-recipe/uboot-extlinux-config.bbclass index 653e583663..2f5c665f97 100644 --- a/meta/classes-recipe/uboot-extlinux-config.bbclass +++ b/meta/classes-recipe/uboot-extlinux-config.bbclass | |||
| @@ -152,7 +152,7 @@ python do_create_extlinux_config() { | |||
| 152 | bb.fatal('Unable to open %s' % (cfile)) | 152 | bb.fatal('Unable to open %s' % (cfile)) |
| 153 | } | 153 | } |
| 154 | UBOOT_EXTLINUX_VARS = "CONSOLE MENU_DESCRIPTION ROOT KERNEL_IMAGE FDTDIR FDT KERNEL_ARGS INITRD" | 154 | UBOOT_EXTLINUX_VARS = "CONSOLE MENU_DESCRIPTION ROOT KERNEL_IMAGE FDTDIR FDT KERNEL_ARGS INITRD" |
| 155 | do_create_extlinux_config[vardeps] += "${@' '.join(['UBOOT_EXTLINUX_%s_%s' % (v, l) for v in d.getVar('UBOOT_EXTLINUX_VARS').split() for l in d.getVar('UBOOT_EXTLINUX_LABELS').split()])}" | 155 | do_create_extlinux_config[vardeps] += "${@' '.join(['UBOOT_EXTLINUX_%s:%s' % (v, l) for v in d.getVar('UBOOT_EXTLINUX_VARS').split() for l in d.getVar('UBOOT_EXTLINUX_LABELS').split()])}" |
| 156 | do_create_extlinux_config[vardepsexclude] += "OVERRIDES" | 156 | do_create_extlinux_config[vardepsexclude] += "OVERRIDES" |
| 157 | 157 | ||
| 158 | addtask create_extlinux_config before do_install do_deploy after do_compile | 158 | addtask create_extlinux_config before do_install do_deploy after do_compile |