diff mbox series

[v1] alsa-utils: fix install package alsa-utils

Message ID 20260119124722.646039-1-max.oss.09@gmail.com
State New
Headers show
Series [v1] alsa-utils: fix install package alsa-utils | expand

Commit Message

Max Krummenacher Jan. 19, 2026, 12:47 p.m. UTC
From: Max Krummenacher <max.krummenacher@toradex.com>

The recent move to using dynamically split packages created a regression.

Some of the dynamically split packages are empty as the files
dynamically adding the package are packaged through explicit FILES_*
into some other package.
This then breaks the meta package alsa-utils which RDEPENDS on the
not existing empty packages.

Empty packages:
alsa-utils-alsabat-test, alsa-utils-aplaymidi, alsa-utils-aplaymidi2,
alsa-utils-arecordmidi, alsa-utils-arecordmidi2, alsa-utils-axfer

Change the RDEPENDS of alsa-utils to RRECOMMENDS to work around the issue.

Fixes do_rootfs when IMAGE_INSTALL contains alsa-utils:
  - nothing provides alsa-utils-aplaymidi needed by alsa-utils-1.2.15.1-r0.armv7at2hf-neon

Fixes: 26a28ee79350 ("alsa-utils: dynamically split the packages instead of doing it manually")
Signed-off-by: Max Krummenacher <max.krummenacher@toradex.com>
---
 meta/recipes-multimedia/alsa/alsa-utils_1.2.15.1.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Ross Burton Jan. 19, 2026, 1:13 p.m. UTC | #1
There’s a better fix on the list already. see '[OE-core] [PATCH] alsa-utils: fix packaging’.

Ross

> On 19 Jan 2026, at 12:47, max.oss.09@gmail.com wrote:
> 
> From: Max Krummenacher <max.krummenacher@toradex.com>
> 
> The recent move to using dynamically split packages created a regression.
> 
> Some of the dynamically split packages are empty as the files
> dynamically adding the package are packaged through explicit FILES_*
> into some other package.
> This then breaks the meta package alsa-utils which RDEPENDS on the
> not existing empty packages.
> 
> Empty packages:
> alsa-utils-alsabat-test, alsa-utils-aplaymidi, alsa-utils-aplaymidi2,
> alsa-utils-arecordmidi, alsa-utils-arecordmidi2, alsa-utils-axfer
> 
> Change the RDEPENDS of alsa-utils to RRECOMMENDS to work around the issue.
> 
> Fixes do_rootfs when IMAGE_INSTALL contains alsa-utils:
>  - nothing provides alsa-utils-aplaymidi needed by alsa-utils-1.2.15.1-r0.armv7at2hf-neon
> 
> Fixes: 26a28ee79350 ("alsa-utils: dynamically split the packages instead of doing it manually")
> Signed-off-by: Max Krummenacher <max.krummenacher@toradex.com>
> ---
> meta/recipes-multimedia/alsa/alsa-utils_1.2.15.1.bb | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta/recipes-multimedia/alsa/alsa-utils_1.2.15.1.bb b/meta/recipes-multimedia/alsa/alsa-utils_1.2.15.1.bb
> index 927f94afb654..1ed4d9c4b54a 100644
> --- a/meta/recipes-multimedia/alsa/alsa-utils_1.2.15.1.bb
> +++ b/meta/recipes-multimedia/alsa/alsa-utils_1.2.15.1.bb
> @@ -70,7 +70,7 @@ python populate_packages:prepend() {
>     pn = d.getVar("PN")
>     packages = do_split_packages(d, d.getVar("bindir"), r"^([^.]+).*$", pn + "-%s", "alsa-utils tool %s", extra_depends="")
>     packages += do_split_packages(d, d.getVar("sbindir"), r"^([^.]+).*$", pn + "-%s", "alsa-utils tool %s", extra_depends="")
> -    d.setVar("RDEPENDS:" + pn, " ".join(packages))
> +    d.setVar("RRECOMMENDS:" + pn, " ".join(packages))
> }
> 
> PACKAGES_DYNAMIC = "^${PN}-.*"
> -- 
> 2.42.0
>
Yannic Moog Jan. 19, 2026, 1:15 p.m. UTC | #2
On Mon, 2026-01-19 at 13:47 +0100, Max Krummenacher via lists.openembedded.org wrote:
> From: Max Krummenacher <max.krummenacher@toradex.com>
> 
> The recent move to using dynamically split packages created a regression.
> 
> Some of the dynamically split packages are empty as the files
> dynamically adding the package are packaged through explicit FILES_*
> into some other package.
> This then breaks the meta package alsa-utils which RDEPENDS on the
> not existing empty packages.
> 
> Empty packages:
> alsa-utils-alsabat-test, alsa-utils-aplaymidi, alsa-utils-aplaymidi2,
> alsa-utils-arecordmidi, alsa-utils-arecordmidi2, alsa-utils-axfer
> 
> Change the RDEPENDS of alsa-utils to RRECOMMENDS to work around the issue.
> 
> Fixes do_rootfs when IMAGE_INSTALL contains alsa-utils:
>   - nothing provides alsa-utils-aplaymidi needed by alsa-utils-1.2.15.1-r0.armv7at2hf-neon
> 
> Fixes: 26a28ee79350 ("alsa-utils: dynamically split the packages instead of doing it manually")
> Signed-off-by: Max Krummenacher <max.krummenacher@toradex.com>

