[PATCH] configure: the error info not consistence with option.

Yonggang Luo posted 1 patch 3 years, 6 months ago
Test docker-quick@centos7 failed
Test docker-mingw@fedora failed
Test checkpatch failed
Test FreeBSD failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200903220622.504-1-luoyonggang@gmail.com
configure | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] configure: the error info not consistence with option.
Posted by Yonggang Luo 3 years, 6 months ago
Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>
---
 configure | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/configure b/configure
index e42d210e5f..ae63c4fd54 100755
--- a/configure
+++ b/configure
@@ -2545,7 +2545,7 @@ fi
 # vhost-user backends
 test "$vhost_net_user" = "" && vhost_net_user=$vhost_user
 if test "$vhost_net_user" = "yes" && test "$vhost_user" = "no"; then
-  error_exit "--enable-vhost-net-user requires --enable-vhost-user"
+  error_exit "--enable-vhost-net requires --enable-vhost-user"
 fi
 test "$vhost_crypto" = "" && vhost_crypto=$vhost_user
 if test "$vhost_crypto" = "yes" && test "$vhost_user" = "no"; then
-- 
2.28.0.windows.1


Re: [PATCH] configure: the error info not consistence with option.
Posted by Paolo Bonzini 3 years, 6 months ago
>
>  test "$vhost_net_user" = "" && vhost_net_user=$vhost_user
>  if test "$vhost_net_user" = "yes" && test "$vhost_user" = "no"; then
> -  error_exit "--enable-vhost-net-user requires --enable-vhost-user"
> +  error_exit "--enable-vhost-net requires --enable-vhost-user"
>  fi
>

This change seems wrong.

Paolo

 test "$vhost_crypto" = "" && vhost_crypto=$vhost_user
>  if test "$vhost_crypto" = "yes" && test "$vhost_user" = "no"; then
> --
> 2.28.0.windows.1
>
>
Re: [PATCH] configure: the error info not consistence with option.
Posted by 罗勇刚 (Yonggang Luo) 3 years, 6 months ago
On Fri, Sep 4, 2020 at 8:18 AM Paolo Bonzini <pbonzini@redhat.com> wrote:

>  test "$vhost_net_user" = "" && vhost_net_user=$vhost_user
>>  if test "$vhost_net_user" = "yes" && test "$vhost_user" = "no"; then
>> -  error_exit "--enable-vhost-net-user requires --enable-vhost-user"
>> +  error_exit "--enable-vhost-net requires --enable-vhost-user"
>>  fi
>>
>
> This change seems wrong.
>
I didn't found --enable-vhost-net-user option handling?
you may grep it?

>
> Paolo
>
>  test "$vhost_crypto" = "" && vhost_crypto=$vhost_user
>>  if test "$vhost_crypto" = "yes" && test "$vhost_user" = "no"; then
>> --
>> 2.28.0.windows.1
>>
>>

-- 
         此致
礼
罗勇刚
Yours
    sincerely,
Yonggang Luo
Re: [PATCH] configure: the error info not consistence with option.
Posted by Paolo Bonzini 3 years, 6 months ago
On 04/09/20 05:54, 罗勇刚(Yonggang Luo) wrote:
> 
>          test "$vhost_net_user" = "" && vhost_net_user=$vhost_user
>          if test "$vhost_net_user" = "yes" && test "$vhost_user" = "no";
>         then
>         -  error_exit "--enable-vhost-net-user requires --enable-vhost-user"
>         +  error_exit "--enable-vhost-net requires --enable-vhost-user"
>          fi
> 
> 
>     This change seems wrong.
> 
> I didn't found --enable-vhost-net-user option handling?
> you may grep it? 
> 
> 

You're right, it is all dead code.  The four lines are equivalent to
just vhost_net_user=$vhost_user.  You could also add
--enable-vhost-net-user though!

Thanks,

Paolo


Re: [PATCH] configure: the error info not consistence with option.
Posted by 罗勇刚 (Yonggang Luo) 3 years, 6 months ago
On Fri, Sep 4, 2020 at 3:54 PM Paolo Bonzini <pbonzini@redhat.com> wrote:

> On 04/09/20 05:54, 罗勇刚(Yonggang Luo) wrote:
> >
> >          test "$vhost_net_user" = "" && vhost_net_user=$vhost_user
> >          if test "$vhost_net_user" = "yes" && test "$vhost_user" = "no";
> >         then
> >         -  error_exit "--enable-vhost-net-user requires
> --enable-vhost-user"
> >         +  error_exit "--enable-vhost-net requires --enable-vhost-user"
> >          fi
> >
> >
> >     This change seems wrong.
> >
> > I didn't found --enable-vhost-net-user option handling?
> > you may grep it?
> >
> >
>
> You're right, it is all dead code.  The four lines are equivalent to
> just vhost_net_user=$vhost_user.  You could also add
> --enable-vhost-net-user though!
>
I remove those four lines and use vhost_net_user=$vhost_user instead for
simplicity,
if we have no strong need of this option, i guess remove it are better

>
> Thanks,
>
> Paolo
>
>

-- 
         此致
礼
罗勇刚
Yours
    sincerely,
Yonggang Luo