[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] configure: linux-user doesn't need neither fdt
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-devel] [PATCH] configure: linux-user doesn't need neither fdt nor slirp |
Date: |
Fri, 21 Jun 2019 15:11:38 +0200 |
Hi
On Fri, Jun 21, 2019 at 3:05 PM Laurent Vivier <address@hidden> wrote:
>
> if softmmu is not enabled, we disable by default fdt and
> slirp as they are only used by -softmmu targets.
>
> A side effect is the git submodules are not cloned
> if they are not needed.
>
> Clone and build can be forced with --enable-fdt and
> --enable-slirp.
>
> Signed-off-by: Laurent Vivier <address@hidden>
lgtm
Reviewed-by: Marc-André Lureau <address@hidden>
> ---
> configure | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/configure b/configure
> index b091b82cb371..4b3853298c79 100755
> --- a/configure
> +++ b/configure
> @@ -4066,6 +4066,11 @@ if test "$fdt_required" = "yes"; then
> fdt=yes
> fi
>
> +# linux-user doesn't need fdt
> +if test -z "$fdt" -a "$softmmu" != "yes" ; then
> + fdt="no"
> +fi
> +
> if test "$fdt" != "no" ; then
> fdt_libs="-lfdt"
> # explicitly check for libfdt_env.h as it is missing in some stable
> installs
> @@ -5923,6 +5928,11 @@ fi
> ##########################################
> # check for slirp
>
> +# linux-user doesn't need slirp
> +if test -z "$slirp" -a "$softmmu" != "yes" ; then
> + slirp="no"
> +fi
> +
> case "$slirp" in
> "" | yes)
> if $pkg_config slirp; then
> --
> 2.21.0
>