[libvirt PATCH 0/4] Use g_strerror instead of virStrerror (glib chronicles)

Ján Tomko posted 4 patches 4 years, 8 months ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1583846630.git.jtomko@redhat.com
docs/hacking.html.in            |  3 +--
src/libvirt_private.syms        |  1 -
src/libxl/libxl_conf.c          |  3 +--
src/libxl/libxl_driver.c        | 11 +++++------
src/libxl/libxl_logger.c        |  6 ++----
src/locking/lock_daemon.c       |  9 +++------
src/logging/log_daemon.c        |  9 +++------
src/lxc/lxc_container.c         |  3 +--
src/lxc/lxc_process.c           |  7 +++----
src/network/bridge_driver.c     |  3 +--
src/qemu/qemu_capabilities.c    |  3 +--
src/qemu/qemu_interface.c       |  3 +--
src/qemu/qemu_process.c         |  9 +++------
src/remote/remote_daemon.c      | 13 ++++---------
src/security/security_manager.c |  5 ++---
src/security/security_selinux.c |  5 ++---
src/storage/storage_driver.c    |  3 +--
src/util/viraudit.c             |  3 +--
src/util/vircommand.c           |  3 +--
src/util/virerror.c             | 25 -------------------------
src/util/virerror.h             |  2 --
src/util/virfile.c              |  3 +--
src/util/virhostcpu.c           |  8 +++-----
src/util/virlockspace.c         |  9 +++------
src/util/virpci.c               | 12 ++++--------
src/util/virpidfile.c           |  3 +--
src/util/virprocess.c           |  3 +--
src/util/virutil.c              |  6 ++----
tools/vsh.c                     | 29 ++++++++++-------------------
29 files changed, 61 insertions(+), 141 deletions(-)
[libvirt PATCH 0/4] Use g_strerror instead of virStrerror (glib chronicles)
Posted by Ján Tomko 4 years, 8 months ago

Ján Tomko (4):
  Use g_strerror instead of virStrerror
  util: remove virStrerror
  tools: vsh.c: remove virstrerror.h include
  docs: hacking: move virStrerror to removed functions

 docs/hacking.html.in            |  3 +--
 src/libvirt_private.syms        |  1 -
 src/libxl/libxl_conf.c          |  3 +--
 src/libxl/libxl_driver.c        | 11 +++++------
 src/libxl/libxl_logger.c        |  6 ++----
 src/locking/lock_daemon.c       |  9 +++------
 src/logging/log_daemon.c        |  9 +++------
 src/lxc/lxc_container.c         |  3 +--
 src/lxc/lxc_process.c           |  7 +++----
 src/network/bridge_driver.c     |  3 +--
 src/qemu/qemu_capabilities.c    |  3 +--
 src/qemu/qemu_interface.c       |  3 +--
 src/qemu/qemu_process.c         |  9 +++------
 src/remote/remote_daemon.c      | 13 ++++---------
 src/security/security_manager.c |  5 ++---
 src/security/security_selinux.c |  5 ++---
 src/storage/storage_driver.c    |  3 +--
 src/util/viraudit.c             |  3 +--
 src/util/vircommand.c           |  3 +--
 src/util/virerror.c             | 25 -------------------------
 src/util/virerror.h             |  2 --
 src/util/virfile.c              |  3 +--
 src/util/virhostcpu.c           |  8 +++-----
 src/util/virlockspace.c         |  9 +++------
 src/util/virpci.c               | 12 ++++--------
 src/util/virpidfile.c           |  3 +--
 src/util/virprocess.c           |  3 +--
 src/util/virutil.c              |  6 ++----
 tools/vsh.c                     | 29 ++++++++++-------------------
 29 files changed, 61 insertions(+), 141 deletions(-)

-- 
2.24.1

Re: [libvirt PATCH 0/4] Use g_strerror instead of virStrerror (glib chronicles)
Posted by Michal Privoznik 4 years, 8 months ago
On 3/10/20 2:24 PM, Ján Tomko wrote:
> 
> 
> Ján Tomko (4):
>    Use g_strerror instead of virStrerror
>    util: remove virStrerror
>    tools: vsh.c: remove virstrerror.h include
>    docs: hacking: move virStrerror to removed functions
> 
>   docs/hacking.html.in            |  3 +--
>   src/libvirt_private.syms        |  1 -
>   src/libxl/libxl_conf.c          |  3 +--
>   src/libxl/libxl_driver.c        | 11 +++++------
>   src/libxl/libxl_logger.c        |  6 ++----
>   src/locking/lock_daemon.c       |  9 +++------
>   src/logging/log_daemon.c        |  9 +++------
>   src/lxc/lxc_container.c         |  3 +--
>   src/lxc/lxc_process.c           |  7 +++----
>   src/network/bridge_driver.c     |  3 +--
>   src/qemu/qemu_capabilities.c    |  3 +--
>   src/qemu/qemu_interface.c       |  3 +--
>   src/qemu/qemu_process.c         |  9 +++------
>   src/remote/remote_daemon.c      | 13 ++++---------
>   src/security/security_manager.c |  5 ++---
>   src/security/security_selinux.c |  5 ++---
>   src/storage/storage_driver.c    |  3 +--
>   src/util/viraudit.c             |  3 +--
>   src/util/vircommand.c           |  3 +--
>   src/util/virerror.c             | 25 -------------------------
>   src/util/virerror.h             |  2 --
>   src/util/virfile.c              |  3 +--
>   src/util/virhostcpu.c           |  8 +++-----
>   src/util/virlockspace.c         |  9 +++------
>   src/util/virpci.c               | 12 ++++--------
>   src/util/virpidfile.c           |  3 +--
>   src/util/virprocess.c           |  3 +--
>   src/util/virutil.c              |  6 ++----
>   tools/vsh.c                     | 29 ++++++++++-------------------
>   29 files changed, 61 insertions(+), 141 deletions(-)
> 

Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

Michal