mbox series

[v2,0/4] Fixes docs building on msys2/mingw

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

Message

罗勇刚(Yonggang Luo) Oct. 15, 2020, 8:10 p.m. UTC
v1 - v2
Also move the docs configure part from
configure to meson, this also fixed the pending
ninjatool removal caused issue that docs  can
not be build under msys2/mingw

Yonggang Luo (4):
  docs: Fixes build docs on msys2/mingw
  configure: the docdir option should passed to meson as is.
  meson: Move the detection logic for sphinx to meson
  cirrus: Enable doc build on msys2/mingw

 .cirrus.yml                   |  6 +++-
 configure                     | 62 +++--------------------------------
 docs/conf.py                  |  2 +-
 docs/meson.build              |  4 +--
 docs/sphinx/kerneldoc.py      |  2 +-
 meson.build                   | 59 +++++++++++++++++++++++++++++----
 meson_options.txt             |  5 ++-
 scripts/rst-sanitize.py       | 21 ++++++++++++
 tests/qapi-schema/meson.build |  7 ++--
 9 files changed, 95 insertions(+), 73 deletions(-)
 create mode 100644 scripts/rst-sanitize.py

-- 
2.28.0.windows.1

Comments

Paolo Bonzini Oct. 15, 2020, 9:52 p.m. UTC | #1
Looks good, apart from the CR removal patch that can simply be dropped.

Paolo

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

> v1 - v2

> Also move the docs configure part from

> configure to meson, this also fixed the pending

> ninjatool removal caused issue that docs  can

> not be build under msys2/mingw

>

> Yonggang Luo (4):

>   docs: Fixes build docs on msys2/mingw

>   configure: the docdir option should passed to meson as is.

>   meson: Move the detection logic for sphinx to meson

>   cirrus: Enable doc build on msys2/mingw

>

>  .cirrus.yml                   |  6 +++-

>  configure                     | 62 +++--------------------------------

>  docs/conf.py                  |  2 +-

>  docs/meson.build              |  4 +--

>  docs/sphinx/kerneldoc.py      |  2 +-

>  meson.build                   | 59 +++++++++++++++++++++++++++++----

>  meson_options.txt             |  5 ++-

>  scripts/rst-sanitize.py       | 21 ++++++++++++

>  tests/qapi-schema/meson.build |  7 ++--

>  9 files changed, 95 insertions(+), 73 deletions(-)

>  create mode 100644 scripts/rst-sanitize.py

>

> --

> 2.28.0.windows.1

>

>

>
<div dir="auto">Looks good, apart from the CR removal patch that can simply be dropped.<div dir="auto"><br></div><div dir="auto">Paolo</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Il gio 15 ott 2020, 22:10 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">v1 - v2<br>
Also move the docs configure part from<br>
configure to meson, this also fixed the pending<br>
ninjatool removal caused issue that docs  can<br>
not be build under msys2/mingw<br>
<br>
Yonggang Luo (4):<br>
  docs: Fixes build docs on msys2/mingw<br>
  configure: the docdir option should passed to meson as is.<br>
  meson: Move the detection logic for sphinx to meson<br>
  cirrus: Enable doc build on msys2/mingw<br>
<br>
 .cirrus.yml                   |  6 +++-<br>
 configure                     | 62 +++--------------------------------<br>
 docs/conf.py                  |  2 +-<br>
 docs/meson.build              |  4 +--<br>
 docs/sphinx/kerneldoc.py      |  2 +-<br>
 meson.build                   | 59 +++++++++++++++++++++++++++++----<br>
 meson_options.txt             |  5 ++-<br>
 scripts/rst-sanitize.py       | 21 ++++++++++++<br>
 tests/qapi-schema/meson.build |  7 ++--<br>
 9 files changed, 95 insertions(+), 73 deletions(-)<br>
 create mode 100644 scripts/rst-sanitize.py<br>
<br>
-- <br>
2.28.0.windows.1<br>
<br>
<br>
</blockquote></div>
罗勇刚(Yonggang Luo) Oct. 15, 2020, 9:55 p.m. UTC | #2
On Fri, Oct 16, 2020 at 5:53 AM Paolo Bonzini <pbonzini@redhat.com> wrote:
>