Reviewed-by: Yannic Moog <y.moog@phytec.de>
Tested-by: Yannic Moog <y.moog@phytec.de>

> ---
>  meta/recipes-multimedia/alsa/alsa-utils_1.2.15.1.bb | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta/recipes-multimedia/alsa/alsa-utils_1.2.15.1.bb b/meta/recipes-
> multimedia/alsa/alsa-utils_1.2.15.1.bb
> index 927f94afb654..1ed4d9c4b54a 100644
> --- a/meta/recipes-multimedia/alsa/alsa-utils_1.2.15.1.bb
> +++ b/meta/recipes-multimedia/alsa/alsa-utils_1.2.15.1.bb
> @@ -70,7 +70,7 @@ python populate_packages:prepend() {
>      pn = d.getVar("PN")
>      packages = do_split_packages(d, d.getVar("bindir"), r"^([^.]+).*$", pn + "-%s", "alsa-utils
> tool %s", extra_depends="")
>      packages += do_split_packages(d, d.getVar("sbindir"), r"^([^.]+).*$", pn + "-%s", "alsa-
> utils tool %s", extra_depends="")
> -    d.setVar("RDEPENDS:" + pn, " ".join(packages))
> +    d.setVar("RRECOMMENDS:" + pn, " ".join(packages))
>  }
>  
>  PACKAGES_DYNAMIC = "^${PN}-.*"
> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#229614): https://lists.openembedded.org/g/openembedded-core/message/229614
> Mute This Topic: https://lists.openembedded.org/mt/117343689/8280565
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [y.moog@phytec.de]
> -=-=-=-=-=-=-=-=-=-=-=-
diff mbox series

Patch

diff --git a/meta/recipes-multimedia/alsa/alsa-utils_1.2.15.1.bb b/meta/recipes-multimedia/alsa/alsa-utils_1.2.15.1.bb
index 927f94afb654..1ed4d9c4b54a 100644
--- a/meta/recipes-multimedia/alsa/alsa-utils_1.2.15.1.bb
+++ b/meta/recipes-multimedia/alsa/alsa-utils_1.2.15.1.bb
@@ -70,7 +70,7 @@  python populate_packages:prepend() {
     pn = d.getVar("PN")
     packages = do_split_packages(d, d.getVar("bindir"), r"^([^.]+).*$", pn + "-%s", "alsa-utils tool %s", extra_depends="")
     packages += do_split_packages(d, d.getVar("sbindir"), r"^([^.]+).*$", pn + "-%s", "alsa-utils tool %s", extra_depends="")
-    d.setVar("RDEPENDS:" + pn, " ".join(packages))
+    d.setVar("RRECOMMENDS:" + pn, " ".join(packages))
 }
 
 PACKAGES_DYNAMIC = "^${PN}-.*"