[libvirt PATCH v2 0/8] meson: Improve handling of tests

Andrea Bolognani posted 8 patches 6 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20231025170701.239137-1-abologna@redhat.com
build-aux/meson.build                         |  99 +++++----
ci/cirrus/build.yml                           |   2 +-
docs/html/meson.build                         |  20 +-
docs/meson.build                              |  24 ++-
meson.build                                   |  75 ++++---
meson_options.txt                             |   2 +-
src/access/meson.build                        |  16 +-
src/meson.build                               | 204 +++++++++---------
.../disk-source-fd.x86_64-latest.args         |  10 +-
.../disk-vhostvdpa.x86_64-latest.args         |   2 +-
tests/qemuxml2argvtest.c                      |   8 +-
11 files changed, 242 insertions(+), 220 deletions(-)
[libvirt PATCH v2 0/8] meson: Improve handling of tests
Posted by Andrea Bolognani 6 months ago
Test pipeline: https://gitlab.com/abologna/libvirt/-/pipelines/1049326523

Changes from [v1]

  * fix test suite on macOS and ensure it is run as part of the
    pipeline (with the previous version it would just always fail);

  * disable a couple of tests (check-html, check-html-references)
    that I had missed the first time around.

[v1] https://listman.redhat.com/archives/libvir-list/2023-October/242491.html

Andrea Bolognani (8):
  tests: Fix some test cases on macOS
  ci: Disable optimizations on macOS
  meson: Do less when not building from git
  meson: Move all handling of test options together
  meson: Handle -Dtests=enabled with Clang
  meson: Make -Dexpensive_tests depend on -Dtests
  meson: Disable all tests when tests are disabled
  meson: Rename build_tests -> tests_enabled

 build-aux/meson.build                         |  99 +++++----
 ci/cirrus/build.yml                           |   2 +-
 docs/html/meson.build                         |  20 +-
 docs/meson.build                              |  24 ++-
 meson.build                                   |  75 ++++---
 meson_options.txt                             |   2 +-
 src/access/meson.build                        |  16 +-
 src/meson.build                               | 204 +++++++++---------
 .../disk-source-fd.x86_64-latest.args         |  10 +-
 .../disk-vhostvdpa.x86_64-latest.args         |   2 +-
 tests/qemuxml2argvtest.c                      |   8 +-
 11 files changed, 242 insertions(+), 220 deletions(-)

-- 
2.41.0
Re: [libvirt PATCH v2 0/8] meson: Improve handling of tests
Posted by Martin Kletzander 6 months ago
On Wed, Oct 25, 2023 at 07:06:53PM +0200, Andrea Bolognani wrote:
>Test pipeline: https://gitlab.com/abologna/libvirt/-/pipelines/1049326523
>
>Changes from [v1]
>
>  * fix test suite on macOS and ensure it is run as part of the
>    pipeline (with the previous version it would just always fail);
>
>  * disable a couple of tests (check-html, check-html-references)
>    that I had missed the first time around.
>
>[v1] https://listman.redhat.com/archives/libvir-list/2023-October/242491.html
>
>Andrea Bolognani (8):
>  tests: Fix some test cases on macOS
>  ci: Disable optimizations on macOS
>  meson: Do less when not building from git
>  meson: Move all handling of test options together
>  meson: Handle -Dtests=enabled with Clang
>  meson: Make -Dexpensive_tests depend on -Dtests
>  meson: Disable all tests when tests are disabled
>  meson: Rename build_tests -> tests_enabled

Reviewed-by: Martin Kletzander <mkletzan@redhat.com>

>
> build-aux/meson.build                         |  99 +++++----
> ci/cirrus/build.yml                           |   2 +-
> docs/html/meson.build                         |  20 +-
> docs/meson.build                              |  24 ++-
> meson.build                                   |  75 ++++---
> meson_options.txt                             |   2 +-
> src/access/meson.build                        |  16 +-
> src/meson.build                               | 204 +++++++++---------
> .../disk-source-fd.x86_64-latest.args         |  10 +-
> .../disk-vhostvdpa.x86_64-latest.args         |   2 +-
> tests/qemuxml2argvtest.c                      |   8 +-
> 11 files changed, 242 insertions(+), 220 deletions(-)
>
>-- 
>2.41.0
>