From: Marc-André Lureau <marcandre.lureau@redhat.com>

Hi,

Perhaps the last series of preliminary patches before I propose a longer series
to add qemu-common & qga meson subprojects. That's why they are mostly
QGA-related cleanups.

Thanks for the reviews!

Marc-André Lureau (16):
  include: move qemu_*_exec_dir() to cutils
  util/win32: simplify qemu_get_local_state_dir()
  tests: make libqmp buildable for win32
  compiler.h: add QEMU_{BEGIN,END}_IGNORE_INITIALIZER_OVERRIDES
  qobject/json-lexer: disable -Winitializer-overrides warnings
  include: adjust header guards after renaming
  qga: flatten safe_open_or_create()
  osdep: export qemu_open_cloexec()
  qga: use qemu_open_cloexec() for safe_open_or_create()
  qapi/error: add g_autoptr(Error) support
  qga: replace qemu_open_old() with qemu_open_cloexec()
  test/qga: use G_TEST_DIR to locate os-release test file
  qga/wixl: prefer variables over environment
  qga/wixl: require Mingw_bin
  qga/wixl: simplify some pre-processing
  qga/wixl: replace QEMU_GA_MSI_MINGW_BIN_PATH with glib bindir

 configure                            |   9 +-
 meson.build                          |   5 +-
 include/qapi/error.h                 |   2 +
 include/qemu/compiler.h              |  11 ++
 include/qemu/cutils.h                |   7 ++
 include/qemu/help-texts.h            |   4 +-
 include/qemu/osdep.h                 |   9 +-
 tests/qtest/libqmp.h                 |   2 +
 qemu-io.c                            |   1 +
 qga/channel-posix.c                  |  18 ++--
 qga/commands-posix.c                 | 146 +++++++++++++--------------
 qobject/json-lexer.c                 |   4 +
 storage-daemon/qemu-storage-daemon.c |   1 +
 tests/qtest/fuzz/fuzz.c              |   1 +
 tests/qtest/libqmp.c                 |  35 +++++--
 tests/unit/test-qga.c                |  11 +-
 util/cutils.c                        | 108 ++++++++++++++++++++
 util/osdep.c                         |  10 +-
 util/oslib-posix.c                   |  81 ---------------
 util/oslib-win32.c                   |  53 +---------
 qga/installer/qemu-ga.wxs            |  83 +++++----------
 qga/meson.build                      |  11 +-
 22 files changed, 313 insertions(+), 299 deletions(-)

-- 
2.36.0.44.g0f828332d5ac

Looks good apart from the two patches I comment on.

Paolo

Il 4 maggio 2022 19:30:09 CEST, marcandre.lureau@redhat.com ha scritto:
>From: Marc-André Lureau <marcandre.lureau@redhat.com>
>
>Hi,
>
>Perhaps the last series of preliminary patches before I propose a longer series
>to add qemu-common & qga meson subprojects. That's why they are mostly
>QGA-related cleanups.
>
>Thanks for the reviews!
>
>Marc-André Lureau (16):
>  include: move qemu_*_exec_dir() to cutils
>  util/win32: simplify qemu_get_local_state_dir()
>  tests: make libqmp buildable for win32
>  compiler.h: add QEMU_{BEGIN,END}_IGNORE_INITIALIZER_OVERRIDES
>  qobject/json-lexer: disable -Winitializer-overrides warnings
>  include: adjust header guards after renaming
>  qga: flatten safe_open_or_create()
>  osdep: export qemu_open_cloexec()
>  qga: use qemu_open_cloexec() for safe_open_or_create()
>  qapi/error: add g_autoptr(Error) support
>  qga: replace qemu_open_old() with qemu_open_cloexec()
>  test/qga: use G_TEST_DIR to locate os-release test file
>  qga/wixl: prefer variables over environment
>  qga/wixl: require Mingw_bin
>  qga/wixl: simplify some pre-processing
>  qga/wixl: replace QEMU_GA_MSI_MINGW_BIN_PATH with glib bindir
>
> configure                            |   9 +-
> meson.build                          |   5 +-
> include/qapi/error.h                 |   2 +
> include/qemu/compiler.h              |  11 ++
> include/qemu/cutils.h                |   7 ++
> include/qemu/help-texts.h            |   4 +-
> include/qemu/osdep.h                 |   9 +-
> tests/qtest/libqmp.h                 |   2 +
> qemu-io.c                            |   1 +
> qga/channel-posix.c                  |  18 ++--
> qga/commands-posix.c                 | 146 +++++++++++++--------------
> qobject/json-lexer.c                 |   4 +
> storage-daemon/qemu-storage-daemon.c |   1 +
> tests/qtest/fuzz/fuzz.c              |   1 +
> tests/qtest/libqmp.c                 |  35 +++++--
> tests/unit/test-qga.c                |  11 +-
> util/cutils.c                        | 108 ++++++++++++++++++++
> util/osdep.c                         |  10 +-
> util/oslib-posix.c                   |  81 ---------------
> util/oslib-win32.c                   |  53 +---------
> qga/installer/qemu-ga.wxs            |  83 +++++----------
> qga/meson.build                      |  11 +-
> 22 files changed, 313 insertions(+), 299 deletions(-)
>