mbox series

[PULL,v3,00/22] Build system + misc changes for 2020-10-16

Message ID 20201017144805.1641371-1-pbonzini@redhat.com
Headers show
Series Build system + misc changes for 2020-10-16 | expand

Message

Paolo Bonzini Oct. 17, 2020, 2:48 p.m. UTC 
The following changes since commit 3e40748834923798aa57e3751db13a069e2c617b:

  Merge remote-tracking branch 'remotes/rth/tags/pull-mb-20201014' into staging (2020-10-15 20:30:24 +0100)

are available in the Git repository at:

  https://gitlab.com/bonzini/qemu.git tags/for-upstream

for you to fetch changes up to c47110d90fa5401bcc42c17f8ae0724a1c96599a:

  ci: include configure and meson logs in all jobs if configure fails (2020-10-17 10:45:53 -0400)

----------------------------------------------------------------
* Drop ninjatool and just require ninja (Paolo)
* Fix docs build under msys2 (Yonggang)
* HAX snafu fix (Claudio)
* Disable signal handlers during fuzzing (Alex)
* Miscellaneous fixes (Bruce, Greg)

----------------------------------------------------------------
Alexander Bulekov (1):
      fuzz: Disable QEMU's SIG{INT,HUP,TERM} handlers

Bruce Rogers (3):
      meson.build: don't condition iconv detection on library detection
      configure: fix handling of --docdir parameter
      meson: Only install icons and qemu.desktop if have_system

Claudio Fontana (1):
      hax: unbreak accelerator cpu code after cpus.c split

Greg Kurz (1):
      Makefile: Ensure cscope.out/tags/TAGS are generated in the source tree

Paolo Bonzini (13):
      submodules: bump meson to 0.55.3
      tests/Makefile.include: unbreak non-tcg builds
      make: run shell with pipefail
      tests: add missing generated sources to testqapi
      configure: move QEMU_INCLUDES to meson
      dockerfiles: enable Centos 8 PowerTools
      add ninja to dockerfiles, CI configurations and test VMs
      build: cleanups to Makefile
      build: replace ninjatool with ninja
      build: add --enable/--disable-libudev
      meson: cleanup curses/iconv test
      meson: move SPHINX_ARGS references within "if build_docs"
      ci: include configure and meson logs in all jobs if configure fails

Yonggang Luo (3):
      docs: Fix Sphinx configuration for msys2/mingw
      meson: Move the detection logic for sphinx to meson
      cirrus: Enable doc build on msys2/mingw

 .cirrus.yml                                |   21 +-
 .gitlab-ci.yml                             |    6 +-
 .travis.yml                                |   21 +-
 Makefile                                   |  135 ++--
 configure                                  |  101 +--
 docs/conf.py                               |    2 +-
 docs/devel/build-system.rst                |    6 +-
 docs/meson.build                           |   46 ++
 docs/sphinx/kerneldoc.py                   |    2 +-
 meson                                      |    2 +-
 meson.build                                |  185 ++---
 meson_options.txt                          |    6 +
 scripts/mtest2make.py                      |    3 +-
 scripts/ninjatool.py                       | 1008 ----------------------------
 target/i386/hax-cpus.c                     |    1 +
 tests/Makefile.include                     |    2 +-
 tests/docker/dockerfiles/centos7.docker    |    1 +
 tests/docker/dockerfiles/centos8.docker    |    5 +-
 tests/docker/dockerfiles/debian10.docker   |    1 +
 tests/docker/dockerfiles/fedora.docker     |    1 +
 tests/docker/dockerfiles/travis.docker     |    2 +-
 tests/docker/dockerfiles/ubuntu.docker     |    1 +
 tests/docker/dockerfiles/ubuntu1804.docker |    1 +
 tests/docker/dockerfiles/ubuntu2004.docker |    1 +
 tests/include/meson.build                  |    8 +-
 tests/meson.build                          |   14 +-
 tests/qapi-schema/meson.build              |   88 +--
 tests/qtest/fuzz/fuzz.c                    |    8 +
 tests/vm/centos                            |    2 +-
 tests/vm/centos.aarch64                    |    2 +-
 tests/vm/fedora                            |    2 +-
 tests/vm/freebsd                           |    1 +
 tests/vm/netbsd                            |    1 +
 tests/vm/openbsd                           |    1 +
 tests/vm/ubuntu.aarch64                    |    2 +-
 tests/vm/ubuntu.i386                       |    2 +-
 ui/meson.build                             |    7 +-
 37 files changed, 389 insertions(+), 1309 deletions(-)
 delete mode 100755 scripts/ninjatool.py

Comments

Peter Maydell Oct. 19, 2020, 8:40 a.m. UTC | #1 
On Sat, 17 Oct 2020 at 15:50, Paolo Bonzini <pbonzini@redhat.com> wrote:
>

> The following changes since commit 3e40748834923798aa57e3751db13a069e2c617b:

>

>   Merge remote-tracking branch 'remotes/rth/tags/pull-mb-20201014' into staging (2020-10-15 20:30:24 +0100)

>

> are available in the Git repository at:

>

>   https://gitlab.com/bonzini/qemu.git tags/for-upstream

