Yocto Meta Virtualization
 help / color / mirror / Atom feed
* [PATCH] xvisor: Remove bb.error when builing for non-supported arch
@ 2022-02-15 14:31 kasper
  2022-02-16  2:36 ` [meta-virtualization] " Alistair Francis
  2022-02-24 15:47 ` Bruce Ashfield
  0 siblings, 2 replies; 3+ messages in thread
From: kasper @ 2022-02-15 14:31 UTC (permalink / raw)
  To: meta-virtualization, mail; +Cc: Kasper Revsbech

From: Kasper Revsbech <kasper.revsbech.ext@siemensgamesa.com>

Change bb.error to bb.note when getting config and arch of target/host
as it is not an error if arch is not supported that should flag when
parsing the recipe.
It is an error if trying to include in image and that is already handled
in COMPATIBLE_HOST

Signed-off-by: Kasper Revsbech <kasper.revsbech.ext@siemensgamesa.com>
---
 recipes-extended/xvisor/xvisor-configs.inc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/recipes-extended/xvisor/xvisor-configs.inc b/recipes-extended/xvisor/xvisor-configs.inc
index f53bba2..cd873cb 100644
--- a/recipes-extended/xvisor/xvisor-configs.inc
+++ b/recipes-extended/xvisor/xvisor-configs.inc
@@ -9,7 +9,7 @@ def get_oemake_config(a, d):
     elif re.match('riscv32(eb|)$', a):        return 'generic-32b-defconfig'
     elif re.match('riscv64(eb|)$', a):        return 'generic-64b-defconfig'
     else:
-        bb.error("cannot map '%s' to a Xvisor defconfig" % a)
+        bb.note("cannot map '%s' to a Xvisor defconfig" % a)
 
 def map_xvisor_arch(a, d):
     import re
@@ -22,4 +22,4 @@ def map_xvisor_arch(a, d):
     elif re.match('aarch64_be_ilp32$', a):      return 'arm'
     elif re.match('riscv(32|64|)(eb|)$', a):    return 'riscv'
     else:
-        bb.error("cannot map '%s' to a Xvisor architecture" % a)
+        bb.note("cannot map '%s' to a Xvisor architecture" % a)
-- 
2.32.0



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [meta-virtualization] [PATCH] xvisor: Remove bb.error when builing for non-supported arch
  2022-02-15 14:31 [PATCH] xvisor: Remove bb.error when builing for non-supported arch kasper
@ 2022-02-16  2:36 ` Alistair Francis
  2022-02-24 15:47 ` Bruce Ashfield
  1 sibling, 0 replies; 3+ messages in thread
From: Alistair Francis @ 2022-02-16  2:36 UTC (permalink / raw)
  To: Kasper; +Cc: meta-virtualization, mail, Kasper Revsbech

On Wed, Feb 16, 2022 at 12:31 AM Kasper <kasper@krinvent.dk> wrote:
>
> From: Kasper Revsbech <kasper.revsbech.ext@siemensgamesa.com>
>
> Change bb.error to bb.note when getting config and arch of target/host
> as it is not an error if arch is not supported that should flag when
> parsing the recipe.
> It is an error if trying to include in image and that is already handled
> in COMPATIBLE_HOST
>
> Signed-off-by: Kasper Revsbech <kasper.revsbech.ext@siemensgamesa.com>

Reviewed-by: Alistair Francis <alistair.francis@wdc.com>

Alistair

