* [PATCH v3 1/4] oelib: utils: Support filtering default features
2026-04-01 17:41 [PATCH v3 0/4] Support opt-out of any default machine and distro features Paul Barker
@ 2026-04-01 17:41 ` Paul Barker
2026-04-01 17:41 ` [PATCH v3 2/4] meta: Support opting out of any distro features Paul Barker
` (2 subsequent siblings)
3 siblings, 0 replies; 10+ messages in thread
From: Paul Barker @ 2026-04-01 17:41 UTC (permalink / raw)
To: openembedded-core; +Cc: Paul Barker
The new library function filter_default_features() allows us to support
opting out of any default features without having to use :remove.
Signed-off-by: Paul Barker <paul@pbarker.dev>
---
meta/lib/oe/utils.py | 34 ++++++++++++++++++++++++++
meta/lib/oeqa/selftest/cases/oelib/utils.py | 38 ++++++++++++++++++++++++++++-
2 files changed, 71 insertions(+), 1 deletion(-)
diff --git a/meta/lib/oe/utils.py b/meta/lib/oe/utils.py
index afcfeda0c6d5..52f2f1bd55e0 100644
--- a/meta/lib/oe/utils.py
+++ b/meta/lib/oe/utils.py
@@ -83,6 +83,31 @@ def set_intersect(variable1, variable2, d):
val2 = set(d.getVar(variable2).split())
return " ".join(val1 & val2)
+def set_difference(variable1, variable2, d):
+ """
+ Expand both variables, interpret them as lists of strings, and return the
+ intersection as a flattened string.
+
+ For example:
+ s1 = "a b c"
+ s2 = "b c d"
+ s3 = set_difference(s1, s2)
+ => s3 = "a"
+ """
+ val1 = d.getVar(variable1)
+ if not val1:
+ return ""
+ val2 = d.getVar(variable2)
+ if not val2:
+ return val1
+
+ val1 = set(val1.split())
+ val2 = set(val2.split())
+
+ # Return a sorted string to ensure that the result is consistent between
+ # parser runs.
+ return " ".join(sorted(val1 - val2))
+
def prune_suffix(var, suffixes, d):
# See if var ends with any of the suffixes listed and
# remove it if found
@@ -133,6 +158,15 @@ def features_backfill(var,d):
if addfeatures:
d.appendVar(var, " " + " ".join(addfeatures))
+def filter_default_features(varname, d):
+ # Process default features to exclude features which the user has opted out
+ # of. The result is appended to the target variable (e.g. DISTRO_FEATURES
+ # or MACHINE_FEATURES).
+ default_features = set_difference(varname + "_DEFAULTS",
+ varname + "_OPTED_OUT",
+ d)
+ d.appendVar(varname, " " + default_features)
+
def all_distro_features(d, features, truevalue="1", falsevalue=""):
"""
Returns truevalue if *all* given features are set in DISTRO_FEATURES,
diff --git a/meta/lib/oeqa/selftest/cases/oelib/utils.py b/meta/lib/oeqa/selftest/cases/oelib/utils.py
index 0cb46425a02d..a72a0f5983f1 100644
--- a/meta/lib/oeqa/selftest/cases/oelib/utils.py
+++ b/meta/lib/oeqa/selftest/cases/oelib/utils.py
@@ -8,7 +8,7 @@ import sys
from unittest.case import TestCase
from contextlib import contextmanager
from io import StringIO
-from oe.utils import packages_filter_out_system, trim_version, multiprocess_launch
+from oe.utils import packages_filter_out_system, trim_version, multiprocess_launch, filter_default_features
class TestPackagesFilterOutSystem(TestCase):
def test_filter(self):
@@ -102,3 +102,39 @@ class TestMultiprocessLaunch(TestCase):
with captured_output() as (out, err):
self.assertRaises(bb.BBHandledException, multiprocess_launch, testfunction, ["1", "2", "3", "4", "5", "6"], d, extraargs=(d,))
self.assertIn("KeyError: 'Invalid number 2'", out.getvalue())
+
+
+class TestDefaultFeatures(TestCase):
+ def test_filter_default_features(self):
+ try:
+ import bb
+ d = bb.data_smart.DataSmart()
+ except ImportError:
+ self.skipTest("Cannot import bb")
+
+ # Test with nothing opted out
+ d.setVar("DISTRO_FEATURES", "")
+ d.setVar("DISTRO_FEATURES_DEFAULTS", "alpha beta gamma")
+ filter_default_features("DISTRO_FEATURES", d)
+ self.assertEqual(d.getVar("DISTRO_FEATURES").strip(), "alpha beta gamma")
+
+ # opt out of a single feature
+ d.setVar("DISTRO_FEATURES", "")
+ d.setVar("DISTRO_FEATURES_DEFAULTS", "alpha beta gamma")
+ d.setVar("DISTRO_FEATURES_OPTED_OUT", "beta")
+ filter_default_features("DISTRO_FEATURES", d)
+ self.assertEqual(d.getVar("DISTRO_FEATURES").strip(), "alpha gamma")
+
+ # opt out of everything
+ d.setVar("DISTRO_FEATURES", "")
+ d.setVar("DISTRO_FEATURES_DEFAULTS", "alpha beta gamma")
+ d.setVar("DISTRO_FEATURES_OPTED_OUT", "gamma alpha beta")
+ filter_default_features("DISTRO_FEATURES", d)
+ self.assertEqual(d.getVar("DISTRO_FEATURES").strip(), "")
+
+ # opt out of something that isn't in our defaults
+ d.setVar("DISTRO_FEATURES", "")
+ d.setVar("DISTRO_FEATURES_DEFAULTS", "alpha beta gamma")
+ d.setVar("DISTRO_FEATURES_OPTED_OUT", "omega")
+ filter_default_features("DISTRO_FEATURES", d)
+ self.assertEqual(d.getVar("DISTRO_FEATURES").strip(), "alpha beta gamma")
--
2.43.0
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH v3 2/4] meta: Support opting out of any distro features
2026-04-01 17:41 [PATCH v3 0/4] Support opt-out of any default machine and distro features Paul Barker
2026-04-01 17:41 ` [PATCH v3 1/4] oelib: utils: Support filtering default features Paul Barker
@ 2026-04-01 17:41 ` Paul Barker
2026-04-22 14:45 ` [OE-core] " Jose Quaresma
[not found] ` <18A8B511426C8EDA.1516946@lists.openembedded.org>
2026-04-01 17:41 ` [PATCH v3 3/4] meta: Support opting out of any machine features Paul Barker
2026-04-01 17:41 ` [PATCH v3 4/4] lib: oe: Drop backfill support Paul Barker
3 siblings, 2 replies; 10+ messages in thread
From: Paul Barker @ 2026-04-01 17:41 UTC (permalink / raw)
To: openembedded-core; +Cc: Paul Barker
Add default values to DISTRO_FEATURES using the new
filter_default_features() function.
This change obsoletes the variables DISTRO_FEATURES_BACKFILL and
DISTRO_FEATURES_BACKFILL_CONSIDERED. Instead, all defaults are added via
DISTRO_FEATURES_DEFAULTS and users can opt out of any of these using
DISTRO_FEATURES_OPTED_OUT. Hopefully the variable naming here is easier
for people to understand and remember.
Migration notes:
- If you have previously assigned DISTRO_FEATURES without using
DISTRO_FEATURES_DEFAULT, you will now get the default features added
automatically. You will need to review these and add any features you
do not want to use to DISTRO_FEATURES_OPTED_OUT.
- DISTRO_FEATURES_DEFAULT is now unused, the new variable name is
slighlty different to ensure that it is not accidentally used if a
layer hasn't been modified to adapt to the new naming.
- If you previously set DISTRO_FEATURES_BACKFILL_CONSIDERED, use the new
variable DISTRO_FEATURES_OPTED_OUT instead.
- If you previously modified DISTRO_FEATURES_BACKFILL, don't do that.
Signed-off-by: Paul Barker <paul@pbarker.dev>
---
meta/classes-global/base.bbclass | 5 ++++-
meta/classes-recipe/crosssdk.bbclass | 8 ++++----
meta/classes-recipe/native.bbclass | 8 ++++----
meta/classes-recipe/nativesdk.bbclass | 8 ++++----
meta/conf/bitbake.conf | 1 -
meta/conf/distro/include/default-distrovars.inc | 15 +++++++++------
meta/conf/distro/include/tclibc-musl.inc | 2 +-
meta/conf/distro/include/tclibc-newlib.inc | 2 +-
meta/conf/distro/include/tclibc-picolibc.inc | 2 +-
meta/conf/documentation.conf | 4 ++--
10 files changed, 30 insertions(+), 25 deletions(-)
diff --git a/meta/classes-global/base.bbclass b/meta/classes-global/base.bbclass
index 22b427a5211b..0a4cfd594c72 100644
--- a/meta/classes-global/base.bbclass
+++ b/meta/classes-global/base.bbclass
@@ -454,8 +454,11 @@ def set_packagetriplet(d):
python () {
import string, re
+ # Filter default features to allow users to opt out of features they don't
+ # want.
+ oe.utils.filter_default_features("DISTRO_FEATURES", d)
+
# Handle backfilling
- oe.utils.features_backfill("DISTRO_FEATURES", d)
oe.utils.features_backfill("MACHINE_FEATURES", d)
# To add a recipe to the skip list , set:
diff --git a/meta/classes-recipe/crosssdk.bbclass b/meta/classes-recipe/crosssdk.bbclass
index a2853e6a9201..dac18b39edb2 100644
--- a/meta/classes-recipe/crosssdk.bbclass
+++ b/meta/classes-recipe/crosssdk.bbclass
@@ -16,13 +16,13 @@ PACKAGE_ARCH = "${SDK_ARCH}"
python () {
# set TUNE_PKGARCH to SDK_ARCH
d.setVar('TUNE_PKGARCH', d.getVar('SDK_ARCH'))
- # Set features here to prevent appends and distro features backfill
- # from modifying nativesdk distro features
+ # Set features here to prevent DISTRO_FEATURES modifications from affecting
+ # crosssdk distro features
features = set(d.getVar("DISTRO_FEATURES_NATIVESDK").split())
- oe.utils.features_backfill("DISTRO_FEATURES", d)
+ oe.utils.filter_default_features("DISTRO_FEATURES", d)
filtered = set(bb.utils.filter("DISTRO_FEATURES", d.getVar("DISTRO_FEATURES_FILTER_NATIVESDK"), d).split())
d.setVar("DISTRO_FEATURES", " ".join(sorted(features | filtered)))
- d.setVar("DISTRO_FEATURES_BACKFILL", "")
+ d.setVar("DISTRO_FEATURES_DEFAULTS", "")
}
STAGING_BINDIR_TOOLCHAIN = "${STAGING_DIR_NATIVE}${bindir_native}/${TARGET_ARCH}${TARGET_VENDOR}-${TARGET_OS}"
diff --git a/meta/classes-recipe/native.bbclass b/meta/classes-recipe/native.bbclass
index 5aa9c8e4145e..9f4ca170f762 100644
--- a/meta/classes-recipe/native.bbclass
+++ b/meta/classes-recipe/native.bbclass
@@ -126,13 +126,13 @@ python native_virtclass_handler () {
return
bpn = d.getVar("BPN")
- # Set features here to prevent appends and distro features backfill
- # from modifying native distro features
+ # Set features here to prevent DISTRO_FEATURES modifications from affecting
+ # native distro features
features = set(d.getVar("DISTRO_FEATURES_NATIVE").split())
- oe.utils.features_backfill("DISTRO_FEATURES", d)
+ oe.utils.filter_default_features("DISTRO_FEATURES", d)
filtered = set(bb.utils.filter("DISTRO_FEATURES", d.getVar("DISTRO_FEATURES_FILTER_NATIVE"), d).split())
d.setVar("DISTRO_FEATURES", " ".join(sorted(features | filtered)))
- d.setVar("DISTRO_FEATURES_BACKFILL", "")
+ d.setVar("DISTRO_FEATURES_DEFAULTS", "")
classextend = d.getVar('BBCLASSEXTEND') or ""
if "native" not in classextend:
diff --git a/meta/classes-recipe/nativesdk.bbclass b/meta/classes-recipe/nativesdk.bbclass
index 5adb7515bf14..a4801c48b2de 100644
--- a/meta/classes-recipe/nativesdk.bbclass
+++ b/meta/classes-recipe/nativesdk.bbclass
@@ -78,13 +78,13 @@ python nativesdk_virtclass_handler () {
if not (pn.endswith("-nativesdk") or pn.startswith("nativesdk-")):
return
- # Set features here to prevent appends and distro features backfill
- # from modifying nativesdk distro features
+ # Set features here to prevent DISTRO_FEATURES modifications from affecting
+ # nativesdk distro features
features = set(d.getVar("DISTRO_FEATURES_NATIVESDK").split())
- oe.utils.features_backfill("DISTRO_FEATURES", d)
+ oe.utils.filter_default_features("DISTRO_FEATURES", d)
filtered = set(bb.utils.filter("DISTRO_FEATURES", d.getVar("DISTRO_FEATURES_FILTER_NATIVESDK"), d).split())
d.setVar("DISTRO_FEATURES", " ".join(sorted(features | filtered)))
- d.setVar("DISTRO_FEATURES_BACKFILL", "")
+ d.setVar("DISTRO_FEATURES_DEFAULTS", "")
e.data.setVar("MLPREFIX", "nativesdk-")
e.data.setVar("PN", "nativesdk-" + e.data.getVar("PN").replace("-nativesdk", "").replace("nativesdk-", ""))
diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index 6b3443695a04..f851e2adc9f7 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -912,7 +912,6 @@ DISTRO_FEATURES_NATIVESDK:mingw32 = "x11 ipv6"
DISTRO_FEATURES_FILTER_NATIVE ?= "api-documentation debuginfod opencl opengl wayland"
DISTRO_FEATURES_FILTER_NATIVESDK ?= "api-documentation debuginfod opencl opengl wayland"
-DISTRO_FEATURES_BACKFILL = "pulseaudio gobject-introspection-data ldconfig opengl ptest multiarch wayland vulkan"
MACHINE_FEATURES_BACKFILL = "rtc qemu-usermode"
COMBINED_FEATURES = "${@oe.utils.set_intersect('DISTRO_FEATURES', 'MACHINE_FEATURES', d)}"
diff --git a/meta/conf/distro/include/default-distrovars.inc b/meta/conf/distro/include/default-distrovars.inc
index 7adcdfad4ff4..daa4d2a74dda 100644
--- a/meta/conf/distro/include/default-distrovars.inc
+++ b/meta/conf/distro/include/default-distrovars.inc
@@ -14,19 +14,22 @@ LOCALE_UTF8_IS_DEFAULT ?= "1"
LOCALE_UTF8_IS_DEFAULT:class-nativesdk = "0"
# seccomp is not yet ported to rv32
-DISTRO_FEATURES_DEFAULT:remove:riscv32 = "seccomp"
+DISTRO_FEATURES_OPTED_OUT:append:riscv32 = "seccomp"
# seccomp is not yet ported to ARC
-DISTRO_FEATURES_DEFAULT:remove:arc = "seccomp"
+DISTRO_FEATURES_OPTED_OUT:append:arc = "seccomp"
# seccomp is not yet ported to microblaze
-DISTRO_FEATURES_DEFAULT:remove:microblaze = "seccomp"
+DISTRO_FEATURES_OPTED_OUT:append:microblaze = "seccomp"
# seccomp is not yet ported to loongarch64
-DISTRO_FEATURES_DEFAULT:remove:loongarch64 = "seccomp"
+DISTRO_FEATURES_OPTED_OUT:append:loongarch64 = "seccomp"
-DISTRO_FEATURES_DEFAULT ?= "acl alsa bluetooth debuginfod ext2 ipv4 ipv6 pcmcia usbgadget usbhost wifi xattr nfs zeroconf pci 3g nfc x11 vfat seccomp"
-DISTRO_FEATURES ?= "${DISTRO_FEATURES_DEFAULT}"
+DISTRO_FEATURES_DEFAULTS ?= " \
+ acl alsa bluetooth debuginfod ext2 ipv4 ipv6 pcmcia usbgadget usbhost \
+ wifi xattr nfs zeroconf pci 3g nfc x11 vfat seccomp pulseaudio \
+ gobject-introspection-data ldconfig opengl ptest multiarch wayland vulkan \
+ "
IMAGE_FEATURES ?= ""
COMMERCIAL_AUDIO_PLUGINS ?= ""
diff --git a/meta/conf/distro/include/tclibc-musl.inc b/meta/conf/distro/include/tclibc-musl.inc
index 98d7a801ac84..a61a6ced0dd2 100644
--- a/meta/conf/distro/include/tclibc-musl.inc
+++ b/meta/conf/distro/include/tclibc-musl.inc
@@ -14,7 +14,7 @@ PREFERRED_PROVIDER_virtual/libc-locale ?= "musl-locales"
PREFERRED_PROVIDER_virtual/nativesdk-libintl ?= "nativesdk-glibc"
PREFERRED_PROVIDER_virtual/nativesdk-libiconv ?= "nativesdk-glibc"
-DISTRO_FEATURES_BACKFILL_CONSIDERED += "ldconfig"
+DISTRO_FEATURES_OPTED_OUT += "ldconfig"
#USE_NLS ?= "no"
diff --git a/meta/conf/distro/include/tclibc-newlib.inc b/meta/conf/distro/include/tclibc-newlib.inc
index 34318b24549c..fff9b9f066fb 100644
--- a/meta/conf/distro/include/tclibc-newlib.inc
+++ b/meta/conf/distro/include/tclibc-newlib.inc
@@ -11,7 +11,7 @@ PREFERRED_PROVIDER_virtual/libintl ?= "newlib"
PREFERRED_PROVIDER_virtual/nativesdk-libintl ?= "nativesdk-glibc"
PREFERRED_PROVIDER_virtual/nativesdk-libiconv ?= "nativesdk-glibc"
-DISTRO_FEATURES_BACKFILL_CONSIDERED += "ldconfig"
+DISTRO_FEATURES_OPTED_OUT += "ldconfig"
#USE_NLS ?= "no"
diff --git a/meta/conf/distro/include/tclibc-picolibc.inc b/meta/conf/distro/include/tclibc-picolibc.inc
index 2cd26cfd7dcd..6134b0d97d6a 100644
--- a/meta/conf/distro/include/tclibc-picolibc.inc
+++ b/meta/conf/distro/include/tclibc-picolibc.inc
@@ -11,7 +11,7 @@ PREFERRED_PROVIDER_virtual/libintl ?= "picolibc"
PREFERRED_PROVIDER_virtual/nativesdk-libintl ?= "nativesdk-glibc"
PREFERRED_PROVIDER_virtual/nativesdk-libiconv ?= "nativesdk-glibc"
-DISTRO_FEATURES_BACKFILL_CONSIDERED += "ldconfig"
+DISTRO_FEATURES_OPTED_OUT += "ldconfig"
IMAGE_LINGUAS = ""
diff --git a/meta/conf/documentation.conf b/meta/conf/documentation.conf
index 1853676fa060..5c93d0e47970 100644
--- a/meta/conf/documentation.conf
+++ b/meta/conf/documentation.conf
@@ -141,8 +141,8 @@ DISTRO[doc] = "The short name of the distribution. If the variable is blank, met
DISTRO_EXTRA_RDEPENDS[doc] = "Specifies a list of distro-specific packages to add to all images. The variable only applies to the images that include packagegroup-base."
DISTRO_EXTRA_RRECOMMENDS[doc] = "Specifies a list of distro-specific packages to add to all images if the packages exist. The list of packages are automatically installed but you can remove them."
DISTRO_FEATURES[doc] = "The features enabled for the distribution."
-DISTRO_FEATURES_BACKFILL[doc] = "Features to be added to DISTRO_FEATURES if not also present in DISTRO_FEATURES_BACKFILL_CONSIDERED. This variable is set in the meta/conf/bitbake.conf file and it is not intended to be user-configurable."
-DISTRO_FEATURES_BACKFILL_CONSIDERED[doc] = "Features from DISTRO_FEATURES_BACKFILL that should not be backfilled (i.e. added to DISTRO_FEATURES) during the build."
+DISTRO_FEATURES_DEFAULTS[doc] = "The default set of distribution features, prior to any filtering. Usually you should leave this alone and modify DISTRO_FEATURES and/or DISTRO_FEATURES_OPTED_OUT."
+DISTRO_FEATURES_OPTED_OUT[doc] = "The set of default distribtion features to disable. Prefer opting out of features using this variable instead of using DISTRO_FEATURES:remove."
DISTRO_NAME[doc] = "The long name of the distribution."
DISTRO_PN_ALIAS[doc] = "Alias names used for the recipe in various Linux distributions."
DISTRO_VERSION[doc] = "The version of the distribution."
--
2.43.0
^ permalink raw reply related [flat|nested] 10+ messages in thread* Re: [OE-core] [PATCH v3 2/4] meta: Support opting out of any distro features
2026-04-01 17:41 ` [PATCH v3 2/4] meta: Support opting out of any distro features Paul Barker
@ 2026-04-22 14:45 ` Jose Quaresma
[not found] ` <18A8B511426C8EDA.1516946@lists.openembedded.org>
1 sibling, 0 replies; 10+ messages in thread
From: Jose Quaresma @ 2026-04-22 14:45 UTC (permalink / raw)
To: paul; +Cc: openembedded-core
[-- Attachment #1: Type: text/plain, Size: 14684 bytes --]
Hi Paul,
I think I found a bug that is somehow related to DISTRO_FEATURES_DEFAULTS.
I don't understand why, but the variables in DISTRO_FEATURES_DEFAULTS cannot
be used in inherit or inherit_defer.
Let's use the following recipe as an example:
-------------------------------------------------------------------------------------------------------------------
mkdir -p recipes-test/inherit
cat <<EOF >recipes-test/inherit/inherit.bb
LICENSE = "MIT"
# FAIL
PACKAGECONFIG ??= " \
${@bb.utils.contains('DISTRO_FEATURES',
'gobject-introspection-data', 'ccache', '', d)} \
${@bb.utils.contains('DISTRO_FEATURES',
'gobject-introspection-data', 'vala', '', d)} \
"
# OK
#PACKAGECONFIG ??= " \
# ${@bb.utils.contains('DISTRO_FEATURES', 'systemd', 'ccache', '',
d)} \
# ${@bb.utils.contains('DISTRO_FEATURES', 'systemd', 'vala', '', d)}
\
#"
# get PACKAGECONFIG
# bitbake-getvar -r inherit PACKAGECONFIG
# get CCACHE_TOP_DIR
# bitbake-getvar -r inherit CCACHE_TOP_DIR
inherit_defer ${@bb.utils.filter('PACKAGECONFIG', 'ccache', d)}
# get VALADEPENDS
# bitbake-getvar -r inherit VALADEPENDS
inherit_defer ${@bb.utils.filter('PACKAGECONFIG', 'vala', d)}
EOF
-------------------------------------------------------------------------------------------------------------------
We can see that inherit is not working by running the following command:
bitbake-getvar -r inherit PACKAGECONFIG
bitbake-getvar -r inherit CCACHE_TOP_DIR
bitbake-getvar -r inherit VALADEPENDS
By commenting out the block that uses 'gobject-introspection-data' and
uncommenting the 'systemd', the inherit will then work.
One difference I see is that 'gobject-introspection-data' belongs to
DISTRO_FEATURES_DEFAULTS and 'systemd' does not.
Am I doing something wrong? Could you give me your opinion?
Thanks for your attention.
Jose
Paul Barker via lists.openembedded.org <paul=
pbarker.dev@lists.openembedded.org> escreveu (quarta, 1/04/2026 à(s) 18:42):
> Add default values to DISTRO_FEATURES using the new
> filter_default_features() function.
>
> This change obsoletes the variables DISTRO_FEATURES_BACKFILL and
> DISTRO_FEATURES_BACKFILL_CONSIDERED. Instead, all defaults are added via
> DISTRO_FEATURES_DEFAULTS and users can opt out of any of these using
> DISTRO_FEATURES_OPTED_OUT. Hopefully the variable naming here is easier
> for people to understand and remember.
>
> Migration notes:
>
> - If you have previously assigned DISTRO_FEATURES without using
> DISTRO_FEATURES_DEFAULT, you will now get the default features added
> automatically. You will need to review these and add any features you
> do not want to use to DISTRO_FEATURES_OPTED_OUT.
>
> - DISTRO_FEATURES_DEFAULT is now unused, the new variable name is
> slighlty different to ensure that it is not accidentally used if a
> layer hasn't been modified to adapt to the new naming.
>
> - If you previously set DISTRO_FEATURES_BACKFILL_CONSIDERED, use the new
> variable DISTRO_FEATURES_OPTED_OUT instead.
>
> - If you previously modified DISTRO_FEATURES_BACKFILL, don't do that.
>
> Signed-off-by: Paul Barker <paul@pbarker.dev>
> ---
> meta/classes-global/base.bbclass | 5 ++++-
> meta/classes-recipe/crosssdk.bbclass | 8 ++++----
> meta/classes-recipe/native.bbclass | 8 ++++----
> meta/classes-recipe/nativesdk.bbclass | 8 ++++----
> meta/conf/bitbake.conf | 1 -
> meta/conf/distro/include/default-distrovars.inc | 15 +++++++++------
> meta/conf/distro/include/tclibc-musl.inc | 2 +-
> meta/conf/distro/include/tclibc-newlib.inc | 2 +-
> meta/conf/distro/include/tclibc-picolibc.inc | 2 +-
> meta/conf/documentation.conf | 4 ++--
> 10 files changed, 30 insertions(+), 25 deletions(-)
>
> diff --git a/meta/classes-global/base.bbclass
> b/meta/classes-global/base.bbclass
> index 22b427a5211b..0a4cfd594c72 100644
> --- a/meta/classes-global/base.bbclass
> +++ b/meta/classes-global/base.bbclass
> @@ -454,8 +454,11 @@ def set_packagetriplet(d):
> python () {
> import string, re
>
> + # Filter default features to allow users to opt out of features they
> don't
> + # want.
> + oe.utils.filter_default_features("DISTRO_FEATURES", d)
> +
> # Handle backfilling
> - oe.utils.features_backfill("DISTRO_FEATURES", d)
> oe.utils.features_backfill("MACHINE_FEATURES", d)
>
> # To add a recipe to the skip list , set:
> diff --git a/meta/classes-recipe/crosssdk.bbclass
> b/meta/classes-recipe/crosssdk.bbclass
> index a2853e6a9201..dac18b39edb2 100644
> --- a/meta/classes-recipe/crosssdk.bbclass
> +++ b/meta/classes-recipe/crosssdk.bbclass
> @@ -16,13 +16,13 @@ PACKAGE_ARCH = "${SDK_ARCH}"
> python () {
> # set TUNE_PKGARCH to SDK_ARCH
> d.setVar('TUNE_PKGARCH', d.getVar('SDK_ARCH'))
> - # Set features here to prevent appends and distro features backfill
> - # from modifying nativesdk distro features
> + # Set features here to prevent DISTRO_FEATURES modifications from
> affecting
> + # crosssdk distro features
> features = set(d.getVar("DISTRO_FEATURES_NATIVESDK").split())
> - oe.utils.features_backfill("DISTRO_FEATURES", d)
> + oe.utils.filter_default_features("DISTRO_FEATURES", d)
> filtered = set(bb.utils.filter("DISTRO_FEATURES",
> d.getVar("DISTRO_FEATURES_FILTER_NATIVESDK"), d).split())
> d.setVar("DISTRO_FEATURES", " ".join(sorted(features | filtered)))
> - d.setVar("DISTRO_FEATURES_BACKFILL", "")
> + d.setVar("DISTRO_FEATURES_DEFAULTS", "")
> }
>
> STAGING_BINDIR_TOOLCHAIN =
> "${STAGING_DIR_NATIVE}${bindir_native}/${TARGET_ARCH}${TARGET_VENDOR}-${TARGET_OS}"
> diff --git a/meta/classes-recipe/native.bbclass
> b/meta/classes-recipe/native.bbclass
> index 5aa9c8e4145e..9f4ca170f762 100644
> --- a/meta/classes-recipe/native.bbclass
> +++ b/meta/classes-recipe/native.bbclass
> @@ -126,13 +126,13 @@ python native_virtclass_handler () {
> return
> bpn = d.getVar("BPN")
>
> - # Set features here to prevent appends and distro features backfill
> - # from modifying native distro features
> + # Set features here to prevent DISTRO_FEATURES modifications from
> affecting
> + # native distro features
> features = set(d.getVar("DISTRO_FEATURES_NATIVE").split())
> - oe.utils.features_backfill("DISTRO_FEATURES", d)
> + oe.utils.filter_default_features("DISTRO_FEATURES", d)
> filtered = set(bb.utils.filter("DISTRO_FEATURES",
> d.getVar("DISTRO_FEATURES_FILTER_NATIVE"), d).split())
> d.setVar("DISTRO_FEATURES", " ".join(sorted(features | filtered)))
> - d.setVar("DISTRO_FEATURES_BACKFILL", "")
> + d.setVar("DISTRO_FEATURES_DEFAULTS", "")
>
> classextend = d.getVar('BBCLASSEXTEND') or ""
> if "native" not in classextend:
> diff --git a/meta/classes-recipe/nativesdk.bbclass
> b/meta/classes-recipe/nativesdk.bbclass
> index 5adb7515bf14..a4801c48b2de 100644
> --- a/meta/classes-recipe/nativesdk.bbclass
> +++ b/meta/classes-recipe/nativesdk.bbclass
> @@ -78,13 +78,13 @@ python nativesdk_virtclass_handler () {
> if not (pn.endswith("-nativesdk") or pn.startswith("nativesdk-")):
> return
>
> - # Set features here to prevent appends and distro features backfill
> - # from modifying nativesdk distro features
> + # Set features here to prevent DISTRO_FEATURES modifications from
> affecting
> + # nativesdk distro features
> features = set(d.getVar("DISTRO_FEATURES_NATIVESDK").split())
> - oe.utils.features_backfill("DISTRO_FEATURES", d)
> + oe.utils.filter_default_features("DISTRO_FEATURES", d)
> filtered = set(bb.utils.filter("DISTRO_FEATURES",
> d.getVar("DISTRO_FEATURES_FILTER_NATIVESDK"), d).split())
> d.setVar("DISTRO_FEATURES", " ".join(sorted(features | filtered)))
> - d.setVar("DISTRO_FEATURES_BACKFILL", "")
> + d.setVar("DISTRO_FEATURES_DEFAULTS", "")
>
> e.data.setVar("MLPREFIX", "nativesdk-")
> e.data.setVar("PN", "nativesdk-" +
> e.data.getVar("PN").replace("-nativesdk", "").replace("nativesdk-", ""))
> diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
> index 6b3443695a04..f851e2adc9f7 100644
> --- a/meta/conf/bitbake.conf
> +++ b/meta/conf/bitbake.conf
> @@ -912,7 +912,6 @@ DISTRO_FEATURES_NATIVESDK:mingw32 = "x11 ipv6"
> DISTRO_FEATURES_FILTER_NATIVE ?= "api-documentation debuginfod opencl
> opengl wayland"
> DISTRO_FEATURES_FILTER_NATIVESDK ?= "api-documentation debuginfod opencl
> opengl wayland"
>
> -DISTRO_FEATURES_BACKFILL = "pulseaudio gobject-introspection-data
> ldconfig opengl ptest multiarch wayland vulkan"
> MACHINE_FEATURES_BACKFILL = "rtc qemu-usermode"
>
> COMBINED_FEATURES = "${@oe.utils.set_intersect('DISTRO_FEATURES',
> 'MACHINE_FEATURES', d)}"
> diff --git a/meta/conf/distro/include/default-distrovars.inc
> b/meta/conf/distro/include/default-distrovars.inc
> index 7adcdfad4ff4..daa4d2a74dda 100644
> --- a/meta/conf/distro/include/default-distrovars.inc
> +++ b/meta/conf/distro/include/default-distrovars.inc
> @@ -14,19 +14,22 @@ LOCALE_UTF8_IS_DEFAULT ?= "1"
> LOCALE_UTF8_IS_DEFAULT:class-nativesdk = "0"
>
> # seccomp is not yet ported to rv32
> -DISTRO_FEATURES_DEFAULT:remove:riscv32 = "seccomp"
> +DISTRO_FEATURES_OPTED_OUT:append:riscv32 = "seccomp"
>
> # seccomp is not yet ported to ARC
> -DISTRO_FEATURES_DEFAULT:remove:arc = "seccomp"
> +DISTRO_FEATURES_OPTED_OUT:append:arc = "seccomp"
>
> # seccomp is not yet ported to microblaze
> -DISTRO_FEATURES_DEFAULT:remove:microblaze = "seccomp"
> +DISTRO_FEATURES_OPTED_OUT:append:microblaze = "seccomp"
>
> # seccomp is not yet ported to loongarch64
> -DISTRO_FEATURES_DEFAULT:remove:loongarch64 = "seccomp"
> +DISTRO_FEATURES_OPTED_OUT:append:loongarch64 = "seccomp"
>
> -DISTRO_FEATURES_DEFAULT ?= "acl alsa bluetooth debuginfod ext2 ipv4 ipv6
> pcmcia usbgadget usbhost wifi xattr nfs zeroconf pci 3g nfc x11 vfat
> seccomp"
> -DISTRO_FEATURES ?= "${DISTRO_FEATURES_DEFAULT}"
> +DISTRO_FEATURES_DEFAULTS ?= " \
> + acl alsa bluetooth debuginfod ext2 ipv4 ipv6 pcmcia usbgadget usbhost
> \
> + wifi xattr nfs zeroconf pci 3g nfc x11 vfat seccomp pulseaudio \
> + gobject-introspection-data ldconfig opengl ptest multiarch wayland
> vulkan \
> + "
> IMAGE_FEATURES ?= ""
>
> COMMERCIAL_AUDIO_PLUGINS ?= ""
> diff --git a/meta/conf/distro/include/tclibc-musl.inc
> b/meta/conf/distro/include/tclibc-musl.inc
> index 98d7a801ac84..a61a6ced0dd2 100644
> --- a/meta/conf/distro/include/tclibc-musl.inc
> +++ b/meta/conf/distro/include/tclibc-musl.inc
> @@ -14,7 +14,7 @@ PREFERRED_PROVIDER_virtual/libc-locale ?= "musl-locales"
> PREFERRED_PROVIDER_virtual/nativesdk-libintl ?= "nativesdk-glibc"
> PREFERRED_PROVIDER_virtual/nativesdk-libiconv ?= "nativesdk-glibc"
>
> -DISTRO_FEATURES_BACKFILL_CONSIDERED += "ldconfig"
> +DISTRO_FEATURES_OPTED_OUT += "ldconfig"
>
> #USE_NLS ?= "no"
>
> diff --git a/meta/conf/distro/include/tclibc-newlib.inc
> b/meta/conf/distro/include/tclibc-newlib.inc
> index 34318b24549c..fff9b9f066fb 100644
> --- a/meta/conf/distro/include/tclibc-newlib.inc
> +++ b/meta/conf/distro/include/tclibc-newlib.inc
> @@ -11,7 +11,7 @@ PREFERRED_PROVIDER_virtual/libintl ?= "newlib"
> PREFERRED_PROVIDER_virtual/nativesdk-libintl ?= "nativesdk-glibc"
> PREFERRED_PROVIDER_virtual/nativesdk-libiconv ?= "nativesdk-glibc"
>
> -DISTRO_FEATURES_BACKFILL_CONSIDERED += "ldconfig"
> +DISTRO_FEATURES_OPTED_OUT += "ldconfig"
>
> #USE_NLS ?= "no"
>
> diff --git a/meta/conf/distro/include/tclibc-picolibc.inc
> b/meta/conf/distro/include/tclibc-picolibc.inc
> index 2cd26cfd7dcd..6134b0d97d6a 100644
> --- a/meta/conf/distro/include/tclibc-picolibc.inc
> +++ b/meta/conf/distro/include/tclibc-picolibc.inc
> @@ -11,7 +11,7 @@ PREFERRED_PROVIDER_virtual/libintl ?= "picolibc"
> PREFERRED_PROVIDER_virtual/nativesdk-libintl ?= "nativesdk-glibc"
> PREFERRED_PROVIDER_virtual/nativesdk-libiconv ?= "nativesdk-glibc"
>
> -DISTRO_FEATURES_BACKFILL_CONSIDERED += "ldconfig"
> +DISTRO_FEATURES_OPTED_OUT += "ldconfig"
>
> IMAGE_LINGUAS = ""
>
> diff --git a/meta/conf/documentation.conf b/meta/conf/documentation.conf
> index 1853676fa060..5c93d0e47970 100644
> --- a/meta/conf/documentation.conf
> +++ b/meta/conf/documentation.conf
> @@ -141,8 +141,8 @@ DISTRO[doc] = "The short name of the distribution. If
> the variable is blank, met
> DISTRO_EXTRA_RDEPENDS[doc] = "Specifies a list of distro-specific
> packages to add to all images. The variable only applies to the images that
> include packagegroup-base."
> DISTRO_EXTRA_RRECOMMENDS[doc] = "Specifies a list of distro-specific
> packages to add to all images if the packages exist. The list of packages
> are automatically installed but you can remove them."
> DISTRO_FEATURES[doc] = "The features enabled for the distribution."
> -DISTRO_FEATURES_BACKFILL[doc] = "Features to be added to DISTRO_FEATURES
> if not also present in DISTRO_FEATURES_BACKFILL_CONSIDERED. This variable
> is set in the meta/conf/bitbake.conf file and it is not intended to be
> user-configurable."
> -DISTRO_FEATURES_BACKFILL_CONSIDERED[doc] = "Features from
> DISTRO_FEATURES_BACKFILL that should not be backfilled (i.e. added to
> DISTRO_FEATURES) during the build."
> +DISTRO_FEATURES_DEFAULTS[doc] = "The default set of distribution
> features, prior to any filtering. Usually you should leave this alone and
> modify DISTRO_FEATURES and/or DISTRO_FEATURES_OPTED_OUT."
> +DISTRO_FEATURES_OPTED_OUT[doc] = "The set of default distribtion features
> to disable. Prefer opting out of features using this variable instead of
> using DISTRO_FEATURES:remove."
> DISTRO_NAME[doc] = "The long name of the distribution."
> DISTRO_PN_ALIAS[doc] = "Alias names used for the recipe in various Linux
> distributions."
> DISTRO_VERSION[doc] = "The version of the distribution."
>
> --
> 2.43.0
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#234452):
> https://lists.openembedded.org/g/openembedded-core/message/234452
> Mute This Topic: https://lists.openembedded.org/mt/118620443/5052612
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [
> quaresma.jose@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>
--
Best regards,
José Quaresma
[-- Attachment #2: Type: text/html, Size: 18158 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread[parent not found: <18A8B511426C8EDA.1516946@lists.openembedded.org>]
* Re: [OE-core] [PATCH v3 2/4] meta: Support opting out of any distro features
[not found] ` <18A8B511426C8EDA.1516946@lists.openembedded.org>
@ 2026-04-22 20:26 ` Jose Quaresma
2026-04-23 9:11 ` Paul Barker
0 siblings, 1 reply; 10+ messages in thread
From: Jose Quaresma @ 2026-04-22 20:26 UTC (permalink / raw)
To: quaresma.jose; +Cc: paul, openembedded-core
[-- Attachment #1: Type: text/plain, Size: 16919 bytes --]
Jose Quaresma via lists.openembedded.org <quaresma.jose=
gmail.com@lists.openembedded.org> escreveu (quarta, 22/04/2026 à(s) 15:45):
> Hi Paul,
>
> I think I found a bug that is somehow related to DISTRO_FEATURES_DEFAULTS.
> I don't understand why, but the variables in
> DISTRO_FEATURES_DEFAULTS cannot
> be used in inherit or inherit_defer.
>
> Let's use the following recipe as an example:
>
> -------------------------------------------------------------------------------------------------------------------
> mkdir -p recipes-test/inherit
> cat <<'EOF' >recipes-test/inherit/inherit.bb
>
> LICENSE = "MIT"
>
> # FAIL
> PACKAGECONFIG ??= " \
> ${@bb.utils.contains('DISTRO_FEATURES',
> 'gobject-introspection-data', 'ccache', '', d)} \
> ${@bb.utils.contains('DISTRO_FEATURES',
> 'gobject-introspection-data', 'vala', '', d)} \
> "
>
> # OK
> #PACKAGECONFIG ??= " \
> # ${@bb.utils.contains('DISTRO_FEATURES', 'systemd', 'ccache', '',
> d)} \
> # ${@bb.utils.contains('DISTRO_FEATURES', 'systemd', 'vala', '',
> d)} \
> #"
>
> # get PACKAGECONFIG
> # bitbake-getvar -r inherit PACKAGECONFIG
>
> # get CCACHE_TOP_DIR
> # bitbake-getvar -r inherit CCACHE_TOP_DIR
> inherit_defer ${@bb.utils.filter('PACKAGECONFIG', 'ccache', d)}
>
> # get VALADEPENDS
> # bitbake-getvar -r inherit VALADEPENDS
> inherit_defer ${@bb.utils.filter('PACKAGECONFIG', 'vala', d)}
> EOF
>
> -------------------------------------------------------------------------------------------------------------------
>
A clearer way to replicate the problem:
#
-------------------------------------------------------------------------------------------------------------------
mkdir -p recipes-test/inherit
cat <<'EOF' >recipes-test/inherit/inherit.bb
LICENSE = "MIT"
DISTRO_FEATURES_DEFAULTS += "aaa"
DISTRO_FEATURES += "bbb"
PACKAGECONFIG = " \
${@bb.utils.contains('DISTRO_FEATURES', 'aaa', 'ccache', '', d)} \
${@bb.utils.contains('DISTRO_FEATURES', 'bbb', 'vala', '', d)} \
"
inherit_defer ${@bb.utils.filter('PACKAGECONFIG', 'ccache', d)}
inherit_defer ${@bb.utils.filter('PACKAGECONFIG', 'vala', d)}
EOF
#
-------------------------------------------------------------------------------------------------------------------
bitbake-getvar --value -r inherit PACKAGECONFIG
bitbake-getvar --value -r inherit CCACHE_TOP_DIR
bitbake-getvar --value -r inherit VALADEPENDS
#
-------------------------------------------------------------------------------------------------------------------
The DISTRO_FEATURES is mapped to inherit/inherit_defer as follows:
'aaa' maps to 'ccache'
'bbb' maps to 'vala'
I chose these two classes, but it could be any others. The idea is to see
if they were inherited by looking at the class variables.
The ccache.bbclass was not inherit and so the variable 'CCACHE_TOP_DIR' is
not defined.
Hence my conclusion that DISTRO_FEATURES_DEFAULTS doesn't work for these
cases, which in my opinion is not expected.
I encountered this problem while debugging the patch
https://lists.openembedded.org/g/openembedded-devel/topic/118939974.
Best regards,
Jose
>
> We can see that inherit is not working by running the following command:
>
> bitbake-getvar -r inherit PACKAGECONFIG
> bitbake-getvar -r inherit CCACHE_TOP_DIR
> bitbake-getvar -r inherit VALADEPENDS
>
>
> By commenting out the block that uses 'gobject-introspection-data' and
> uncommenting the 'systemd', the inherit will then work.
> One difference I see is that 'gobject-introspection-data' belongs to
> DISTRO_FEATURES_DEFAULTS and 'systemd' does not.
>
> Am I doing something wrong? Could you give me your opinion?
>
> Thanks for your attention.
>
> Jose
>
> Paul Barker via lists.openembedded.org <paul=
> pbarker.dev@lists.openembedded.org> escreveu (quarta, 1/04/2026 à(s)
> 18:42):
>
>> Add default values to DISTRO_FEATURES using the new
>> filter_default_features() function.
>>
>> This change obsoletes the variables DISTRO_FEATURES_BACKFILL and
>> DISTRO_FEATURES_BACKFILL_CONSIDERED. Instead, all defaults are added via
>> DISTRO_FEATURES_DEFAULTS and users can opt out of any of these using
>> DISTRO_FEATURES_OPTED_OUT. Hopefully the variable naming here is easier
>> for people to understand and remember.
>>
>> Migration notes:
>>
>> - If you have previously assigned DISTRO_FEATURES without using
>> DISTRO_FEATURES_DEFAULT, you will now get the default features added
>> automatically. You will need to review these and add any features you
>> do not want to use to DISTRO_FEATURES_OPTED_OUT.
>>
>> - DISTRO_FEATURES_DEFAULT is now unused, the new variable name is
>> slighlty different to ensure that it is not accidentally used if a
>> layer hasn't been modified to adapt to the new naming.
>>
>> - If you previously set DISTRO_FEATURES_BACKFILL_CONSIDERED, use the new
>> variable DISTRO_FEATURES_OPTED_OUT instead.
>>
>> - If you previously modified DISTRO_FEATURES_BACKFILL, don't do that.
>>
>> Signed-off-by: Paul Barker <paul@pbarker.dev>
>> ---
>> meta/classes-global/base.bbclass | 5 ++++-
>> meta/classes-recipe/crosssdk.bbclass | 8 ++++----
>> meta/classes-recipe/native.bbclass | 8 ++++----
>> meta/classes-recipe/nativesdk.bbclass | 8 ++++----
>> meta/conf/bitbake.conf | 1 -
>> meta/conf/distro/include/default-distrovars.inc | 15 +++++++++------
>> meta/conf/distro/include/tclibc-musl.inc | 2 +-
>> meta/conf/distro/include/tclibc-newlib.inc | 2 +-
>> meta/conf/distro/include/tclibc-picolibc.inc | 2 +-
>> meta/conf/documentation.conf | 4 ++--
>> 10 files changed, 30 insertions(+), 25 deletions(-)
>>
>> diff --git a/meta/classes-global/base.bbclass
>> b/meta/classes-global/base.bbclass
>> index 22b427a5211b..0a4cfd594c72 100644
>> --- a/meta/classes-global/base.bbclass
>> +++ b/meta/classes-global/base.bbclass
>> @@ -454,8 +454,11 @@ def set_packagetriplet(d):
>> python () {
>> import string, re
>>
>> + # Filter default features to allow users to opt out of features they
>> don't
>> + # want.
>> + oe.utils.filter_default_features("DISTRO_FEATURES", d)
>> +
>> # Handle backfilling
>> - oe.utils.features_backfill("DISTRO_FEATURES", d)
>> oe.utils.features_backfill("MACHINE_FEATURES", d)
>>
>> # To add a recipe to the skip list , set:
>> diff --git a/meta/classes-recipe/crosssdk.bbclass
>> b/meta/classes-recipe/crosssdk.bbclass
>> index a2853e6a9201..dac18b39edb2 100644
>> --- a/meta/classes-recipe/crosssdk.bbclass
>> +++ b/meta/classes-recipe/crosssdk.bbclass
>> @@ -16,13 +16,13 @@ PACKAGE_ARCH = "${SDK_ARCH}"
>> python () {
>> # set TUNE_PKGARCH to SDK_ARCH
>> d.setVar('TUNE_PKGARCH', d.getVar('SDK_ARCH'))
>> - # Set features here to prevent appends and distro features backfill
>> - # from modifying nativesdk distro features
>> + # Set features here to prevent DISTRO_FEATURES modifications from
>> affecting
>> + # crosssdk distro features
>> features = set(d.getVar("DISTRO_FEATURES_NATIVESDK").split())
>> - oe.utils.features_backfill("DISTRO_FEATURES", d)
>> + oe.utils.filter_default_features("DISTRO_FEATURES", d)
>> filtered = set(bb.utils.filter("DISTRO_FEATURES",
>> d.getVar("DISTRO_FEATURES_FILTER_NATIVESDK"), d).split())
>> d.setVar("DISTRO_FEATURES", " ".join(sorted(features | filtered)))
>> - d.setVar("DISTRO_FEATURES_BACKFILL", "")
>> + d.setVar("DISTRO_FEATURES_DEFAULTS", "")
>> }
>>
>> STAGING_BINDIR_TOOLCHAIN =
>> "${STAGING_DIR_NATIVE}${bindir_native}/${TARGET_ARCH}${TARGET_VENDOR}-${TARGET_OS}"
>> diff --git a/meta/classes-recipe/native.bbclass
>> b/meta/classes-recipe/native.bbclass
>> index 5aa9c8e4145e..9f4ca170f762 100644
>> --- a/meta/classes-recipe/native.bbclass
>> +++ b/meta/classes-recipe/native.bbclass
>> @@ -126,13 +126,13 @@ python native_virtclass_handler () {
>> return
>> bpn = d.getVar("BPN")
>>
>> - # Set features here to prevent appends and distro features backfill
>> - # from modifying native distro features
>> + # Set features here to prevent DISTRO_FEATURES modifications from
>> affecting
>> + # native distro features
>> features = set(d.getVar("DISTRO_FEATURES_NATIVE").split())
>> - oe.utils.features_backfill("DISTRO_FEATURES", d)
>> + oe.utils.filter_default_features("DISTRO_FEATURES", d)
>> filtered = set(bb.utils.filter("DISTRO_FEATURES",
>> d.getVar("DISTRO_FEATURES_FILTER_NATIVE"), d).split())
>> d.setVar("DISTRO_FEATURES", " ".join(sorted(features | filtered)))
>> - d.setVar("DISTRO_FEATURES_BACKFILL", "")
>> + d.setVar("DISTRO_FEATURES_DEFAULTS", "")
>>
>> classextend = d.getVar('BBCLASSEXTEND') or ""
>> if "native" not in classextend:
>> diff --git a/meta/classes-recipe/nativesdk.bbclass
>> b/meta/classes-recipe/nativesdk.bbclass
>> index 5adb7515bf14..a4801c48b2de 100644
>> --- a/meta/classes-recipe/nativesdk.bbclass
>> +++ b/meta/classes-recipe/nativesdk.bbclass
>> @@ -78,13 +78,13 @@ python nativesdk_virtclass_handler () {
>> if not (pn.endswith("-nativesdk") or pn.startswith("nativesdk-")):
>> return
>>
>> - # Set features here to prevent appends and distro features backfill
>> - # from modifying nativesdk distro features
>> + # Set features here to prevent DISTRO_FEATURES modifications from
>> affecting
>> + # nativesdk distro features
>> features = set(d.getVar("DISTRO_FEATURES_NATIVESDK").split())
>> - oe.utils.features_backfill("DISTRO_FEATURES", d)
>> + oe.utils.filter_default_features("DISTRO_FEATURES", d)
>> filtered = set(bb.utils.filter("DISTRO_FEATURES",
>> d.getVar("DISTRO_FEATURES_FILTER_NATIVESDK"), d).split())
>> d.setVar("DISTRO_FEATURES", " ".join(sorted(features | filtered)))
>> - d.setVar("DISTRO_FEATURES_BACKFILL", "")
>> + d.setVar("DISTRO_FEATURES_DEFAULTS", "")
>>
>> e.data.setVar("MLPREFIX", "nativesdk-")
>> e.data.setVar("PN", "nativesdk-" +
>> e.data.getVar("PN").replace("-nativesdk", "").replace("nativesdk-", ""))
>> diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
>> index 6b3443695a04..f851e2adc9f7 100644
>> --- a/meta/conf/bitbake.conf
>> +++ b/meta/conf/bitbake.conf
>> @@ -912,7 +912,6 @@ DISTRO_FEATURES_NATIVESDK:mingw32 = "x11 ipv6"
>> DISTRO_FEATURES_FILTER_NATIVE ?= "api-documentation debuginfod opencl
>> opengl wayland"
>> DISTRO_FEATURES_FILTER_NATIVESDK ?= "api-documentation debuginfod opencl
>> opengl wayland"
>>
>> -DISTRO_FEATURES_BACKFILL = "pulseaudio gobject-introspection-data
>> ldconfig opengl ptest multiarch wayland vulkan"
>> MACHINE_FEATURES_BACKFILL = "rtc qemu-usermode"
>>
>> COMBINED_FEATURES = "${@oe.utils.set_intersect('DISTRO_FEATURES',
>> 'MACHINE_FEATURES', d)}"
>> diff --git a/meta/conf/distro/include/default-distrovars.inc
>> b/meta/conf/distro/include/default-distrovars.inc
>> index 7adcdfad4ff4..daa4d2a74dda 100644
>> --- a/meta/conf/distro/include/default-distrovars.inc
>> +++ b/meta/conf/distro/include/default-distrovars.inc
>> @@ -14,19 +14,22 @@ LOCALE_UTF8_IS_DEFAULT ?= "1"
>> LOCALE_UTF8_IS_DEFAULT:class-nativesdk = "0"
>>
>> # seccomp is not yet ported to rv32
>> -DISTRO_FEATURES_DEFAULT:remove:riscv32 = "seccomp"
>> +DISTRO_FEATURES_OPTED_OUT:append:riscv32 = "seccomp"
>>
>> # seccomp is not yet ported to ARC
>> -DISTRO_FEATURES_DEFAULT:remove:arc = "seccomp"
>> +DISTRO_FEATURES_OPTED_OUT:append:arc = "seccomp"
>>
>> # seccomp is not yet ported to microblaze
>> -DISTRO_FEATURES_DEFAULT:remove:microblaze = "seccomp"
>> +DISTRO_FEATURES_OPTED_OUT:append:microblaze = "seccomp"
>>
>> # seccomp is not yet ported to loongarch64
>> -DISTRO_FEATURES_DEFAULT:remove:loongarch64 = "seccomp"
>> +DISTRO_FEATURES_OPTED_OUT:append:loongarch64 = "seccomp"
>>
>> -DISTRO_FEATURES_DEFAULT ?= "acl alsa bluetooth debuginfod ext2 ipv4 ipv6
>> pcmcia usbgadget usbhost wifi xattr nfs zeroconf pci 3g nfc x11 vfat
>> seccomp"
>> -DISTRO_FEATURES ?= "${DISTRO_FEATURES_DEFAULT}"
>> +DISTRO_FEATURES_DEFAULTS ?= " \
>> + acl alsa bluetooth debuginfod ext2 ipv4 ipv6 pcmcia usbgadget
>> usbhost \
>> + wifi xattr nfs zeroconf pci 3g nfc x11 vfat seccomp pulseaudio \
>> + gobject-introspection-data ldconfig opengl ptest multiarch wayland
>> vulkan \
>> + "
>> IMAGE_FEATURES ?= ""
>>
>> COMMERCIAL_AUDIO_PLUGINS ?= ""
>> diff --git a/meta/conf/distro/include/tclibc-musl.inc
>> b/meta/conf/distro/include/tclibc-musl.inc
>> index 98d7a801ac84..a61a6ced0dd2 100644
>> --- a/meta/conf/distro/include/tclibc-musl.inc
>> +++ b/meta/conf/distro/include/tclibc-musl.inc
>> @@ -14,7 +14,7 @@ PREFERRED_PROVIDER_virtual/libc-locale ?= "musl-locales"
>> PREFERRED_PROVIDER_virtual/nativesdk-libintl ?= "nativesdk-glibc"
>> PREFERRED_PROVIDER_virtual/nativesdk-libiconv ?= "nativesdk-glibc"
>>
>> -DISTRO_FEATURES_BACKFILL_CONSIDERED += "ldconfig"
>> +DISTRO_FEATURES_OPTED_OUT += "ldconfig"
>>
>> #USE_NLS ?= "no"
>>
>> diff --git a/meta/conf/distro/include/tclibc-newlib.inc
>> b/meta/conf/distro/include/tclibc-newlib.inc
>> index 34318b24549c..fff9b9f066fb 100644
>> --- a/meta/conf/distro/include/tclibc-newlib.inc
>> +++ b/meta/conf/distro/include/tclibc-newlib.inc
>> @@ -11,7 +11,7 @@ PREFERRED_PROVIDER_virtual/libintl ?= "newlib"
>> PREFERRED_PROVIDER_virtual/nativesdk-libintl ?= "nativesdk-glibc"
>> PREFERRED_PROVIDER_virtual/nativesdk-libiconv ?= "nativesdk-glibc"
>>
>> -DISTRO_FEATURES_BACKFILL_CONSIDERED += "ldconfig"
>> +DISTRO_FEATURES_OPTED_OUT += "ldconfig"
>>
>> #USE_NLS ?= "no"
>>
>> diff --git a/meta/conf/distro/include/tclibc-picolibc.inc
>> b/meta/conf/distro/include/tclibc-picolibc.inc
>> index 2cd26cfd7dcd..6134b0d97d6a 100644
>> --- a/meta/conf/distro/include/tclibc-picolibc.inc
>> +++ b/meta/conf/distro/include/tclibc-picolibc.inc
>> @@ -11,7 +11,7 @@ PREFERRED_PROVIDER_virtual/libintl ?= "picolibc"
>> PREFERRED_PROVIDER_virtual/nativesdk-libintl ?= "nativesdk-glibc"
>> PREFERRED_PROVIDER_virtual/nativesdk-libiconv ?= "nativesdk-glibc"
>>
>> -DISTRO_FEATURES_BACKFILL_CONSIDERED += "ldconfig"
>> +DISTRO_FEATURES_OPTED_OUT += "ldconfig"
>>
>> IMAGE_LINGUAS = ""
>>
>> diff --git a/meta/conf/documentation.conf b/meta/conf/documentation.conf
>> index 1853676fa060..5c93d0e47970 100644
>> --- a/meta/conf/documentation.conf
>> +++ b/meta/conf/documentation.conf
>> @@ -141,8 +141,8 @@ DISTRO[doc] = "The short name of the distribution. If
>> the variable is blank, met
>> DISTRO_EXTRA_RDEPENDS[doc] = "Specifies a list of distro-specific
>> packages to add to all images. The variable only applies to the images that
>> include packagegroup-base."
>> DISTRO_EXTRA_RRECOMMENDS[doc] = "Specifies a list of distro-specific
>> packages to add to all images if the packages exist. The list of packages
>> are automatically installed but you can remove them."
>> DISTRO_FEATURES[doc] = "The features enabled for the distribution."
>> -DISTRO_FEATURES_BACKFILL[doc] = "Features to be added to DISTRO_FEATURES
>> if not also present in DISTRO_FEATURES_BACKFILL_CONSIDERED. This variable
>> is set in the meta/conf/bitbake.conf file and it is not intended to be
>> user-configurable."
>> -DISTRO_FEATURES_BACKFILL_CONSIDERED[doc] = "Features from
>> DISTRO_FEATURES_BACKFILL that should not be backfilled (i.e. added to
>> DISTRO_FEATURES) during the build."
>> +DISTRO_FEATURES_DEFAULTS[doc] = "The default set of distribution
>> features, prior to any filtering. Usually you should leave this alone and
>> modify DISTRO_FEATURES and/or DISTRO_FEATURES_OPTED_OUT."
>> +DISTRO_FEATURES_OPTED_OUT[doc] = "The set of default distribtion
>> features to disable. Prefer opting out of features using this variable
>> instead of using DISTRO_FEATURES:remove."
>> DISTRO_NAME[doc] = "The long name of the distribution."
>> DISTRO_PN_ALIAS[doc] = "Alias names used for the recipe in various Linux
>> distributions."
>> DISTRO_VERSION[doc] = "The version of the distribution."
>>
>> --
>> 2.43.0
>>
>>
>>
>>
>>
>
> --
> Best regards,
>
> José Quaresma
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#235727):
> https://lists.openembedded.org/g/openembedded-core/message/235727
> Mute This Topic: https://lists.openembedded.org/mt/118620443/5052612
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [
> quaresma.jose@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>
--
Best regards,
José Quaresma
[-- Attachment #2: Type: text/html, Size: 21417 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [OE-core] [PATCH v3 2/4] meta: Support opting out of any distro features
2026-04-22 20:26 ` Jose Quaresma
@ 2026-04-23 9:11 ` Paul Barker
2026-04-23 9:42 ` Jose Quaresma
[not found] ` <18A8F319FD743690.2028604@lists.openembedded.org>
0 siblings, 2 replies; 10+ messages in thread
From: Paul Barker @ 2026-04-23 9:11 UTC (permalink / raw)
To: Jose Quaresma; +Cc: openembedded-core
[-- Attachment #1: Type: text/plain, Size: 4310 bytes --]
On Wed, 2026-04-22 at 21:26 +0100, Jose Quaresma wrote:
> Jose Quaresma via lists.openembedded.org <quaresma.jose=
> gmail.com@lists.openembedded.org> escreveu (quarta, 22/04/2026 à(s) 15:45):
>
> > Hi Paul,
> >
> > I think I found a bug that is somehow related to DISTRO_FEATURES_DEFAULTS.
> > I don't understand why, but the variables in
> > DISTRO_FEATURES_DEFAULTS cannot
> > be used in inherit or inherit_defer.
> >
> > Let's use the following recipe as an example:
> >
> > -------------------------------------------------------------------------------------------------------------------
> > mkdir -p recipes-test/inherit
> > cat <<'EOF' >recipes-test/inherit/inherit.bb
> >
> > LICENSE = "MIT"
> >
> > # FAIL
> > PACKAGECONFIG ??= " \
> > ${@bb.utils.contains('DISTRO_FEATURES',
> > 'gobject-introspection-data', 'ccache', '', d)} \
> > ${@bb.utils.contains('DISTRO_FEATURES',
> > 'gobject-introspection-data', 'vala', '', d)} \
> > "
> >
> > # OK
> > #PACKAGECONFIG ??= " \
> > # ${@bb.utils.contains('DISTRO_FEATURES', 'systemd', 'ccache', '',
> > d)} \
> > # ${@bb.utils.contains('DISTRO_FEATURES', 'systemd', 'vala', '',
> > d)} \
> > #"
> >
> > # get PACKAGECONFIG
> > # bitbake-getvar -r inherit PACKAGECONFIG
> >
> > # get CCACHE_TOP_DIR
> > # bitbake-getvar -r inherit CCACHE_TOP_DIR
> > inherit_defer ${@bb.utils.filter('PACKAGECONFIG', 'ccache', d)}
> >
> > # get VALADEPENDS
> > # bitbake-getvar -r inherit VALADEPENDS
> > inherit_defer ${@bb.utils.filter('PACKAGECONFIG', 'vala', d)}
> > EOF
> >
> > -------------------------------------------------------------------------------------------------------------------
> >
>
> A clearer way to replicate the problem:
>
> #
> -------------------------------------------------------------------------------------------------------------------
> mkdir -p recipes-test/inherit
> cat <<'EOF' >recipes-test/inherit/inherit.bb
> LICENSE = "MIT"
>
> DISTRO_FEATURES_DEFAULTS += "aaa"
> DISTRO_FEATURES += "bbb"
>
> PACKAGECONFIG = " \
> ${@bb.utils.contains('DISTRO_FEATURES', 'aaa', 'ccache', '', d)} \
> ${@bb.utils.contains('DISTRO_FEATURES', 'bbb', 'vala', '', d)} \
> "
>
> inherit_defer ${@bb.utils.filter('PACKAGECONFIG', 'ccache', d)}
>
> inherit_defer ${@bb.utils.filter('PACKAGECONFIG', 'vala', d)}
> EOF
> #
> -------------------------------------------------------------------------------------------------------------------
> bitbake-getvar --value -r inherit PACKAGECONFIG
> bitbake-getvar --value -r inherit CCACHE_TOP_DIR
> bitbake-getvar --value -r inherit VALADEPENDS
> #
> -------------------------------------------------------------------------------------------------------------------
>
>
> The DISTRO_FEATURES is mapped to inherit/inherit_defer as follows:
> 'aaa' maps to 'ccache'
> 'bbb' maps to 'vala'
>
> I chose these two classes, but it could be any others. The idea is to see
> if they were inherited by looking at the class variables.
> The ccache.bbclass was not inherit and so the variable 'CCACHE_TOP_DIR' is
> not defined.
>
>
> Hence my conclusion that DISTRO_FEATURES_DEFAULTS doesn't work for these
> cases, which in my opinion is not expected.
>
> I encountered this problem while debugging the patch
> https://lists.openembedded.org/g/openembedded-devel/topic/118939974.
>
> Best regards,
> Jose
>
Hi Jose,
We can simplify this even further to demonstrate what bitbake is doing:
LICENSE = "MIT"
python () {
d.setVar('MAYBE_VALA', 'vala')
}
inherit_defer ${@bb.utils.filter('MAYBE_VALA', 'vala', d)}
Looking at the output of `bitbake -e`, MAYBE_VALA is set to "vala" but
the vala bbclass is not included.
If we simplify the last line to `inherit_defer ${MAYBE_VALA}` we get a
parse error. It also doesn't work with plain inherit.
This contadicts the example in the bitbake docs [1], which suggests that
a variable assigned in an anonymous Python block can be used with
inherit_defer.
[1]: https://docs.yoctoproject.org/bitbake/dev/bitbake-user-manual/bitbake-user-manual-metadata.html#inherit-defer-directive
I will investigate further!
Best regards,
--
Paul Barker
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 252 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [OE-core] [PATCH v3 2/4] meta: Support opting out of any distro features
2026-04-23 9:11 ` Paul Barker
@ 2026-04-23 9:42 ` Jose Quaresma
[not found] ` <18A8F319FD743690.2028604@lists.openembedded.org>
1 sibling, 0 replies; 10+ messages in thread
From: Jose Quaresma @ 2026-04-23 9:42 UTC (permalink / raw)
To: Paul Barker; +Cc: openembedded-core
[-- Attachment #1: Type: text/plain, Size: 5373 bytes --]
Paul Barker <paul@pbarker.dev> escreveu (quinta, 23/04/2026 à(s) 10:12):
> On Wed, 2026-04-22 at 21:26 +0100, Jose Quaresma wrote:
> > Jose Quaresma via lists.openembedded.org <quaresma.jose=
> > gmail.com@lists.openembedded.org> escreveu (quarta, 22/04/2026 à(s)
> 15:45):
> >
> > > Hi Paul,
> > >
> > > I think I found a bug that is somehow related to
> DISTRO_FEATURES_DEFAULTS.
> > > I don't understand why, but the variables in
> > > DISTRO_FEATURES_DEFAULTS cannot
> > > be used in inherit or inherit_defer.
> > >
> > > Let's use the following recipe as an example:
> > >
> > >
> -------------------------------------------------------------------------------------------------------------------
> > > mkdir -p recipes-test/inherit
> > > cat <<'EOF' >recipes-test/inherit/inherit.bb
> > >
> > > LICENSE = "MIT"
> > >
> > > # FAIL
> > > PACKAGECONFIG ??= " \
> > > ${@bb.utils.contains('DISTRO_FEATURES',
> > > 'gobject-introspection-data', 'ccache', '', d)} \
> > > ${@bb.utils.contains('DISTRO_FEATURES',
> > > 'gobject-introspection-data', 'vala', '', d)} \
> > > "
> > >
> > > # OK
> > > #PACKAGECONFIG ??= " \
> > > # ${@bb.utils.contains('DISTRO_FEATURES', 'systemd', 'ccache',
> '',
> > > d)} \
> > > # ${@bb.utils.contains('DISTRO_FEATURES', 'systemd', 'vala', '',
> > > d)} \
> > > #"
> > >
> > > # get PACKAGECONFIG
> > > # bitbake-getvar -r inherit PACKAGECONFIG
> > >
> > > # get CCACHE_TOP_DIR
> > > # bitbake-getvar -r inherit CCACHE_TOP_DIR
> > > inherit_defer ${@bb.utils.filter('PACKAGECONFIG', 'ccache', d)}
> > >
> > > # get VALADEPENDS
> > > # bitbake-getvar -r inherit VALADEPENDS
> > > inherit_defer ${@bb.utils.filter('PACKAGECONFIG', 'vala', d)}
> > > EOF
> > >
> > >
> -------------------------------------------------------------------------------------------------------------------
> > >
> >
> > A clearer way to replicate the problem:
> >
> > #
> >
> -------------------------------------------------------------------------------------------------------------------
> > mkdir -p recipes-test/inherit
> > cat <<'EOF' >recipes-test/inherit/inherit.bb
> > LICENSE = "MIT"
> >
> > DISTRO_FEATURES_DEFAULTS += "aaa"
> > DISTRO_FEATURES += "bbb"
> >
> > PACKAGECONFIG = " \
> > ${@bb.utils.contains('DISTRO_FEATURES', 'aaa', 'ccache', '', d)}
> \
> > ${@bb.utils.contains('DISTRO_FEATURES', 'bbb', 'vala', '', d)} \
> > "
> >
> > inherit_defer ${@bb.utils.filter('PACKAGECONFIG', 'ccache', d)}
> >
> > inherit_defer ${@bb.utils.filter('PACKAGECONFIG', 'vala', d)}
> > EOF
> > #
> >
> -------------------------------------------------------------------------------------------------------------------
> > bitbake-getvar --value -r inherit PACKAGECONFIG
> > bitbake-getvar --value -r inherit CCACHE_TOP_DIR
> > bitbake-getvar --value -r inherit VALADEPENDS
> > #
> >
> -------------------------------------------------------------------------------------------------------------------
> >
> >
> > The DISTRO_FEATURES is mapped to inherit/inherit_defer as follows:
> > 'aaa' maps to 'ccache'
> > 'bbb' maps to 'vala'
> >
> > I chose these two classes, but it could be any others. The idea is to see
> > if they were inherited by looking at the class variables.
> > The ccache.bbclass was not inherit and so the variable 'CCACHE_TOP_DIR'
> is
> > not defined.
> >
> >
> > Hence my conclusion that DISTRO_FEATURES_DEFAULTS doesn't work for these
> > cases, which in my opinion is not expected.
> >
> > I encountered this problem while debugging the patch
> > https://lists.openembedded.org/g/openembedded-devel/topic/118939974.
> >
> > Best regards,
> > Jose
> >
>
> Hi Jose,
>
> We can simplify this even further to demonstrate what bitbake is doing:
>
> LICENSE = "MIT"
>
> python () {
> d.setVar('MAYBE_VALA', 'vala')
> }
>
> inherit_defer ${@bb.utils.filter('MAYBE_VALA', 'vala', d)}
>
> Looking at the output of `bitbake -e`, MAYBE_VALA is set to "vala" but
> the vala bbclass is not included.
>
> If we simplify the last line to `inherit_defer ${MAYBE_VALA}` we get a
> parse error. It also doesn't work with plain inherit.
> This contadicts the example in the bitbake docs [1], which suggests that
> a variable assigned in an anonymous Python block can be used with
> inherit_defer.
>
> [1]:
> https://docs.yoctoproject.org/bitbake/dev/bitbake-user-manual/bitbake-user-manual-metadata.html#inherit-defer-directive
>
> I will investigate further!
>
Looks like you found a much simpler way to reproduce.
Yesterday I spent all afternoon trying to figure out a way to reproduce it
because at first it didn't make any sense at all.
It seems the problem is somehow related to the python setvar usage.
Given that some of the values in DISTRO_FEATURES are now set using the
setvar function,
like the ones from DISTRO_FEATURES_DEFAULTS, it will suffer from this
problem.
Thus some of the elements in DISTRO_FEATURES won't be usable in
inherit/inherit_defer.
This is a serious bug affecting DISTRO_FEATURES, which is widely used.
I will continue my analysis as well, but I doubt I will find any solution.
Thanks
Jose
> Best regards,
>
> --
> Paul Barker
>
>
--
Best regards,
José Quaresma
[-- Attachment #2: Type: text/html, Size: 8041 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread[parent not found: <18A8F319FD743690.2028604@lists.openembedded.org>]
* Re: [OE-core] [PATCH v3 2/4] meta: Support opting out of any distro features
[not found] ` <18A8F319FD743690.2028604@lists.openembedded.org>
@ 2026-04-24 8:44 ` Jose Quaresma
0 siblings, 0 replies; 10+ messages in thread
From: Jose Quaresma @ 2026-04-24 8:44 UTC (permalink / raw)
To: quaresma.jose; +Cc: Paul Barker, openembedded-core
[-- Attachment #1: Type: text/plain, Size: 6542 bytes --]
Jose Quaresma via lists.openembedded.org <quaresma.jose=
gmail.com@lists.openembedded.org> escreveu (quinta, 23/04/2026 à(s) 10:42):
>
>
> Paul Barker <paul@pbarker.dev> escreveu (quinta, 23/04/2026 à(s) 10:12):
>
>> On Wed, 2026-04-22 at 21:26 +0100, Jose Quaresma wrote:
>> > Jose Quaresma via lists.openembedded.org <quaresma.jose=
>> > gmail.com@lists.openembedded.org> escreveu (quarta, 22/04/2026 à(s)
>> 15:45):
>> >
>> > > Hi Paul,
>> > >
>> > > I think I found a bug that is somehow related to
>> DISTRO_FEATURES_DEFAULTS.
>> > > I don't understand why, but the variables in
>> > > DISTRO_FEATURES_DEFAULTS cannot
>> > > be used in inherit or inherit_defer.
>> > >
>> > > Let's use the following recipe as an example:
>> > >
>> > >
>> -------------------------------------------------------------------------------------------------------------------
>> > > mkdir -p recipes-test/inherit
>> > > cat <<'EOF' >recipes-test/inherit/inherit.bb
>> > >
>> > > LICENSE = "MIT"
>> > >
>> > > # FAIL
>> > > PACKAGECONFIG ??= " \
>> > > ${@bb.utils.contains('DISTRO_FEATURES',
>> > > 'gobject-introspection-data', 'ccache', '', d)} \
>> > > ${@bb.utils.contains('DISTRO_FEATURES',
>> > > 'gobject-introspection-data', 'vala', '', d)} \
>> > > "
>> > >
>> > > # OK
>> > > #PACKAGECONFIG ??= " \
>> > > # ${@bb.utils.contains('DISTRO_FEATURES', 'systemd', 'ccache',
>> '',
>> > > d)} \
>> > > # ${@bb.utils.contains('DISTRO_FEATURES', 'systemd', 'vala',
>> '',
>> > > d)} \
>> > > #"
>> > >
>> > > # get PACKAGECONFIG
>> > > # bitbake-getvar -r inherit PACKAGECONFIG
>> > >
>> > > # get CCACHE_TOP_DIR
>> > > # bitbake-getvar -r inherit CCACHE_TOP_DIR
>> > > inherit_defer ${@bb.utils.filter('PACKAGECONFIG', 'ccache', d)}
>> > >
>> > > # get VALADEPENDS
>> > > # bitbake-getvar -r inherit VALADEPENDS
>> > > inherit_defer ${@bb.utils.filter('PACKAGECONFIG', 'vala', d)}
>> > > EOF
>> > >
>> > >
>> -------------------------------------------------------------------------------------------------------------------
>> > >
>> >
>> > A clearer way to replicate the problem:
>> >
>> > #
>> >
>> -------------------------------------------------------------------------------------------------------------------
>> > mkdir -p recipes-test/inherit
>> > cat <<'EOF' >recipes-test/inherit/inherit.bb
>> > LICENSE = "MIT"
>> >
>> > DISTRO_FEATURES_DEFAULTS += "aaa"
>> > DISTRO_FEATURES += "bbb"
>> >
>> > PACKAGECONFIG = " \
>> > ${@bb.utils.contains('DISTRO_FEATURES', 'aaa', 'ccache', '',
>> d)} \
>> > ${@bb.utils.contains('DISTRO_FEATURES', 'bbb', 'vala', '', d)} \
>> > "
>> >
>> > inherit_defer ${@bb.utils.filter('PACKAGECONFIG', 'ccache', d)}
>> >
>> > inherit_defer ${@bb.utils.filter('PACKAGECONFIG', 'vala', d)}
>> > EOF
>> > #
>> >
>> -------------------------------------------------------------------------------------------------------------------
>> > bitbake-getvar --value -r inherit PACKAGECONFIG
>> > bitbake-getvar --value -r inherit CCACHE_TOP_DIR
>> > bitbake-getvar --value -r inherit VALADEPENDS
>> > #
>> >
>> -------------------------------------------------------------------------------------------------------------------
>> >
>> >
>> > The DISTRO_FEATURES is mapped to inherit/inherit_defer as follows:
>> > 'aaa' maps to 'ccache'
>> > 'bbb' maps to 'vala'
>> >
>> > I chose these two classes, but it could be any others. The idea is to
>> see
>> > if they were inherited by looking at the class variables.
>> > The ccache.bbclass was not inherit and so the variable 'CCACHE_TOP_DIR'
>> is
>> > not defined.
>> >
>> >
>> > Hence my conclusion that DISTRO_FEATURES_DEFAULTS doesn't work for these
>> > cases, which in my opinion is not expected.
>> >
>> > I encountered this problem while debugging the patch
>> > https://lists.openembedded.org/g/openembedded-devel/topic/118939974.
>> >
>> > Best regards,
>> > Jose
>> >
>>
>> Hi Jose,
>>
>> We can simplify this even further to demonstrate what bitbake is doing:
>>
>> LICENSE = "MIT"
>>
>> python () {
>> d.setVar('MAYBE_VALA', 'vala')
>> }
>>
>> inherit_defer ${@bb.utils.filter('MAYBE_VALA', 'vala', d)}
>>
>> Looking at the output of `bitbake -e`, MAYBE_VALA is set to "vala" but
>> the vala bbclass is not included.
>>
>> If we simplify the last line to `inherit_defer ${MAYBE_VALA}` we get a
>> parse error. It also doesn't work with plain inherit.
>
>
>> This contadicts the example in the bitbake docs [1], which suggests that
>> a variable assigned in an anonymous Python block can be used with
>> inherit_defer.
>>
>> [1]:
>> https://docs.yoctoproject.org/bitbake/dev/bitbake-user-manual/bitbake-user-manual-metadata.html#inherit-defer-directive
>>
>> I will investigate further!
>>
>
> Looks like you found a much simpler way to reproduce.
> Yesterday I spent all afternoon trying to figure out a way to reproduce it
> because at first it didn't make any sense at all.
>
> It seems the problem is somehow related to the python setvar usage.
> Given that some of the values in DISTRO_FEATURES are now set using the
> setvar function,
> like the ones from DISTRO_FEATURES_DEFAULTS, it will suffer from this
> problem.
>
> Thus some of the elements in DISTRO_FEATURES won't be usable in
> inherit/inherit_defer.
> This is a serious bug affecting DISTRO_FEATURES, which is widely used.
>
The problem was solved by Richard [1] and the inherit is now working again
as expected.
Thank you for your effort and record-time resolution! I would never have
been able to solve this problem myself.
[1]
https://git.openembedded.org/openembedded-core/commit/?id=6b553a5042b9d5828a9da675ede16e10f06dae90
Jose
>
> I will continue my analysis as well, but I doubt I will find any solution.
>
> Thanks
>
> Jose
>
>
>> Best regards,
>>
>> --
>> Paul Barker
>>
>>
>
> --
> Best regards,
>
> José Quaresma
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#235753):
> https://lists.openembedded.org/g/openembedded-core/message/235753
> Mute This Topic: https://lists.openembedded.org/mt/118620443/5052612
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [
> quaresma.jose@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>
--
Best regards,
José Quaresma
[-- Attachment #2: Type: text/html, Size: 10486 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v3 3/4] meta: Support opting out of any machine features
2026-04-01 17:41 [PATCH v3 0/4] Support opt-out of any default machine and distro features Paul Barker
2026-04-01 17:41 ` [PATCH v3 1/4] oelib: utils: Support filtering default features Paul Barker
2026-04-01 17:41 ` [PATCH v3 2/4] meta: Support opting out of any distro features Paul Barker
@ 2026-04-01 17:41 ` Paul Barker
2026-04-01 17:41 ` [PATCH v3 4/4] lib: oe: Drop backfill support Paul Barker
3 siblings, 0 replies; 10+ messages in thread
From: Paul Barker @ 2026-04-01 17:41 UTC (permalink / raw)
To: openembedded-core; +Cc: Paul Barker
Add default values to MACHINE_FEATURES using the new
filter_default_features() function.
This change obsoletes the variables MACHINE_FEATURES_BACKFILL and
MACHINE_FEATURES_BACKFILL_CONSIDERED. Instead, all defaults are added
via MACHINE_FEATURES_DEFAULTS and users can opt out of any of these
using MACHINE_FEATURES_OPTED_OUT. Hopefully the variable naming here is
easier for people to understand and remember.
Migration notes:
- MACHINE_FEATURES will now get the default features added
automatically. You will need to review these and add any features you
do not want to use to MACHINE_FEATURES_OPTED_OUT.
- If you previously set MACHINE_FEATURES_BACKFILL_CONSIDERED, use the new
variable MACHINE_FEATURES_OPTED_OUT instead.
- If you previously modified MACHINE_FEATURES_BACKFILL, don't do that.
Signed-off-by: Paul Barker <paul@pbarker.dev>
---
meta/classes-global/base.bbclass | 4 +---
meta/classes-recipe/nativesdk.bbclass | 1 -
meta/conf/bitbake.conf | 2 +-
meta/conf/documentation.conf | 4 ++--
meta/conf/machine/include/mips/arch-mips.inc | 2 +-
meta/conf/machine/include/powerpc/tune-power5.inc | 2 +-
meta/conf/machine/include/powerpc/tune-power6.inc | 2 +-
meta/conf/machine/include/powerpc/tune-power7.inc | 2 +-
meta/conf/machine/include/powerpc/tune-ppce5500.inc | 2 +-
meta/conf/machine/include/powerpc/tune-ppce6500.inc | 2 +-
meta/conf/machine/include/x86/arch-x86.inc | 2 +-
11 files changed, 11 insertions(+), 14 deletions(-)
diff --git a/meta/classes-global/base.bbclass b/meta/classes-global/base.bbclass
index 0a4cfd594c72..76fd0ac046a9 100644
--- a/meta/classes-global/base.bbclass
+++ b/meta/classes-global/base.bbclass
@@ -457,9 +457,7 @@ python () {
# Filter default features to allow users to opt out of features they don't
# want.
oe.utils.filter_default_features("DISTRO_FEATURES", d)
-
- # Handle backfilling
- oe.utils.features_backfill("MACHINE_FEATURES", d)
+ oe.utils.filter_default_features("MACHINE_FEATURES", d)
# To add a recipe to the skip list , set:
# SKIP_RECIPE[pn] = "message"
diff --git a/meta/classes-recipe/nativesdk.bbclass b/meta/classes-recipe/nativesdk.bbclass
index a4801c48b2de..80f716fc3c34 100644
--- a/meta/classes-recipe/nativesdk.bbclass
+++ b/meta/classes-recipe/nativesdk.bbclass
@@ -17,7 +17,6 @@ CLASSOVERRIDE = "class-nativesdk"
MACHINEOVERRIDES = ""
MACHINE_FEATURES = "${SDK_MACHINE_FEATURES}"
-MACHINE_FEATURES_BACKFILL = ""
MULTILIBS = ""
diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index f851e2adc9f7..84450386d984 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -912,7 +912,7 @@ DISTRO_FEATURES_NATIVESDK:mingw32 = "x11 ipv6"
DISTRO_FEATURES_FILTER_NATIVE ?= "api-documentation debuginfod opencl opengl wayland"
DISTRO_FEATURES_FILTER_NATIVESDK ?= "api-documentation debuginfod opencl opengl wayland"
-MACHINE_FEATURES_BACKFILL = "rtc qemu-usermode"
+MACHINE_FEATURES_DEFAULTS = "rtc qemu-usermode"
COMBINED_FEATURES = "${@oe.utils.set_intersect('DISTRO_FEATURES', 'MACHINE_FEATURES', d)}"
COMBINED_FEATURES[vardeps] += "DISTRO_FEATURES MACHINE_FEATURES"
diff --git a/meta/conf/documentation.conf b/meta/conf/documentation.conf
index 5c93d0e47970..3c2088754188 100644
--- a/meta/conf/documentation.conf
+++ b/meta/conf/documentation.conf
@@ -277,8 +277,8 @@ MACHINE_ESSENTIAL_EXTRA_RRECOMMENDS[doc] = "A list of recommended machine-specif
MACHINE_EXTRA_RDEPENDS[doc] = "A list of machine-specific packages to install as part of the image being built that are not essential for the machine to boot. However, the build process for more fully-featured images depends on the packages being present."
MACHINE_EXTRA_RRECOMMENDS[doc] = "A list of machine-specific packages to install as part of the image being built that are not essential for booting the machine. The image being built has no build dependencies on the packages in this list."
MACHINE_FEATURES[doc] = "Specifies the list of hardware features the MACHINE supports."
-MACHINE_FEATURES_BACKFILL[doc] = "Features to be added to MACHINE_FEATURES if not also present in MACHINE_FEATURES_BACKFILL_CONSIDERED. This variable is set in the meta/conf/bitbake.conf file and is not intended to be user-configurable."
-MACHINE_FEATURES_BACKFILL_CONSIDERED[doc] = "Features from MACHINE_FEATURES_BACKFILL that should not be backfilled (i.e. added to MACHINE_FEATURES) during the build."
+MACHINE_FEATURES_DEFAULTS[doc] = "The default set of machine features, prior to any filtering. Usually you should leave this alone and modify MACHINE_FEATURES and/or MACHINE_FEATURES_OPTED_OUT."
+MACHINE_FEATURES_OPTED_OUT[doc] = "The set of default machine features to disable. Prefer opting out of features using this variable instead of using MACHINE_FEATURES:remove."
MACHINEOVERRIDES[doc] = "Lists overrides specific to the current machine. By default, this list includes the value of MACHINE."
MAINTAINER[doc] = "The email address of the distribution maintainer."
MIRRORS[doc] = "Specifies additional paths from which the OpenEmbedded build system gets source code."
diff --git a/meta/conf/machine/include/mips/arch-mips.inc b/meta/conf/machine/include/mips/arch-mips.inc
index baadc61d652c..fff2ff639279 100644
--- a/meta/conf/machine/include/mips/arch-mips.inc
+++ b/meta/conf/machine/include/mips/arch-mips.inc
@@ -21,7 +21,7 @@ ABIEXTENSION .= "${@bb.utils.filter('TUNE_FEATURES', 'n32', d)}"
TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'n32', ' -mabi=n32', '', d)}"
# user mode qemu doesn't support mips64 n32: "Invalid ELF image for this architecture"
-MACHINE_FEATURES_BACKFILL_CONSIDERED:append = " ${@bb.utils.contains('TUNE_FEATURES', 'n32', 'qemu-usermode', '', d)}"
+MACHINE_FEATURES_OPTED_OUT += "${@bb.utils.contains('TUNE_FEATURES', 'n32', 'qemu-usermode', '', d)}"
TUNEVALID[n64] = "MIPS64 n64 ABI"
TUNECONFLICTS[n64] = "o32 n32"
diff --git a/meta/conf/machine/include/powerpc/tune-power5.inc b/meta/conf/machine/include/powerpc/tune-power5.inc
index e70e4012176c..452897d2d43d 100644
--- a/meta/conf/machine/include/powerpc/tune-power5.inc
+++ b/meta/conf/machine/include/powerpc/tune-power5.inc
@@ -21,4 +21,4 @@ GLIBC_EXTRA_OECONF:powerpc64 += "${@bb.utils.contains('TUNE_FEATURES', 'power5',
GLIBC_EXTRA_OECONF:powerpc += "${@bb.utils.contains('TUNE_FEATURES', 'power5', '--with-cpu=power5', '', d)}"
# QEMU usermode fails with invalid instruction error
-MACHINE_FEATURES_BACKFILL_CONSIDERED:append = "${@bb.utils.contains('TUNE_FEATURES', 'power5', ' qemu-usermode', '', d)}"
+MACHINE_FEATURES_OPTED_OUT += "${@bb.utils.contains('TUNE_FEATURES', 'power5', 'qemu-usermode', '', d)}"
diff --git a/meta/conf/machine/include/powerpc/tune-power6.inc b/meta/conf/machine/include/powerpc/tune-power6.inc
index eaf89515cad7..da8831058094 100644
--- a/meta/conf/machine/include/powerpc/tune-power6.inc
+++ b/meta/conf/machine/include/powerpc/tune-power6.inc
@@ -21,4 +21,4 @@ GLIBC_EXTRA_OECONF:powerpc64 += "${@bb.utils.contains('TUNE_FEATURES', 'power6',
GLIBC_EXTRA_OECONF:powerpc += "${@bb.utils.contains('TUNE_FEATURES', 'power6', '--with-cpu=power6', '', d)}"
# QEMU usermode fails with invalid instruction error
-MACHINE_FEATURES_BACKFILL_CONSIDERED:append = "${@bb.utils.contains('TUNE_FEATURES', 'power6', ' qemu-usermode', '', d)}"
+MACHINE_FEATURES_OPTED_OUT += "${@bb.utils.contains('TUNE_FEATURES', 'power6', 'qemu-usermode', '', d)}"
diff --git a/meta/conf/machine/include/powerpc/tune-power7.inc b/meta/conf/machine/include/powerpc/tune-power7.inc
index 4531ddd85f48..47887c553f97 100644
--- a/meta/conf/machine/include/powerpc/tune-power7.inc
+++ b/meta/conf/machine/include/powerpc/tune-power7.inc
@@ -21,4 +21,4 @@ GLIBC_EXTRA_OECONF:powerpc64 += "${@bb.utils.contains('TUNE_FEATURES', 'power7',
GLIBC_EXTRA_OECONF:powerpc += "${@bb.utils.contains('TUNE_FEATURES', 'power7', '--with-cpu=power7', '', d)}"
# QEMU usermode fails with invalid instruction error
-MACHINE_FEATURES_BACKFILL_CONSIDERED:append = "${@bb.utils.contains('TUNE_FEATURES', 'power7', ' qemu-usermode', '', d)}"
+MACHINE_FEATURES_OPTED_OUT += "${@bb.utils.contains('TUNE_FEATURES', 'power7', 'qemu-usermode', '', d)}"
diff --git a/meta/conf/machine/include/powerpc/tune-ppce5500.inc b/meta/conf/machine/include/powerpc/tune-ppce5500.inc
index 446b344c3d25..63e745b4a29c 100644
--- a/meta/conf/machine/include/powerpc/tune-ppce5500.inc
+++ b/meta/conf/machine/include/powerpc/tune-ppce5500.inc
@@ -19,4 +19,4 @@ PACKAGE_EXTRA_ARCHS:tune-ppc64e5500 = "${PACKAGE_EXTRA_ARCHS:tune-powerpc64} ppc
QEMU_EXTRAOPTIONS:tune-ppc64e5500 = " -cpu e500mc"
# QEMU usermode fails with invalid instruction error (YOCTO: #10304)
-MACHINE_FEATURES_BACKFILL_CONSIDERED:append = "${@bb.utils.contains('TUNE_FEATURES', 'e5500', ' qemu-usermode', '', d)}"
+MACHINE_FEATURES_OPTED_OUT += "${@bb.utils.contains('TUNE_FEATURES', 'e5500', 'qemu-usermode', '', d)}"
diff --git a/meta/conf/machine/include/powerpc/tune-ppce6500.inc b/meta/conf/machine/include/powerpc/tune-ppce6500.inc
index 4444705b2d6d..77de2f9982bc 100644
--- a/meta/conf/machine/include/powerpc/tune-ppce6500.inc
+++ b/meta/conf/machine/include/powerpc/tune-ppce6500.inc
@@ -19,4 +19,4 @@ PACKAGE_EXTRA_ARCHS:tune-ppc64e6500 = "${PACKAGE_EXTRA_ARCHS:tune-powerpc64} ppc
QEMU_EXTRAOPTIONS:tune-ppc64e6500 = " -cpu e500mc"
# QEMU usermode fails with invalid instruction error (YOCTO: #10304)
-MACHINE_FEATURES_BACKFILL_CONSIDERED:append = "${@bb.utils.contains('TUNE_FEATURES', 'e6500', ' qemu-usermode', '', d)}"
+MACHINE_FEATURES_OPTED_OUT += "${@bb.utils.contains('TUNE_FEATURES', 'e6500', 'qemu-usermode', '', d)}"
diff --git a/meta/conf/machine/include/x86/arch-x86.inc b/meta/conf/machine/include/x86/arch-x86.inc
index 28742e794d63..95b5fe52114c 100644
--- a/meta/conf/machine/include/x86/arch-x86.inc
+++ b/meta/conf/machine/include/x86/arch-x86.inc
@@ -25,7 +25,7 @@ TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'mx32', ' -mx32', '', d)}"
TUNE_LDARGS += "${@bb.utils.contains('TUNE_FEATURES', 'mx32', '-m elf32_x86_64', '', d)}"
TUNE_ASARGS += "${@bb.utils.contains('TUNE_FEATURES', 'mx32', '-x32', '', d)}"
# user mode qemu doesn't support x32
-MACHINE_FEATURES_BACKFILL_CONSIDERED:append = " ${@bb.utils.contains('TUNE_FEATURES', 'mx32', 'qemu-usermode', '', d)}"
+MACHINE_FEATURES_OPTED_OUT += "${@bb.utils.contains('TUNE_FEATURES', 'mx32', 'qemu-usermode', '', d)}"
MACHINEOVERRIDES =. "${@bb.utils.contains('TUNE_FEATURES', 'mx32', 'x86-x32:', '', d)}"
# ELF64 ABI
--
2.43.0
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH v3 4/4] lib: oe: Drop backfill support
2026-04-01 17:41 [PATCH v3 0/4] Support opt-out of any default machine and distro features Paul Barker
` (2 preceding siblings ...)
2026-04-01 17:41 ` [PATCH v3 3/4] meta: Support opting out of any machine features Paul Barker
@ 2026-04-01 17:41 ` Paul Barker
3 siblings, 0 replies; 10+ messages in thread
From: Paul Barker @ 2026-04-01 17:41 UTC (permalink / raw)
To: openembedded-core; +Cc: Paul Barker
We have removed DISTRO_FEATURES_BACKFILL and MACHINE_FEATURES_BACKFILL,
so we no longer need the features_backfill() function.
Signed-off-by: Paul Barker <paul@pbarker.dev>
---
meta/lib/oe/utils.py | 21 ---------------------
1 file changed, 21 deletions(-)
diff --git a/meta/lib/oe/utils.py b/meta/lib/oe/utils.py
index 52f2f1bd55e0..5c722d230842 100644
--- a/meta/lib/oe/utils.py
+++ b/meta/lib/oe/utils.py
@@ -137,27 +137,6 @@ def inherits(d, *classes):
"""Return True if the metadata inherits any of the specified classes"""
return any(bb.data.inherits_class(cls, d) for cls in classes)
-def features_backfill(var,d):
- # This construct allows the addition of new features to variable specified
- # as var
- # Example for var = "DISTRO_FEATURES"
- # This construct allows the addition of new features to DISTRO_FEATURES
- # that if not present would disable existing functionality, without
- # disturbing distributions that have already set DISTRO_FEATURES.
- # Distributions wanting to elide a value in DISTRO_FEATURES_BACKFILL should
- # add the feature to DISTRO_FEATURES_BACKFILL_CONSIDERED
- features = (d.getVar(var) or "").split()
- backfill = (d.getVar(var+"_BACKFILL") or "").split()
- considered = (d.getVar(var+"_BACKFILL_CONSIDERED") or "").split()
-
- addfeatures = []
- for feature in backfill:
- if feature not in features and feature not in considered:
- addfeatures.append(feature)
-
- if addfeatures:
- d.appendVar(var, " " + " ".join(addfeatures))
-
def filter_default_features(varname, d):
# Process default features to exclude features which the user has opted out
# of. The result is appended to the target variable (e.g. DISTRO_FEATURES
--
2.43.0
^ permalink raw reply related [flat|nested] 10+ messages in thread