> Looks good, apart from the CR removal patch that can simply be dropped.

Resend with fixes, the CR removal patch composite two part.

>

> Paolo

>

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

>>

>> v1 - v2

>> Also move the docs configure part from

>> configure to meson, this also fixed the pending

>> ninjatool removal caused issue that docs  can

>> not be build under msys2/mingw

>>

>> Yonggang Luo (4):

>>   docs: Fixes build docs on msys2/mingw

>>   configure: the docdir option should passed to meson as is.

>>   meson: Move the detection logic for sphinx to meson

>>   cirrus: Enable doc build on msys2/mingw

>>

>>  .cirrus.yml                   |  6 +++-

>>  configure                     | 62 +++--------------------------------

>>  docs/conf.py                  |  2 +-

>>  docs/meson.build              |  4 +--

>>  docs/sphinx/kerneldoc.py      |  2 +-

>>  meson.build                   | 59 +++++++++++++++++++++++++++++----

>>  meson_options.txt             |  5 ++-

>>  scripts/rst-sanitize.py       | 21 ++++++++++++

>>  tests/qapi-schema/meson.build |  7 ++--

>>  9 files changed, 95 insertions(+), 73 deletions(-)

>>  create mode 100644 scripts/rst-sanitize.py

>>

>> --

>> 2.28.0.windows.1

>>

>>



--
         此致
礼
罗勇刚
Yours
    sincerely,
Yonggang Luo
<div dir="ltr"><br><br>On Fri, Oct 16, 2020 at 5:53 AM Paolo Bonzini &lt;<a href="mailto:pbonzini@redhat.com">pbonzini@redhat.com</a>&gt; wrote:<br>&gt;<br>&gt; Looks good, apart from the CR removal patch that can simply be dropped.<div>Resend with fixes, the CR removal patch composite two part.</div><div><br>&gt;<br>&gt; Paolo<br>&gt;<br>&gt; Il gio 15 ott 2020, 22:10 Yonggang Luo &lt;<a href="mailto:luoyonggang@gmail.com">luoyonggang@gmail.com</a>&gt; ha scritto:<br>&gt;&gt;<br>&gt;&gt; v1 - v2<br>&gt;&gt; Also move the docs configure part from<br>&gt;&gt; configure to meson, this also fixed the pending<br>&gt;&gt; ninjatool removal caused issue that docs  can<br>&gt;&gt; not be build under msys2/mingw<br>&gt;&gt;<br>&gt;&gt; Yonggang Luo (4):<br>&gt;&gt;   docs: Fixes build docs on msys2/mingw<br>&gt;&gt;   configure: the docdir option should passed to meson as is.<br>&gt;&gt;   meson: Move the detection logic for sphinx to meson<br>&gt;&gt;   cirrus: Enable doc build on msys2/mingw<br>&gt;&gt;<br>&gt;&gt;  .cirrus.yml                   |  6 +++-<br>&gt;&gt;  configure                     | 62 +++--------------------------------<br>&gt;&gt;  docs/conf.py                  |  2 +-<br>&gt;&gt;  docs/meson.build              |  4 +--<br>&gt;&gt;  docs/sphinx/kerneldoc.py      |  2 +-<br>&gt;&gt;  meson.build                   | 59 +++++++++++++++++++++++++++++----<br>&gt;&gt;  meson_options.txt             |  5 ++-<br>&gt;&gt;  scripts/rst-sanitize.py       | 21 ++++++++++++<br>&gt;&gt;  tests/qapi-schema/meson.build |  7 ++--<br>&gt;&gt;  9 files changed, 95 insertions(+), 73 deletions(-)<br>&gt;&gt;  create mode 100644 scripts/rst-sanitize.py<br>&gt;&gt;<br>&gt;&gt; --<br>&gt;&gt; 2.28.0.windows.1<br>&gt;&gt;<br>&gt;&gt;<br><br><br>--<br>         此致<br>礼<br>罗勇刚<br>Yours<br>    sincerely,<br>Yonggang Luo</div></div>