> ---
>  recipes-extended/xvisor/xvisor-configs.inc | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/recipes-extended/xvisor/xvisor-configs.inc b/recipes-extended/xvisor/xvisor-configs.inc
> index f53bba2..cd873cb 100644
> --- a/recipes-extended/xvisor/xvisor-configs.inc
> +++ b/recipes-extended/xvisor/xvisor-configs.inc
> @@ -9,7 +9,7 @@ def get_oemake_config(a, d):
>      elif re.match('riscv32(eb|)$', a):        return 'generic-32b-defconfig'
>      elif re.match('riscv64(eb|)$', a):        return 'generic-64b-defconfig'
>      else:
> -        bb.error("cannot map '%s' to a Xvisor defconfig" % a)
> +        bb.note("cannot map '%s' to a Xvisor defconfig" % a)
>
>  def map_xvisor_arch(a, d):
>      import re
> @@ -22,4 +22,4 @@ def map_xvisor_arch(a, d):
>      elif re.match('aarch64_be_ilp32$', a):      return 'arm'
>      elif re.match('riscv(32|64|)(eb|)$', a):    return 'riscv'
>      else:
> -        bb.error("cannot map '%s' to a Xvisor architecture" % a)
> +        bb.note("cannot map '%s' to a Xvisor architecture" % a)
> --
> 2.32.0
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#7060): https://lists.yoctoproject.org/g/meta-virtualization/message/7060
> Mute This Topic: https://lists.yoctoproject.org/mt/89161541/3619028
> Group Owner: meta-virtualization+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-virtualization/unsub [alistair23@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [meta-virtualization] [PATCH] xvisor: Remove bb.error when builing for non-supported arch
  2022-02-15 14:31 [PATCH] xvisor: Remove bb.error when builing for non-supported arch kasper
  2022-02-16  2:36 ` [meta-virtualization] " Alistair Francis
@ 2022-02-24 15:47 ` Bruce Ashfield
  1 sibling, 0 replies; 3+ messages in thread
From: Bruce Ashfield @ 2022-02-24 15:47 UTC (permalink / raw)
  To: Kasper; +Cc: meta-virtualization, mail, Kasper Revsbech

merged!

Bruce

In message: [meta-virtualization] [PATCH] xvisor: Remove bb.error when builing for non-supported arch
on 15/02/2022 Kasper wrote:

> From: Kasper Revsbech <kasper.revsbech.ext@siemensgamesa.com>
> 
> Change bb.error to bb.note when getting config and arch of target/host
> as it is not an error if arch is not supported that should flag when
> parsing the recipe.
> It is an error if trying to include in image and that is already handled
> in COMPATIBLE_HOST
> 
> Signed-off-by: Kasper Revsbech <kasper.revsbech.ext@siemensgamesa.com>
> ---
>  recipes-extended/xvisor/xvisor-configs.inc | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/recipes-extended/xvisor/xvisor-configs.inc b/recipes-extended/xvisor/xvisor-configs.inc
> index f53bba2..cd873cb 100644
> --- a/recipes-extended/xvisor/xvisor-configs.inc
> +++ b/recipes-extended/xvisor/xvisor-configs.inc
> @@ -9,7 +9,7 @@ def get_oemake_config(a, d):
>      elif re.match('riscv32(eb|)$', a):        return 'generic-32b-defconfig'
>      elif re.match('riscv64(eb|)$', a):        return 'generic-64b-defconfig'
>      else:
> -        bb.error("cannot map '%s' to a Xvisor defconfig" % a)
> +        bb.note("cannot map '%s' to a Xvisor defconfig" % a)
>  
>  def map_xvisor_arch(a, d):
>      import re
> @@ -22,4 +22,4 @@ def map_xvisor_arch(a, d):
>      elif re.match('aarch64_be_ilp32$', a):      return 'arm'
>      elif re.match('riscv(32|64|)(eb|)$', a):    return 'riscv'
>      else:
> -        bb.error("cannot map '%s' to a Xvisor architecture" % a)
> +        bb.note("cannot map '%s' to a Xvisor architecture" % a)
> -- 
> 2.32.0
> 

> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#7060): https://lists.yoctoproject.org/g/meta-virtualization/message/7060
> Mute This Topic: https://lists.yoctoproject.org/mt/89161541/1050810
> Group Owner: meta-virtualization+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-virtualization/unsub [bruce.ashfield@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
> 



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-02-24 15:48 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-02-15 14:31 [PATCH] xvisor: Remove bb.error when builing for non-supported arch kasper
2022-02-16  2:36 ` [meta-virtualization] " Alistair Francis
2022-02-24 15:47 ` Bruce Ashfield

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox