mbox series

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

Message ID 20201015201035.1964-1-luoyonggang@gmail.com (mailing list archive)
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

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
>
>
>
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