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

Yonggang Luo posted 4 patches 4 days ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20201015201035.1964-1-luoyonggang@gmail.com
Maintainers: Cleber Rosa <crosa@redhat.com>, Peter Maydell <peter.maydell@linaro.org>, Li-Wen Hsu <lwhsu@freebsd.org>, Eduardo Habkost <ehabkost@redhat.com>, Markus Armbruster <armbru@redhat.com>, Michael Roth <mdroth@linux.vnet.ibm.com>, Ed Maste <emaste@freebsd.org>
.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

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

Posted by Yonggang Luo 4 days ago
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


Re: [PATCH v2 0/4] Fixes docs building on msys2/mingw

Posted by Paolo Bonzini 4 days ago
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
>
>
>

Re: [PATCH v2 0/4] Fixes docs building on msys2/mingw

Posted by 罗勇刚 (Yonggang Luo) 4 days ago
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