All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Zhang, Chen" <chen.zhang@intel.com>
To: Laurent Vivier <laurent@vivier.eu>, qemu-dev <qemu-devel@nongnu.org>
Cc: Juan Quintela <quintela@redhat.com>,
	"Dr . David Alan Gilbert" <dgilbert@redhat.com>,
	Zhang Chen <zhangckid@gmail.com>
Subject: RE: [PATCH] configure: Fix configure error.
Date: Fri, 13 Mar 2020 08:51:21 +0000	[thread overview]
Message-ID: <3fdff67c4e4047859c0e8aa183bd53d9@intel.com> (raw)
In-Reply-To: <e240baab-18b8-94b9-7ed5-e7ec9daed489@vivier.eu>



> -----Original Message-----
> From: Laurent Vivier <laurent@vivier.eu>
> Sent: Friday, March 13, 2020 4:18 PM
> To: Zhang, Chen <chen.zhang@intel.com>; qemu-dev <qemu-
> devel@nongnu.org>
> Cc: Zhang Chen <zhangckid@gmail.com>; Dr . David Alan Gilbert
> <dgilbert@redhat.com>; Juan Quintela <quintela@redhat.com>
> Subject: Re: [PATCH] configure: Fix configure error.
> 
> Le 13/03/2020 à 07:55, Zhang Chen a écrit :
> > From: Zhang Chen <chen.zhang@intel.com>
> >
> > When run the ./configure will always get this error:
> > Unknown option --exist
> >
> > It caused by this patch:
> > commit 3a67848134d0c07da49033f9ed08bf0ddeec0c6d
> > Author: Juan Quintela <quintela@redhat.com>
> > Date:   Tue Dec 17 21:15:24 2019 +0100
> >
> >     configure: Enable test and libs for zstd
> >
> >     Add it to several build systems to make testing good.
> >
> > Signed-off-by: Zhang Chen <chen.zhang@intel.com>
> > ---
> >  configure | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/configure b/configure
> > index 3c7470096f..305591f7e0 100755
> > --- a/configure
> > +++ b/configure
> > @@ -2475,7 +2475,7 @@ fi
> >  # zstd check
> >
> >  if test "$zstd" != "no" ; then
> > -    if $pkg_config --exist libzstd ; then
> > +    if $pkg_config --exists libzstd ; then
> >          zstd_cflags="$($pkg_config --cflags libzstd)"
> >          zstd_libs="$($pkg_config --libs libzstd)"
> >          LIBS="$zstd_libs $LIBS"
> >
> 
> Juan already sent a patch to fix that:
> 
> [PATCH v2] configure: Improve zstd test
> https://patchew.org/QEMU/20200310111431.173151-1-
> quintela@redhat.com/

OK, Looks haven't merged to upstream.

Thanks
Zhang Chen

> 
> Thanks,
> Laurent

  reply	other threads:[~2020-03-13  8:52 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-03-13  6:55 [PATCH] configure: Fix configure error Zhang Chen
2020-03-13  8:17 ` Laurent Vivier
2020-03-13  8:51   ` Zhang, Chen [this message]
2020-03-13  9:12     ` Dr. David Alan Gilbert

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=3fdff67c4e4047859c0e8aa183bd53d9@intel.com \
    --to=chen.zhang@intel.com \
    --cc=dgilbert@redhat.com \
    --cc=laurent@vivier.eu \
    --cc=qemu-devel@nongnu.org \
    --cc=quintela@redhat.com \
    --cc=zhangckid@gmail.com \
    /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.