mbox

[PULL,v2,00/15] Misc patches

Message ID 20220529225137.232359-1-marcandre.lureau@redhat.com
State New
Headers show

Pull-request

git@gitlab.com:marcandre.lureau/qemu.git tags/misc-pull-request

Message

Marc-André Lureau May 29, 2022, 10:51 p.m. UTC
From: Marc-André Lureau <marcandre.lureau@redhat.com>

The following changes since commit 2417cbd5916d043e0c56408221fbe9935d0bc8da:

  Merge tag 'ak-pull-request' of https://gitlab.com/berrange/qemu into staging (2022-05-26 07:00:04 -0700)

are available in the Git repository at:

  git@gitlab.com:marcandre.lureau/qemu.git tags/misc-pull-request

for you to fetch changes up to bb6960a11ae9c9d727bab45c22cb4a9eb12cde55:

  test/qga: use g_auto wherever sensible (2022-05-28 11:42:56 +0200)

----------------------------------------------------------------
Misc cleanups

Mostly qemu-ga related cleanups.

----------------------------------------------------------------

Marc-André Lureau (15):
  include: move qemu_*_exec_dir() to cutils
  util/win32: simplify qemu_get_local_state_dir()
  tests: make libqmp buildable for win32
  qga: flatten safe_open_or_create()
  qga: add qga_open_cloexec() helper
  qga: use qga_open_cloexec() for safe_open_or_create()
  qga: throw an Error in ga_channel_open()
  qga: replace qemu_open_old() with qga_open_cloexec()
  qga: make build_fs_mount_list() return a bool
  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
  test/qga: use g_auto wherever sensible

 configure                            |   9 +-
 meson.build                          |   5 +-
 include/qemu/cutils.h                |   7 ++
 include/qemu/osdep.h                 |   8 --
 qga/cutils.h                         |   8 ++
 tests/qtest/libqmp.h                 |   2 +
 qemu-io.c                            |   1 +
 qga/channel-posix.c                  |  55 +++++-----
 qga/commands-posix.c                 | 154 +++++++++++++--------------
 qga/cutils.c                         |  33 ++++++
 storage-daemon/qemu-storage-daemon.c |   1 +
 tests/qtest/fuzz/fuzz.c              |   1 +
 tests/qtest/libqmp.c                 |  34 +++++-
 tests/unit/test-qga.c                | 130 ++++++++--------------
 util/cutils.c                        | 117 ++++++++++++++++++++
 util/oslib-posix.c                   |  86 +--------------
 util/oslib-win32.c                   |  53 +--------
 qga/installer/qemu-ga.wxs            |  83 +++++----------
 qga/meson.build                      |  12 +--
 19 files changed, 396 insertions(+), 403 deletions(-)
 create mode 100644 qga/cutils.h
 create mode 100644 qga/cutils.c

Comments

Richard Henderson May 30, 2022, 1:58 a.m. UTC | #1
On 5/29/22 15:51, marcandre.lureau@redhat.com wrote:
> From: Marc-André Lureau <marcandre.lureau@redhat.com>
> 
> The following changes since commit 2417cbd5916d043e0c56408221fbe9935d0bc8da:
> 
>    Merge tag 'ak-pull-request' of https://gitlab.com/berrange/qemu into staging (2022-05-26 07:00:04 -0700)
> 
> are available in the Git repository at:
> 
>    git@gitlab.com:marcandre.lureau/qemu.git tags/misc-pull-request
> 
> for you to fetch changes up to bb6960a11ae9c9d727bab45c22cb4a9eb12cde55:
> 
>    test/qga: use g_auto wherever sensible (2022-05-28 11:42:56 +0200)
> 
> ----------------------------------------------------------------
> Misc cleanups
> 
> Mostly qemu-ga related cleanups.

Applied, thanks.  Please update https://wiki.qemu.org/ChangeLog/7.1 as appropriate.


r~


> 
> ----------------------------------------------------------------
> 
> Marc-André Lureau (15):
>    include: move qemu_*_exec_dir() to cutils
>    util/win32: simplify qemu_get_local_state_dir()
>    tests: make libqmp buildable for win32
>    qga: flatten safe_open_or_create()
>    qga: add qga_open_cloexec() helper
>    qga: use qga_open_cloexec() for safe_open_or_create()
>    qga: throw an Error in ga_channel_open()
>    qga: replace qemu_open_old() with qga_open_cloexec()
>    qga: make build_fs_mount_list() return a bool
>    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
>    test/qga: use g_auto wherever sensible
> 
>   configure                            |   9 +-
>   meson.build                          |   5 +-
>   include/qemu/cutils.h                |   7 ++
>   include/qemu/osdep.h                 |   8 --
>   qga/cutils.h                         |   8 ++
>   tests/qtest/libqmp.h                 |   2 +
>   qemu-io.c                            |   1 +
>   qga/channel-posix.c                  |  55 +++++-----
>   qga/commands-posix.c                 | 154 +++++++++++++--------------
>   qga/cutils.c                         |  33 ++++++
>   storage-daemon/qemu-storage-daemon.c |   1 +
>   tests/qtest/fuzz/fuzz.c              |   1 +
>   tests/qtest/libqmp.c                 |  34 +++++-
>   tests/unit/test-qga.c                | 130 ++++++++--------------
>   util/cutils.c                        | 117 ++++++++++++++++++++
>   util/oslib-posix.c                   |  86 +--------------
>   util/oslib-win32.c                   |  53 +--------
>   qga/installer/qemu-ga.wxs            |  83 +++++----------
>   qga/meson.build                      |  12 +--
>   19 files changed, 396 insertions(+), 403 deletions(-)
>   create mode 100644 qga/cutils.h
>   create mode 100644 qga/cutils.c
>