>

> for you to fetch changes up to c47110d90fa5401bcc42c17f8ae0724a1c96599a:

>

>   ci: include configure and meson logs in all jobs if configure fails (2020-10-17 10:45:53 -0400)

>

> ----------------------------------------------------------------

> * Drop ninjatool and just require ninja (Paolo)

> * Fix docs build under msys2 (Yonggang)

> * HAX snafu fix (Claudio)

> * Disable signal handlers during fuzzing (Alex)

> * Miscellaneous fixes (Bruce, Greg)

>


Applied, thanks.

Please update the changelog at https://wiki.qemu.org/ChangeLog/5.2
for any user-visible changes.

-- PMM
Peter Maydell Oct. 31, 2020, 3:46 p.m. UTC | #2 
On Sat, 17 Oct 2020 at 15:50, Paolo Bonzini <pbonzini@redhat.com> wrote:
> ----------------------------------------------------------------

> * Drop ninjatool and just require ninja (Paolo)

> * Fix docs build under msys2 (Yonggang)

> * HAX snafu fix (Claudio)

> * Disable signal handlers during fuzzing (Alex)

> * Miscellaneous fixes (Bruce, Greg)

>

> Yonggang Luo (3):

>       docs: Fix Sphinx configuration for msys2/mingw

>       meson: Move the detection logic for sphinx to meson

>       cirrus: Enable doc build on msys2/mingw


I've just noticed that there seems to be a minor bug with
the new sphinx detection logic: if the Sphinx is the
wrong version then it prints:

Program sphinx-build found: YES
../../docs/meson.build:30: WARNING:  exists but it is either too old
or uses too old a Python version

ie it hasn't actually managed to substitute in the
program name, so there's just a double-space after
WARNING: instead...

thanks
-- PMM
Paolo Bonzini Oct. 31, 2020, 7:14 p.m. UTC | #3 
Yup, it works only with --sphix-build which obviously is how I tested it...
I will include a fix in my next pull request (I don't really have anything
planned, but something will most likely pop up).

Paolo

Il sab 31 ott 2020, 16:46 Peter Maydell <peter.maydell@linaro.org> ha
scritto:

> On Sat, 17 Oct 2020 at 15:50, Paolo Bonzini <pbonzini@redhat.com> wrote:

> > ----------------------------------------------------------------

> > * Drop ninjatool and just require ninja (Paolo)

> > * Fix docs build under msys2 (Yonggang)

> > * HAX snafu fix (Claudio)

> > * Disable signal handlers during fuzzing (Alex)

> > * Miscellaneous fixes (Bruce, Greg)

> >

> > Yonggang Luo (3):

> >       docs: Fix Sphinx configuration for msys2/mingw

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

> >       cirrus: Enable doc build on msys2/mingw

>

> I've just noticed that there seems to be a minor bug with

> the new sphinx detection logic: if the Sphinx is the

> wrong version then it prints:

>

> Program sphinx-build found: YES

> ../../docs/meson.build:30: WARNING:  exists but it is either too old

> or uses too old a Python version

>

> ie it hasn't actually managed to substitute in the

> program name, so there's just a double-space after

> WARNING: instead...

>

> thanks

> -- PMM

>

>
<div dir="auto">Yup, it works only with --sphix-build which obviously is how I tested it... I will include a fix in my next pull request (I don&#39;t really have anything planned, but something will most likely pop up).<div dir="auto"><br></div><div dir="auto">Paolo</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Il sab 31 ott 2020, 16:46 Peter Maydell &lt;<a href="mailto:peter.maydell@linaro.org">peter.maydell@linaro.org</a>&gt; ha scritto:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Sat, 17 Oct 2020 at 15:50, Paolo Bonzini &lt;<a href="mailto:pbonzini@redhat.com" target="_blank" rel="noreferrer">pbonzini@redhat.com</a>&gt; wrote:<br>
&gt; ----------------------------------------------------------------<br>
&gt; * Drop ninjatool and just require ninja (Paolo)<br>
&gt; * Fix docs build under msys2 (Yonggang)<br>
&gt; * HAX snafu fix (Claudio)<br>
&gt; * Disable signal handlers during fuzzing (Alex)<br>
&gt; * Miscellaneous fixes (Bruce, Greg)<br>
&gt;<br>
&gt; Yonggang Luo (3):<br>
&gt;       docs: Fix Sphinx configuration for msys2/mingw<br>
&gt;       meson: Move the detection logic for sphinx to meson<br>
&gt;       cirrus: Enable doc build on msys2/mingw<br>
<br>
I&#39;ve just noticed that there seems to be a minor bug with<br>
the new sphinx detection logic: if the Sphinx is the<br>
wrong version then it prints:<br>
<br>
Program sphinx-build found: YES<br>
../../docs/meson.build:30: WARNING:  exists but it is either too old<br>
or uses too old a Python version<br>
<br>
ie it hasn&#39;t actually managed to substitute in the<br>
program name, so there&#39;s just a double-space after<br>
WARNING: instead...<br>
<br>
thanks<br>
-- PMM<br>
<br>
</blockquote></div>