diff mbox series

[v2,2/4] configure: the docdir option should passed to meson as is.

Message ID 20201015201035.1964-3-luoyonggang@gmail.com
State Superseded
Headers show
Series Fixes docs building on msys2/mingw | expand

Commit Message

罗勇刚(Yonggang Luo) Oct. 15, 2020, 8:10 p.m. UTC
Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>
---
 configure | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Comments

Paolo Bonzini Oct. 15, 2020, 9:23 p.m. UTC | #1
Why?

Paolo

Il gio 15 ott 2020, 22:11 Yonggang Luo <luoyonggang@gmail.com> ha scritto:

> Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>
> ---
>  configure | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/configure b/configure
> index f839c2a557..1ce31f97b4 100755
> --- a/configure
> +++ b/configure
> @@ -971,7 +971,7 @@ for opt do
>    ;;
>    --with-suffix=*) qemu_suffix="$optarg"
>    ;;
> -  --docdir=*) qemu_docdir="$optarg"
> +  --docdir=*) docdir="$optarg"
>    ;;
>    --sysconfdir=*) sysconfdir="$optarg"
>    ;;
> @@ -5770,7 +5770,6 @@ fi
>  qemu_confdir="$sysconfdir/$qemu_suffix"
>  qemu_moddir="$libdir/$qemu_suffix"
>  qemu_datadir="$datadir/$qemu_suffix"
> -qemu_docdir="$docdir/$qemu_suffix"
>  qemu_localedir="$datadir/locale"
>  qemu_icondir="$datadir/icons"
>  qemu_desktopdir="$datadir/applications"
> --
> 2.28.0.windows.1
>
>
<div dir="auto"><div>Why?</div><div dir="auto"><br></div><div dir="auto">Paolo<br><br><div class="gmail_quote" dir="auto"><div dir="ltr" class="gmail_attr">Il gio 15 ott 2020, 22:11 Yonggang Luo &lt;<a href="mailto:luoyonggang@gmail.com">luoyonggang@gmail.com</a>&gt; ha scritto:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Signed-off-by: Yonggang Luo &lt;<a href="mailto:luoyonggang@gmail.com" target="_blank" rel="noreferrer">luoyonggang@gmail.com</a>&gt;<br>
---<br>
 configure | 3 +--<br>
 1 file changed, 1 insertion(+), 2 deletions(-)<br>
<br>
diff --git a/configure b/configure<br>
index f839c2a557..1ce31f97b4 100755<br>
--- a/configure<br>
+++ b/configure<br>
@@ -971,7 +971,7 @@ for opt do<br>
   ;;<br>
   --with-suffix=*) qemu_suffix=&quot;$optarg&quot;<br>
   ;;<br>
-  --docdir=*) qemu_docdir=&quot;$optarg&quot;<br>
+  --docdir=*) docdir=&quot;$optarg&quot;<br>
   ;;<br>
   --sysconfdir=*) sysconfdir=&quot;$optarg&quot;<br>
   ;;<br>
@@ -5770,7 +5770,6 @@ fi<br>
 qemu_confdir=&quot;$sysconfdir/$qemu_suffix&quot;<br>
 qemu_moddir=&quot;$libdir/$qemu_suffix&quot;<br>
 qemu_datadir=&quot;$datadir/$qemu_suffix&quot;<br>
-qemu_docdir=&quot;$docdir/$qemu_suffix&quot;<br>
 qemu_localedir=&quot;$datadir/locale&quot;<br>
 qemu_icondir=&quot;$datadir/icons&quot;<br>
 qemu_desktopdir=&quot;$datadir/applications&quot;<br>
-- <br>
2.28.0.windows.1<br>
<br>
</blockquote></div></div></div>
罗勇刚(Yonggang Luo) Oct. 15, 2020, 9:39 p.m. UTC | #2
Grep  qemu_docdir and docdir  in configure, you will know why,  qemu_docdir
not used at all in configure,

On Fri, Oct 16, 2020 at 5:24 AM Paolo Bonzini <pbonzini@redhat.com> wrote:
>
> Why?
>
> Paolo
>
> Il gio 15 ott 2020, 22:11 Yonggang Luo <luoyonggang@gmail.com> ha scritto:
>>
>> Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>
>> ---
>>  configure | 3 +--
>>  1 file changed, 1 insertion(+), 2 deletions(-)
>>
>> diff --git a/configure b/configure
>> index f839c2a557..1ce31f97b4 100755
>> --- a/configure
>> +++ b/configure
>> @@ -971,7 +971,7 @@ for opt do
>>    ;;
>>    --with-suffix=*) qemu_suffix="$optarg"
>>    ;;
>> -  --docdir=*) qemu_docdir="$optarg"
>> +  --docdir=*) docdir="$optarg"
>>    ;;
>>    --sysconfdir=*) sysconfdir="$optarg"
>>    ;;
>> @@ -5770,7 +5770,6 @@ fi
>>  qemu_confdir="$sysconfdir/$qemu_suffix"
>>  qemu_moddir="$libdir/$qemu_suffix"
>>  qemu_datadir="$datadir/$qemu_suffix"
>> -qemu_docdir="$docdir/$qemu_suffix"
>>  qemu_localedir="$datadir/locale"
>>  qemu_icondir="$datadir/icons"
>>  qemu_desktopdir="$datadir/applications"
>> --
>> 2.28.0.windows.1
>>


