From d876cfc5bfafa516dee55d04b50b319a22165640 Mon Sep 17 00:00:00 2001 From: Bruce Ashfield Date: Wed, 28 Jul 2021 14:22:43 -0400 Subject: global: overrides syntax conversion OEcore/bitbake are moving to use the clearer ":" as an overrides separator. This is pass one of updating the meta-virt recipes to use that syntax. This has only been minimally build/runtime tested, more changes will be required for missed overrides, or incorrect conversions Note: A recent bitbake is required: commit 75fad23fc06c008a03414a1fc288a8614c6af9ca Author: Richard Purdie Date: Sun Jul 18 12:59:15 2021 +0100 bitbake: data_smart/parse: Allow ':' characters in variable/function names It is becomming increasingly clear we need to find a way to show what is/is not an override in our syntax. We need to do this in a way which is clear to users, readable and in a way we can transition to. The most effective way I've found to this is to use the ":" charater to directly replace "_" where an override is being specified. This includes "append", "prepend" and "remove" which are effectively special override directives. This patch simply adds the character to the parser so bitbake accepts the value but maps it back to "_" internally so there is no behaviour change. This change is simple enough it could potentially be backported to older version of bitbake meaning layers using the new syntax/markup could work with older releases. Even if other no other changes are accepted at this time and we don't backport, it does set us on a path where at some point in future we could require a more explict syntax. I've tested this patch by converting oe-core/meta-yocto to the new syntax for overrides (9000+ changes) and then seeing that builds continue to work with this patch. (Bitbake rev: 0dbbb4547cb2570d2ce607e9a53459df3c0ac284) Signed-off-by: Richard Purdie Signed-off-by: Bruce Ashfield --- recipes-core/packagegroups/packagegroup-container.bb | 12 ++++++------ recipes-core/packagegroups/packagegroup-kubernetes.bb | 10 +++++----- 2 files changed, 11 insertions(+), 11 deletions(-) (limited to 'recipes-core/packagegroups') diff --git a/recipes-core/packagegroups/packagegroup-container.bb b/recipes-core/packagegroups/packagegroup-container.bb index 89b9c01c..82d4a2e7 100644 --- a/recipes-core/packagegroups/packagegroup-container.bb +++ b/recipes-core/packagegroups/packagegroup-container.bb @@ -16,32 +16,32 @@ PACKAGES = "\ # everything and the kitchen sink, used for building/installing # many things at once. -RDEPENDS_${PN} = " \ +RDEPENDS:${PN} = " \ packagegroup-lxc \ packagegroup-docker \ packagegroup-oci \ " -RDEPENDS_packagegroup-lxc = " \ +RDEPENDS:packagegroup-lxc = " \ lxc \ " -RDEPENDS_packagegroup-docker = " \ +RDEPENDS:packagegroup-docker = " \ docker \ " -RDEPENDS_packagegroup-podman = " \ +RDEPENDS:packagegroup-podman = " \ podman \ " -RDEPENDS_packagegroup-oci = " \ +RDEPENDS:packagegroup-oci = " \ virtual/runc \ oci-systemd-hook \ oci-runtime-tools \ oci-image-tools \ " -RDEPENDS_packagegroup-containerd = " \ +RDEPENDS:packagegroup-containerd = " \ virtual/containerd \ " diff --git a/recipes-core/packagegroups/packagegroup-kubernetes.bb b/recipes-core/packagegroups/packagegroup-kubernetes.bb index 237e72e8..8bfe942e 100644 --- a/recipes-core/packagegroups/packagegroup-kubernetes.bb +++ b/recipes-core/packagegroups/packagegroup-kubernetes.bb @@ -18,26 +18,26 @@ PACKAGES = "\ KUBERNETES_CRI ?= "containerd" -RDEPENDS_packagegroup-kubernetes-base = " \ +RDEPENDS:packagegroup-kubernetes-base = " \ packagegroup-${KUBERNETES_CRI} \ packagegroup-oci \ " -RDEPENDS_packagegroup-k8s-host = " \ +RDEPENDS:packagegroup-k8s-host = " \ packagegroup-kubernetes-base \ " -RDEPENDS_packagegroup-k8s-node = " \ +RDEPENDS:packagegroup-k8s-node = " \ packagegroup-kubernetes-base \ " -RDEPENDS_packagegroup-k3s-host = " \ +RDEPENDS:packagegroup-k3s-host = " \ packagegroup-kubernetes-base \ k3s-server \ k3s-cni \ " -RDEPENDS_packagegroup-k3s-node = " \ +RDEPENDS:packagegroup-k3s-node = " \ packagegroup-kubernetes-base \ k3s-agent \ k3s-cni \ -- cgit v1.2.3-54-g00ecf