On Wed, 2020-03-25 at 11:18 +0100, Michal Privoznik wrote: > The virDomainGenerateMachineName() function doesn't belong in > src/conf/ really, because it has nothing to do with domain XML > parsing. It landed there because of lack of better place in the > past. But now that we have src/hypervisor/ the function should > live there. At the same time, the function name is changed to > match new location. > > Signed-off-by: Michal Privoznik <mpriv...@redhat.com> > --- > src/conf/domain_conf.c | 72 --------------------------------- > src/conf/domain_conf.h | 7 ---- > src/hypervisor/domain_driver.c | 74 ++++++++++++++++++++++++++++++++++ > src/hypervisor/domain_driver.h | 7 ++++ > src/libvirt_private.syms | 2 +- > src/lxc/lxc_domain.c | 3 +- > src/qemu/qemu_domain.c | 7 ++-- > tests/virsystemdtest.c | 5 ++- > 8 files changed, 91 insertions(+), 86 deletions(-)
Reviewed-by: Andrea Bolognani <abolo...@redhat.com> -- Andrea Bolognani / Red Hat / Virtualization