--
         此致
礼
罗勇刚
Yours
    sincerely,
Yonggang Luo
<div dir="ltr">Grep 

qemu_docdir and docdir  in configure, you will know why, 

qemu_docdir not used at all in configure,<div><br>On Fri, Oct 16, 2020 at 5:24 AM Paolo Bonzini &lt;<a href="mailto:pbonzini@redhat.com">pbonzini@redhat.com</a>&gt; wrote:<br>&gt;<br>&gt; Why?<br>&gt;<br>&gt; Paolo<br>&gt;<br>&gt; Il gio 15 ott 2020, 22:11 Yonggang Luo &lt;<a href="mailto:luoyonggang@gmail.com">luoyonggang@gmail.com</a>&gt; ha scritto:<br>&gt;&gt;<br>&gt;&gt; Signed-off-by: Yonggang Luo &lt;<a href="mailto:luoyonggang@gmail.com">luoyonggang@gmail.com</a>&gt;<br>&gt;&gt; ---<br>&gt;&gt;  configure | 3 +--<br>&gt;&gt;  1 file changed, 1 insertion(+), 2 deletions(-)<br>&gt;&gt;<br>&gt;&gt; diff --git a/configure b/configure<br>&gt;&gt; index f839c2a557..1ce31f97b4 100755<br>&gt;&gt; --- a/configure<br>&gt;&gt; +++ b/configure<br>&gt;&gt; @@ -971,7 +971,7 @@ for opt do<br>&gt;&gt;    ;;<br>&gt;&gt;    --with-suffix=*) qemu_suffix=&quot;$optarg&quot;<br>&gt;&gt;    ;;<br>&gt;&gt; -  --docdir=*) qemu_docdir=&quot;$optarg&quot;<br>&gt;&gt; +  --docdir=*) docdir=&quot;$optarg&quot;<br>&gt;&gt;    ;;<br>&gt;&gt;    --sysconfdir=*) sysconfdir=&quot;$optarg&quot;<br>&gt;&gt;    ;;<br>&gt;&gt; @@ -5770,7 +5770,6 @@ fi<br>&gt;&gt;  qemu_confdir=&quot;$sysconfdir/$qemu_suffix&quot;<br>&gt;&gt;  qemu_moddir=&quot;$libdir/$qemu_suffix&quot;<br>&gt;&gt;  qemu_datadir=&quot;$datadir/$qemu_suffix&quot;<br>&gt;&gt; -qemu_docdir=&quot;$docdir/$qemu_suffix&quot;<br>&gt;&gt;  qemu_localedir=&quot;$datadir/locale&quot;<br>&gt;&gt;  qemu_icondir=&quot;$datadir/icons&quot;<br>&gt;&gt;  qemu_desktopdir=&quot;$datadir/applications&quot;<br>&gt;&gt; --<br>&gt;&gt; 2.28.0.windows.1<br>&gt;&gt;<br><br><br>--<br>         此致<br>礼<br>罗勇刚<br>Yours<br>    sincerely,<br>Yonggang Luo</div></div>
diff mbox series

Patch

diff --git a/configure b/configure
index f839c2a557..1ce31f97b4 100755
--- a/configure
+++ b/configure
@@ -971,7 +971,7 @@  for opt do
   ;;
   --with-suffix=*) qemu_suffix="$optarg"
   ;;
-  --docdir=*) qemu_docdir="$optarg"
+  --docdir=*) docdir="$optarg"
   ;;
   --sysconfdir=*) sysconfdir="$optarg"
   ;;
@@ -5770,7 +5770,6 @@  fi
 qemu_confdir="$sysconfdir/$qemu_suffix"
 qemu_moddir="$libdir/$qemu_suffix"
 qemu_datadir="$datadir/$qemu_suffix"
-qemu_docdir="$docdir/$qemu_suffix"
 qemu_localedir="$datadir/locale"
 qemu_icondir="$datadir/icons"
 qemu_desktopdir="$datadir/applications"