All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Bruce Ashfield" <bruce.ashfield@gmail.com>
To: Christopher Clark <christopher.w.clark@gmail.com>
Cc: meta-virtualization@lists.yoctoproject.org, cardoe@gentoo.org,
	jdmason@kudzu.us, christopher.clark@starlab.io
Subject: Re: [meta-virtualization][PATCH] xen-image-minimal: fix aarch64 build for non-qemuboot-enabled machines
Date: Sun, 1 Aug 2021 23:48:58 -0400	[thread overview]
Message-ID: <20210802034857.GC16820@gmail.com> (raw)
In-Reply-To: <20210802012656.40174-1-christopher.w.clark@gmail.com>

Looks good to me.

I'll wait to hear from Jon that it fixes his problem before merging.

Cheers,

Bruce

In message: [meta-virtualization][PATCH] xen-image-minimal: fix aarch64 build for non-qemuboot-enabled machines
on 01/08/2021 Christopher Clark wrote:
> The qemuboot device tree generation task in qemuboot-xen-dtb.bbclass
> requires the QB_SYSTEM_NAME variable to be set to identify the Qemu
> binary to use. Skip the task if it is not set.
> 
> Reported-by: Jon Mason <jdmason@kudzu.us>
> Signed-off-by: Christopher Clark <christopher.clark@starlab.io>
> ---
>  classes/qemuboot-xen-dtb.bbclass | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/classes/qemuboot-xen-dtb.bbclass b/classes/qemuboot-xen-dtb.bbclass
> index 08f9b02..2d37e91 100644
> --- a/classes/qemuboot-xen-dtb.bbclass
> +++ b/classes/qemuboot-xen-dtb.bbclass
> @@ -176,7 +176,7 @@ do_write_xen_qemuboot_dtb() {
>      # Not all architectures qemuboot with a device tree binary, so check
>      # to see if one is needed. This allows this bbclass file to be used
>      # in the same image recipe for multiple architectures.
> -    if [ -n "${QB_DTB}" ] ; then
> +    if [ -n "${QB_DTB}" ] && [ -n "${QB_SYSTEM_NAME}" ] ; then
>          generate_xen_qemuboot_dtb
>      fi
>  }
> -- 
> 2.25.1
> 

  reply	other threads:[~2021-08-02  3:49 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-08-02  1:26 [meta-virtualization][PATCH] xen-image-minimal: fix aarch64 build for non-qemuboot-enabled machines Christopher Clark
2021-08-02  3:48 ` Bruce Ashfield [this message]
2021-08-02 13:55   ` Jon Mason
2021-08-02 19:51 ` Bruce Ashfield

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210802034857.GC16820@gmail.com \
    --to=bruce.ashfield@gmail.com \
    --cc=cardoe@gentoo.org \
    --cc=christopher.clark@starlab.io \
    --cc=christopher.w.clark@gmail.com \
    --cc=jdmason@kudzu.us \
    --cc=meta-virtualization@lists.yoctoproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.