[libvirt PATCH 0/4] Do not depend on conf/ in util/

Ján Tomko posted 4 patches 4 years, 1 month ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1582467387.git.jtomko@redhat.com
build-aux/syntax-check.mk                    |  5 +-
po/POTFILES.in                               |  4 +-
src/bhyve/Makefile.inc.am                    |  1 +
src/conf/domain_conf.c                       | 44 ++++++++++-
src/conf/domain_conf.h                       | 10 +++
src/hypervisor/Makefile.inc.am               |  4 +
src/{util => hypervisor}/virclosecallbacks.c |  0
src/{util => hypervisor}/virclosecallbacks.h |  0
src/{util => hypervisor}/virhostdev.c        | 43 ----------
src/{util => hypervisor}/virhostdev.h        |  9 ---
src/libvirt_private.syms                     | 83 ++++++++++----------
src/libxl/Makefile.inc.am                    |  1 +
src/util/Makefile.inc.am                     |  4 -
tests/Makefile.am                            |  1 +
14 files changed, 105 insertions(+), 104 deletions(-)
rename src/{util => hypervisor}/virclosecallbacks.c (100%)
rename src/{util => hypervisor}/virclosecallbacks.h (100%)
rename src/{util => hypervisor}/virhostdev.c (98%)
rename src/{util => hypervisor}/virhostdev.h (97%)
[libvirt PATCH 0/4] Do not depend on conf/ in util/
Posted by Ján Tomko 4 years, 1 month ago

Ján Tomko (4):
  syntax-check: inclusion rule for src/hypervisor
  conf: move virHostdevIs functions
  virhostdev: move to src/hypervisor
  virclosecallbacks: move to src/hypervisor

 build-aux/syntax-check.mk                    |  5 +-
 po/POTFILES.in                               |  4 +-
 src/bhyve/Makefile.inc.am                    |  1 +
 src/conf/domain_conf.c                       | 44 ++++++++++-
 src/conf/domain_conf.h                       | 10 +++
 src/hypervisor/Makefile.inc.am               |  4 +
 src/{util => hypervisor}/virclosecallbacks.c |  0
 src/{util => hypervisor}/virclosecallbacks.h |  0
 src/{util => hypervisor}/virhostdev.c        | 43 ----------
 src/{util => hypervisor}/virhostdev.h        |  9 ---
 src/libvirt_private.syms                     | 83 ++++++++++----------
 src/libxl/Makefile.inc.am                    |  1 +
 src/util/Makefile.inc.am                     |  4 -
 tests/Makefile.am                            |  1 +
 14 files changed, 105 insertions(+), 104 deletions(-)
 rename src/{util => hypervisor}/virclosecallbacks.c (100%)
 rename src/{util => hypervisor}/virclosecallbacks.h (100%)
 rename src/{util => hypervisor}/virhostdev.c (98%)
 rename src/{util => hypervisor}/virhostdev.h (97%)

-- 
2.24.1

Re: [libvirt PATCH 0/4] Do not depend on conf/ in util/
Posted by Michal Privoznik 4 years, 1 month ago
On 2/23/20 3:16 PM, Ján Tomko wrote:
> 
> 
> Ján Tomko (4):
>    syntax-check: inclusion rule for src/hypervisor
>    conf: move virHostdevIs functions
>    virhostdev: move to src/hypervisor
>    virclosecallbacks: move to src/hypervisor
> 
>   build-aux/syntax-check.mk                    |  5 +-
>   po/POTFILES.in                               |  4 +-
>   src/bhyve/Makefile.inc.am                    |  1 +
>   src/conf/domain_conf.c                       | 44 ++++++++++-
>   src/conf/domain_conf.h                       | 10 +++
>   src/hypervisor/Makefile.inc.am               |  4 +
>   src/{util => hypervisor}/virclosecallbacks.c |  0
>   src/{util => hypervisor}/virclosecallbacks.h |  0
>   src/{util => hypervisor}/virhostdev.c        | 43 ----------
>   src/{util => hypervisor}/virhostdev.h        |  9 ---
>   src/libvirt_private.syms                     | 83 ++++++++++----------
>   src/libxl/Makefile.inc.am                    |  1 +
>   src/util/Makefile.inc.am                     |  4 -
>   tests/Makefile.am                            |  1 +
>   14 files changed, 105 insertions(+), 104 deletions(-)
>   rename src/{util => hypervisor}/virclosecallbacks.c (100%)
>   rename src/{util => hypervisor}/virclosecallbacks.h (100%)
>   rename src/{util => hypervisor}/virhostdev.c (98%)
>   rename src/{util => hypervisor}/virhostdev.h (97%)
> 

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

Michal