[meta-arago][master/kirkstone][PATCH] arago-prefs: Change how variants interact with the kernel choice

[meta-arago][master/kirkstone][PATCH] arago-prefs: Change how variants interact with the kernel choice

[Ryan Eatmon]

Rather than just explicitly locking the PREFERRED_PROVIDER_virtual/kernel
to linux-ti-staging and the suffix, use the :append to add the suffix to
whatever value PREFERRED_PROVIDER_virtual/kernel is.  This allows a
machine in a BSP layer to control which kernel to use as the bsse.

In the event that the kernel/variant choice is not valid the error
report from bitbake is very vague, but it does fail so you will not get
an incorrect build.

Signed-off-by: Ryan Eatmon <reatmon@ti.com>
---
 meta-arago-distro/conf/distro/include/arago-prefs.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-arago-distro/conf/distro/include/arago-prefs.inc b/meta-arago-distro/conf/distro/include/arago-prefs.inc
index 2dc4f212..34c3c539 100644
--- a/meta-arago-distro/conf/distro/include/arago-prefs.inc
+++ b/meta-arago-distro/conf/distro/include/arago-prefs.inc
@@ -6,7 +6,7 @@ PREFERRED_PROVIDER_ltp-ddt = "ltp-ddt"
 
 PREFERRED_PROVIDER_virtual/gettext = "gettext"
 
-PREFERRED_PROVIDER_virtual/kernel = "linux-ti-staging${ARAGO_KERNEL_SUFFIX}"
+PREFERRED_PROVIDER_virtual/kernel:append = "${ARAGO_KERNEL_SUFFIX}"
 
 PREFERRED_PROVIDER_jpeg = "libjpeg-turbo"
 
-- 
2.17.1

Re: [meta-arago][master/kirkstone][PATCH] arago-prefs: Change how variants interact with the kernel choice

[Denys Dmytriyenko]

On Wed, Feb 01, 2023 at 08:43:00AM -0600, Ryan Eatmon via lists.yoctoproject.org wrote:
> Rather than just explicitly locking the PREFERRED_PROVIDER_virtual/kernel
> to linux-ti-staging and the suffix, use the :append to add the suffix to
> whatever value PREFERRED_PROVIDER_virtual/kernel is.  This allows a
> machine in a BSP layer to control which kernel to use as the bsse.
> 
> In the event that the kernel/variant choice is not valid the error
> report from bitbake is very vague, but it does fail so you will not get
> an incorrect build.
> 
> Signed-off-by: Ryan Eatmon <reatmon@ti.com>

Acked-by: Denys Dmytriyenko <denys@konsulko.com>


> ---
>  meta-arago-distro/conf/distro/include/arago-prefs.inc | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta-arago-distro/conf/distro/include/arago-prefs.inc b/meta-arago-distro/conf/distro/include/arago-prefs.inc
> index 2dc4f212..34c3c539 100644
> --- a/meta-arago-distro/conf/distro/include/arago-prefs.inc
> +++ b/meta-arago-distro/conf/distro/include/arago-prefs.inc
> @@ -6,7 +6,7 @@ PREFERRED_PROVIDER_ltp-ddt = "ltp-ddt"
>  
>  PREFERRED_PROVIDER_virtual/gettext = "gettext"
>  
> -PREFERRED_PROVIDER_virtual/kernel = "linux-ti-staging${ARAGO_KERNEL_SUFFIX}"
> +PREFERRED_PROVIDER_virtual/kernel:append = "${ARAGO_KERNEL_SUFFIX}"
>  
>  PREFERRED_PROVIDER_jpeg = "libjpeg-turbo"