[libvirt] [libvirt-glib 1/4] Introduce libvirt-gconfig-private.h
This header groups all libvirt-gconfig private headers. --- libvirt-gconfig/Makefile.am | 1 + .../libvirt-gconfig-domain-chardev-source-pty.c | 2 +- libvirt-gconfig/libvirt-gconfig-domain-chardev.c | 2 +- libvirt-gconfig/libvirt-gconfig-domain-clock.c | 3 +- libvirt-gconfig/libvirt-gconfig-domain-device.c | 1 + libvirt-gconfig/libvirt-gconfig-domain-disk.c | 3 +- libvirt-gconfig/libvirt-gconfig-domain-filesys.c | 3 +- .../libvirt-gconfig-domain-graphics-sdl.c | 3 +- .../libvirt-gconfig-domain-graphics-spice.c | 3 +- .../libvirt-gconfig-domain-graphics-vnc.c | 3 +- libvirt-gconfig/libvirt-gconfig-domain-input.c | 3 +- .../libvirt-gconfig-domain-interface-network.c | 3 +- .../libvirt-gconfig-domain-interface-user.c | 3 +- libvirt-gconfig/libvirt-gconfig-domain-interface.c | 2 +- .../libvirt-gconfig-domain-memballoon.c | 3 +- libvirt-gconfig/libvirt-gconfig-domain-os.c | 3 +- libvirt-gconfig/libvirt-gconfig-domain-seclabel.c | 3 +- libvirt-gconfig/libvirt-gconfig-domain-sound.c | 2 +- libvirt-gconfig/libvirt-gconfig-domain-video.c | 3 +- libvirt-gconfig/libvirt-gconfig-domain.c | 3 +- libvirt-gconfig/libvirt-gconfig-object.c | 4 +-- libvirt-gconfig/libvirt-gconfig-private.h | 30 ++++++++++++++++++++ .../libvirt-gconfig-storage-permissions.c | 2 +- .../libvirt-gconfig-storage-pool-source.c | 2 +- .../libvirt-gconfig-storage-pool-target.c | 2 +- libvirt-gconfig/libvirt-gconfig-storage-pool.c | 2 +- .../libvirt-gconfig-storage-vol-backing-store.c | 2 +- .../libvirt-gconfig-storage-vol-target.c | 2 +- libvirt-gconfig/libvirt-gconfig-storage-vol.c | 2 +- 29 files changed, 58 insertions(+), 42 deletions(-) create mode 100644 libvirt-gconfig/libvirt-gconfig-private.h diff --git a/libvirt-gconfig/Makefile.am b/libvirt-gconfig/Makefile.am index 6342431..d9eb25a 100644 --- a/libvirt-gconfig/Makefile.am +++ b/libvirt-gconfig/Makefile.am @@ -51,6 +51,7 @@ GCONFIG_HEADER_FILES = \ libvirt-gconfig-storage-vol-backing-store.h \ libvirt-gconfig-storage-vol-target.h noinst_HEADERS = \ + libvirt-gconfig-private.h \ libvirt-gconfig-helpers-private.h \ libvirt-gconfig-object-private.h \ libvirt-gconfig-xml-doc.h diff --git a/libvirt-gconfig/libvirt-gconfig-domain-chardev-source-pty.c b/libvirt-gconfig/libvirt-gconfig-domain-chardev-source-pty.c index 1965444..e3c4c25 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-chardev-source-pty.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-chardev-source-pty.c @@ -23,7 +23,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_DOMAIN_CHARDEV_SOURCE_PTY_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_CHARDEV_SOURCE_PTY, GVirConfigDomainChardevSourcePtyPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-chardev.c b/libvirt-gconfig/libvirt-gconfig-domain-chardev.c index 1359f8f..620773c 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-chardev.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-chardev.c @@ -23,7 +23,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_DOMAIN_CHARDEV_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_CHARDEV, GVirConfigDomainChardevPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-clock.c b/libvirt-gconfig/libvirt-gconfig-domain-clock.c index 16d75d5..3cb8ffb 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-clock.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-clock.c @@ -23,8 +23,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_DOMAIN_CLOCK_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_CLOCK, GVirConfigDomainClockPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-device.c b/libvirt-gconfig/libvirt-gconfig-domain-device.c index 980ae70..8fcf60f 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-device.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-device.c @@ -23,6 +23,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_DOMAIN_DEVICE_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_DEVICE, GVirConfigDomainDevicePrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-disk.c b/libvirt-gconfig/libvirt-gconfig-domain-disk.c index 18cdb1c..1f60213 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-disk.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-disk.c @@ -23,8 +23,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_DOMAIN_DISK_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_DISK, GVirConfigDomainDiskPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-filesys.c b/libvirt-gconfig/libvirt-gconfig-domain-filesys.c index 6c3aa9d..ef942f8 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-filesys.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-filesys.c @@ -23,8 +23,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_DOMAIN_FILESYS_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_FILESYS, GVirConfigDomainFilesysPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-graphics-sdl.c b/libvirt-gconfig/libvirt-gconfig-domain-graphics-sdl.c index c60e72b..1e33a1b 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-graphics-sdl.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-graphics-sdl.c @@ -23,8 +23,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_DOMAIN_GRAPHICS_SDL_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_GRAPHICS_SDL, GVirConfigDomainGraphicsSdlPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-graphics-spice.c b/libvirt-gconfig/libvirt-gconfig-domain-graphics-spice.c index 321b914..7aa3e2c 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-graphics-spice.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-graphics-spice.c @@ -23,8 +23,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_DOMAIN_GRAPHICS_SPICE_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_GRAPHICS_SPICE, GVirConfigDomainGraphicsSpicePrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-graphics-vnc.c b/libvirt-gconfig/libvirt-gconfig-domain-graphics-vnc.c index f7fd6db..b00a155 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-graphics-vnc.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-graphics-vnc.c @@ -23,8 +23,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_DOMAIN_GRAPHICS_VNC_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_GRAPHICS_VNC, GVirConfigDomainGraphicsVncPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-input.c b/libvirt-gconfig/libvirt-gconfig-domain-input.c index 5bc70c8..bacc693 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-input.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-input.c @@ -23,8 +23,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_DOMAIN_INPUT_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_INPUT, GVirConfigDomainInputPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-interface-network.c b/libvirt-gconfig/libvirt-gconfig-domain-interface-network.c index becddc8..6f24708 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-interface-network.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-interface-network.c @@ -25,8 +25,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_DOMAIN_INTERFACE_NETWORK_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_INTERFACE_NETWORK, GVirConfigDomainInterfaceNetworkPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-interface-user.c b/libvirt-gconfig/libvirt-gconfig-domain-interface-user.c index 2c03af2..25770e0 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-interface-user.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-interface-user.c @@ -25,8 +25,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_DOMAIN_INTERFACE_USER_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_INTERFACE_USER, GVirConfigDomainInterfaceUserPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-interface.c b/libvirt-gconfig/libvirt-gconfig-domain-interface.c index 834ce5d..2d87040 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-interface.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-interface.c @@ -25,7 +25,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_DOMAIN_INTERFACE_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_INTERFACE, GVirConfigDomainInterfacePrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-memballoon.c b/libvirt-gconfig/libvirt-gconfig-domain-memballoon.c index ca74a72..9df7e52 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-memballoon.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-memballoon.c @@ -23,8 +23,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_DOMAIN_MEMBALLOON_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_MEMBALLOON, GVirConfigDomainMemballoonPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-os.c b/libvirt-gconfig/libvirt-gconfig-domain-os.c index 2b0cdff..6818abf 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-os.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-os.c @@ -23,8 +23,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_DOMAIN_OS_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_OS, GVirConfigDomainOsPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-seclabel.c b/libvirt-gconfig/libvirt-gconfig-domain-seclabel.c index 7537a78..2d69c6f 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-seclabel.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-seclabel.c @@ -23,8 +23,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_DOMAIN_SECLABEL_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_SECLABEL, GVirConfigDomainSeclabelPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-sound.c b/libvirt-gconfig/libvirt-gconfig-domain-sound.c index 56149ca..abf9a14 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-sound.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-sound.c @@ -23,7 +23,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_DOMAIN_SOUND_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_SOUND, GVirConfigDomainSoundPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-video.c b/libvirt-gconfig/libvirt-gconfig-domain-video.c index bba8073..90c71da 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-video.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-video.c @@ -23,8 +23,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_DOMAIN_VIDEO_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_VIDEO, GVirConfigDomainVideoPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain.c b/libvirt-gconfig/libvirt-gconfig-domain.c index c5df963..ed48d57 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain.c +++ b/libvirt-gconfig/libvirt-gconfig-domain.c @@ -24,8 +24,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_DOMAIN_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN, GVirConfigDomainPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-object.c b/libvirt-gconfig/libvirt-gconfig-object.c index cf71264..ce6cd21 100644 --- a/libvirt-gconfig/libvirt-gconfig-object.c +++ b/libvirt-gconfig/libvirt-gconfig-object.c @@ -28,9 +28,7 @@ #include <libxml/relaxng.h> #include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" -#include "libvirt-gconfig/libvirt-gconfig-xml-doc.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_OBJECT_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_OBJECT, GVirConfigObjectPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-private.h b/libvirt-gconfig/libvirt-gconfig-private.h new file mode 100644 index 0000000..e985ebd --- /dev/null +++ b/libvirt-gconfig/libvirt-gconfig-private.h @@ -0,0 +1,30 @@ +/* + * libvirt-gconfig-private.h: libvirt domain device configuration + * + * Copyright (C) 2011 Red Hat, Inc. + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA + * + * Author: Christophe Fergeau <cfergeau@gmail.com> + */ + +#ifndef __LIBVIRT_GCONFIG_PRIVATE_H__ +#define __LIBVIRT_GCONFIG_PRIVATE_H__ + +#include <libvirt-gconfig/libvirt-gconfig-helpers-private.h> +#include <libvirt-gconfig/libvirt-gconfig-object-private.h> +#include <libvirt-gconfig/libvirt-gconfig-xml-doc.h> + +#endif /* __LIBVIRT_GCONFIG_PRIVATE_H__ */ diff --git a/libvirt-gconfig/libvirt-gconfig-storage-permissions.c b/libvirt-gconfig/libvirt-gconfig-storage-permissions.c index c5f9f49..49e28ac 100644 --- a/libvirt-gconfig/libvirt-gconfig-storage-permissions.c +++ b/libvirt-gconfig/libvirt-gconfig-storage-permissions.c @@ -23,7 +23,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_STORAGE_PERMISSIONS_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_STORAGE_PERMISSIONS, GVirConfigStoragePermissionsPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-storage-pool-source.c b/libvirt-gconfig/libvirt-gconfig-storage-pool-source.c index b2223a2..fafb755 100644 --- a/libvirt-gconfig/libvirt-gconfig-storage-pool-source.c +++ b/libvirt-gconfig/libvirt-gconfig-storage-pool-source.c @@ -23,7 +23,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_STORAGE_POOL_SOURCE_GET_PRIVATE(obj) \ diff --git a/libvirt-gconfig/libvirt-gconfig-storage-pool-target.c b/libvirt-gconfig/libvirt-gconfig-storage-pool-target.c index a5af381..4ce5a12 100644 --- a/libvirt-gconfig/libvirt-gconfig-storage-pool-target.c +++ b/libvirt-gconfig/libvirt-gconfig-storage-pool-target.c @@ -23,7 +23,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_STORAGE_POOL_TARGET_GET_PRIVATE(obj) \ diff --git a/libvirt-gconfig/libvirt-gconfig-storage-pool.c b/libvirt-gconfig/libvirt-gconfig-storage-pool.c index 95609bb..c22e160 100644 --- a/libvirt-gconfig/libvirt-gconfig-storage-pool.c +++ b/libvirt-gconfig/libvirt-gconfig-storage-pool.c @@ -25,7 +25,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_STORAGE_POOL_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_STORAGE_POOL, GVirConfigStoragePoolPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-storage-vol-backing-store.c b/libvirt-gconfig/libvirt-gconfig-storage-vol-backing-store.c index 2530ed3..1791ba0 100644 --- a/libvirt-gconfig/libvirt-gconfig-storage-vol-backing-store.c +++ b/libvirt-gconfig/libvirt-gconfig-storage-vol-backing-store.c @@ -23,7 +23,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_STORAGE_VOL_BACKING_STORE_GET_PRIVATE(obj) \ diff --git a/libvirt-gconfig/libvirt-gconfig-storage-vol-target.c b/libvirt-gconfig/libvirt-gconfig-storage-vol-target.c index 6a3bb0c..373f684 100644 --- a/libvirt-gconfig/libvirt-gconfig-storage-vol-target.c +++ b/libvirt-gconfig/libvirt-gconfig-storage-vol-target.c @@ -23,7 +23,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_STORAGE_VOL_TARGET_GET_PRIVATE(obj) \ diff --git a/libvirt-gconfig/libvirt-gconfig-storage-vol.c b/libvirt-gconfig/libvirt-gconfig-storage-vol.c index 079a5fd..1864447 100644 --- a/libvirt-gconfig/libvirt-gconfig-storage-vol.c +++ b/libvirt-gconfig/libvirt-gconfig-storage-vol.c @@ -24,7 +24,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_STORAGE_VOL_GET_PRIVATE(obj) \ -- 1.7.7.5
This will be helpful to implement GVirConfig getters. --- libvirt-gconfig/libvirt-gconfig-helpers-private.h | 5 ++ libvirt-gconfig/libvirt-gconfig-helpers.c | 52 ++++++++++++++++++-- 2 files changed, 51 insertions(+), 6 deletions(-) diff --git a/libvirt-gconfig/libvirt-gconfig-helpers-private.h b/libvirt-gconfig/libvirt-gconfig-helpers-private.h index 4c4a539..5c98a58 100644 --- a/libvirt-gconfig/libvirt-gconfig-helpers-private.h +++ b/libvirt-gconfig/libvirt-gconfig-helpers-private.h @@ -50,7 +50,12 @@ xmlChar * gvir_config_xml_get_child_element_content (xmlNode *node, const char *child_name); char *gvir_config_xml_get_child_element_content_glib (xmlNode *node, const char *child_name); +xmlChar *gvir_config_xml_get_attribute_content(xmlNodePtr node, + const char *attr_name); +char *gvir_config_xml_get_attribute_content_glib(xmlNodePtr node, + const char *attr_name); const char *gvir_config_genum_get_nick (GType enum_type, gint value); +int gvir_config_genum_get_value (GType enum_type, const char *nick); G_END_DECLS #endif /* __LIBVIRT_GCONFIG_HELPERS_PRIVATE_H__ */ diff --git a/libvirt-gconfig/libvirt-gconfig-helpers.c b/libvirt-gconfig/libvirt-gconfig-helpers.c index c1c23da..75ad2b2 100644 --- a/libvirt-gconfig/libvirt-gconfig-helpers.c +++ b/libvirt-gconfig/libvirt-gconfig-helpers.c @@ -210,22 +210,43 @@ gvir_config_xml_get_child_element_content (xmlNode *node, return xmlNodeGetContent (child_node); } +static char *libxml_to_glib(xmlChar *str) +{ + char *g_str; + + if (str == NULL) + return NULL; + g_str = g_strdup((char *)str); + xmlFree(str); + + return g_str; +} + char * gvir_config_xml_get_child_element_content_glib (xmlNode *node, const char *child_name) { xmlChar *content; - char *copy; content = gvir_config_xml_get_child_element_content (node, child_name); - if (!content) - return NULL; - copy = g_strdup ((char *) content); + return libxml_to_glib(content); +} + +G_GNUC_INTERNAL xmlChar * +gvir_config_xml_get_attribute_content(xmlNodePtr node, const char *attr_name) +{ + return xmlGetProp(node, (const xmlChar*)attr_name); +} + +G_GNUC_INTERNAL char * +gvir_config_xml_get_attribute_content_glib(xmlNodePtr node, const char *attr_name) +{ + xmlChar *attr; - xmlFree (content); + attr = gvir_config_xml_get_attribute_content(node, attr_name); - return copy; + return libxml_to_glib(attr); } const char *gvir_config_genum_get_nick (GType enum_type, gint value) @@ -244,3 +265,22 @@ const char *gvir_config_genum_get_nick (GType enum_type, gint value) g_return_val_if_reached(NULL); } + +G_GNUC_INTERNAL int +gvir_config_genum_get_value (GType enum_type, const char *nick) +{ + GEnumClass *enum_class; + GEnumValue *enum_value; + + g_return_val_if_fail(G_TYPE_IS_ENUM(enum_type), -1); + g_return_val_if_fail(nick != NULL, -1); + + enum_class = g_type_class_ref(enum_type); + enum_value = g_enum_get_value_by_nick(enum_class, nick); + g_type_class_unref(enum_class); + + if (enum_value != NULL) + return enum_value->value; + + g_return_val_if_reached(-1); +} -- 1.7.7.5
This helper function will be useful to create instances of GVirConfigXXX objects when we already have a "top-level" instance (eg to create a GVirConfigDomainDisk using the tree stored in a GVirConfigDomain instance). --- libvirt-gconfig/libvirt-gconfig-object-private.h | 6 ++++++ libvirt-gconfig/libvirt-gconfig-object.c | 15 +++++++++++++++ 2 files changed, 21 insertions(+), 0 deletions(-) diff --git a/libvirt-gconfig/libvirt-gconfig-object-private.h b/libvirt-gconfig/libvirt-gconfig-object-private.h index 26da723..38c129f 100644 --- a/libvirt-gconfig/libvirt-gconfig-object-private.h +++ b/libvirt-gconfig/libvirt-gconfig-object-private.h @@ -24,6 +24,12 @@ G_BEGIN_DECLS +#include <libvirt-gconfig/libvirt-gconfig-xml-doc.h> + +GVirConfigObject *gvir_config_object_new_from_tree(GType type, + GVirConfigXmlDoc *doc, + const char *schema, + xmlNodePtr tree); xmlNodePtr gvir_config_object_get_xml_node(GVirConfigObject *config); char *gvir_config_object_get_node_content(GVirConfigObject *object, const char *node_name); diff --git a/libvirt-gconfig/libvirt-gconfig-object.c b/libvirt-gconfig/libvirt-gconfig-object.c index ce6cd21..7a6d062 100644 --- a/libvirt-gconfig/libvirt-gconfig-object.c +++ b/libvirt-gconfig/libvirt-gconfig-object.c @@ -514,6 +514,21 @@ GVirConfigObject *gvir_config_object_new_from_xml(GType type, return object; } +G_GNUC_INTERNAL GVirConfigObject * +gvir_config_object_new_from_tree(GType type, GVirConfigXmlDoc *doc, + const char *schema, xmlNodePtr tree) +{ + g_return_val_if_fail(g_type_is_a(type, GVIR_TYPE_CONFIG_OBJECT), NULL); + g_return_val_if_fail(GVIR_IS_CONFIG_XML_DOC(doc), NULL); + g_return_val_if_fail(tree != NULL, NULL); + + return GVIR_CONFIG_OBJECT(g_object_new(type, + "doc", doc, + "node", tree, + "schema", schema, + NULL)); +} + GVirConfigObject *gvir_config_object_new(GType type, const char *root_name, const char *schema) -- 1.7.7.5
There are some devices which we are not able to parse yet, in these cases a runtime warning is printed to tell what's going on. --- libvirt-gconfig/Makefile.am | 1 + .../libvirt-gconfig-domain-device-private.h | 45 ++++++++++++++ libvirt-gconfig/libvirt-gconfig-domain-device.c | 62 ++++++++++++++++++++ libvirt-gconfig/libvirt-gconfig-domain-disk.c | 29 +++++++++- libvirt-gconfig/libvirt-gconfig-domain-graphics.c | 33 ++++++++++ libvirt-gconfig/libvirt-gconfig-domain-interface.c | 38 ++++++++++++ libvirt-gconfig/libvirt-gconfig-domain.c | 41 +++++++++++++ libvirt-gconfig/libvirt-gconfig-domain.h | 1 + libvirt-gconfig/libvirt-gconfig-private.h | 1 + libvirt-gconfig/libvirt-gconfig.sym | 1 + 10 files changed, 251 insertions(+), 1 deletions(-) create mode 100644 libvirt-gconfig/libvirt-gconfig-domain-device-private.h diff --git a/libvirt-gconfig/Makefile.am b/libvirt-gconfig/Makefile.am index d9eb25a..34c5cc5 100644 --- a/libvirt-gconfig/Makefile.am +++ b/libvirt-gconfig/Makefile.am @@ -52,6 +52,7 @@ GCONFIG_HEADER_FILES = \ libvirt-gconfig-storage-vol-target.h noinst_HEADERS = \ libvirt-gconfig-private.h \ + libvirt-gconfig-domain-device-private.h \ libvirt-gconfig-helpers-private.h \ libvirt-gconfig-object-private.h \ libvirt-gconfig-xml-doc.h diff --git a/libvirt-gconfig/libvirt-gconfig-domain-device-private.h b/libvirt-gconfig/libvirt-gconfig-domain-device-private.h new file mode 100644 index 0000000..7ba7bc6 --- /dev/null +++ b/libvirt-gconfig/libvirt-gconfig-domain-device-private.h @@ -0,0 +1,45 @@ +/* + * libvirt-gconfig-domain-device-private.h: libvirt domain device configuration + * + * Copyright (C) 2011 Red Hat, Inc. + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA + * + * Author: Christophe Fergeau <cfergeau@gmail.com> + */ + +#ifndef __LIBVIRT_GCONFIG_DOMAIN_DEVICE_PRIVATE_H__ +#define __LIBVIRT_GCONFIG_DOMAIN_DEVICE_PRIVATE_H__ + +#include <libvirt-gconfig/libvirt-gconfig-xml-doc.h> + +G_BEGIN_DECLS + +GVirConfigDomainDevice * +gvir_config_domain_device_new_from_tree(GVirConfigXmlDoc *doc, + xmlNodePtr tree); +GVirConfigDomainDevice * +gvir_config_domain_disk_new_from_tree(GVirConfigXmlDoc *doc, + xmlNodePtr tree); +GVirConfigDomainDevice * +gvir_config_domain_graphics_new_from_tree(GVirConfigXmlDoc *doc, + xmlNodePtr tree); +GVirConfigDomainDevice * +gvir_config_domain_interface_new_from_tree(GVirConfigXmlDoc *doc, + xmlNodePtr tree); + +G_END_DECLS + +#endif /* __LIBVIRT_GCONFIG_DOMAIN_H__ */ diff --git a/libvirt-gconfig/libvirt-gconfig-domain-device.c b/libvirt-gconfig/libvirt-gconfig-domain-device.c index 8fcf60f..667ce8d 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-device.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-device.c @@ -48,3 +48,65 @@ static void gvir_config_domain_device_init(GVirConfigDomainDevice *device) device->priv = GVIR_CONFIG_DOMAIN_DEVICE_GET_PRIVATE(device); } + +G_GNUC_INTERNAL GVirConfigDomainDevice * +gvir_config_domain_device_new_from_tree(GVirConfigXmlDoc *doc, + xmlNodePtr tree) +{ + GType type; + + g_return_val_if_fail(GVIR_IS_CONFIG_XML_DOC(doc), NULL); + g_return_val_if_fail(tree != NULL, NULL); + g_return_val_if_fail(tree->name != NULL, NULL); + + if (xmlStrEqual(tree->name, (xmlChar*)"disk")) { + return gvir_config_domain_disk_new_from_tree(doc, tree); + } else if (xmlStrEqual(tree->name, (xmlChar*)"filesystem")) { + type = GVIR_TYPE_CONFIG_DOMAIN_FILESYS; + } else if (xmlStrEqual(tree->name, (xmlChar*)"controller")) { + goto unimplemented; + } else if (xmlStrEqual(tree->name, (xmlChar*)"lease")) { + goto unimplemented; + } else if (xmlStrEqual(tree->name, (xmlChar*)"hostdev")) { + goto unimplemented; + } else if (xmlStrEqual(tree->name, (xmlChar*)"redirdev")) { + goto unimplemented; + } else if (xmlStrEqual(tree->name, (xmlChar*)"smartcard")) { + goto unimplemented; + } else if (xmlStrEqual(tree->name, (xmlChar*)"interface")) { + return gvir_config_domain_interface_new_from_tree(doc, tree); + } else if (xmlStrEqual(tree->name, (xmlChar*)"input")) { + type = GVIR_TYPE_CONFIG_DOMAIN_INPUT; + } else if (xmlStrEqual(tree->name, (xmlChar*)"hub")) { + goto unimplemented; + } else if (xmlStrEqual(tree->name, (xmlChar*)"graphics")) { + return gvir_config_domain_graphics_new_from_tree(doc, tree); + } else if (xmlStrEqual(tree->name, (xmlChar*)"video")) { + type = GVIR_TYPE_CONFIG_DOMAIN_VIDEO; + } else if (xmlStrEqual(tree->name, (xmlChar*)"parallel")) { + goto unimplemented; + } else if (xmlStrEqual(tree->name, (xmlChar*)"serial")) { + goto unimplemented; + } else if (xmlStrEqual(tree->name, (xmlChar*)"console")) { + type = GVIR_TYPE_CONFIG_DOMAIN_CONSOLE; + } else if (xmlStrEqual(tree->name, (xmlChar*)"channel")) { + goto unimplemented; + } else if (xmlStrEqual(tree->name, (xmlChar*)"watchdog")) { + goto unimplemented; + } else if (xmlStrEqual(tree->name, (xmlChar*)"sound")) { + goto unimplemented; + } else if (xmlStrEqual(tree->name, (xmlChar*)"memballoon")) { + type = GVIR_TYPE_CONFIG_DOMAIN_MEMBALLOON; + } else { + g_warning("Unknown device node: %s", tree->name); + return NULL; + } + + g_return_val_if_fail(g_type_is_a(type, GVIR_TYPE_CONFIG_DOMAIN_DEVICE), NULL); + + return GVIR_CONFIG_DOMAIN_DEVICE(gvir_config_object_new_from_tree(type, doc, NULL, tree)); +unimplemented: + g_warning("Parsing of '%s' device nodes is unimplemented", + tree->name); + return NULL; +} diff --git a/libvirt-gconfig/libvirt-gconfig-domain-disk.c b/libvirt-gconfig/libvirt-gconfig-domain-disk.c index 1f60213..04516ce 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-disk.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-disk.c @@ -63,12 +63,39 @@ GVirConfigDomainDisk *gvir_config_domain_disk_new_from_xml(const gchar *xml, GError **error) { GVirConfigObject *object; - object = gvir_config_object_new_from_xml(GVIR_TYPE_CONFIG_DOMAIN_DISK, "disk", NULL, xml, error); + return GVIR_CONFIG_DOMAIN_DISK(object); } +GVirConfigDomainDevice * +gvir_config_domain_disk_new_from_tree(GVirConfigXmlDoc *doc, + xmlNodePtr tree) +{ + GVirConfigObject *object; + GVirConfigDomainDisk *disk; + GVirConfigDomainDiskType type; + xmlChar *type_str; + + type_str = gvir_config_xml_get_attribute_content(tree, "type"); + if (type_str == NULL) + return NULL; + + type = gvir_config_genum_get_value(GVIR_TYPE_CONFIG_DOMAIN_DISK_TYPE, + (char *)type_str); + xmlFree(type_str); + if (type == -1) + return NULL; + + object = gvir_config_object_new_from_tree(GVIR_TYPE_CONFIG_DOMAIN_DISK, + doc, NULL, tree); + disk = GVIR_CONFIG_DOMAIN_DISK(object); + disk->priv->type = type; + + return GVIR_CONFIG_DOMAIN_DEVICE(object); +} + void gvir_config_domain_disk_set_type(GVirConfigDomainDisk *disk, GVirConfigDomainDiskType type) { diff --git a/libvirt-gconfig/libvirt-gconfig-domain-graphics.c b/libvirt-gconfig/libvirt-gconfig-domain-graphics.c index 53a663b..53d5e66 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-graphics.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-graphics.c @@ -23,6 +23,7 @@ #include <config.h> #include "libvirt-gconfig/libvirt-gconfig.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h" #define GVIR_CONFIG_DOMAIN_GRAPHICS_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_GRAPHICS, GVirConfigDomainGraphicsPrivate)) @@ -47,3 +48,35 @@ static void gvir_config_domain_graphics_init(GVirConfigDomainGraphics *graphics) graphics->priv = GVIR_CONFIG_DOMAIN_GRAPHICS_GET_PRIVATE(graphics); } + +G_GNUC_INTERNAL GVirConfigDomainDevice * +gvir_config_domain_graphics_new_from_tree(GVirConfigXmlDoc *doc, + xmlNodePtr tree) +{ + xmlChar *type; + GType gtype; + + type = gvir_config_xml_get_attribute_content(tree, "type"); + if (type == NULL) + return NULL; + + if (xmlStrEqual(type, (xmlChar*)"sdl")) { + gtype = GVIR_TYPE_CONFIG_DOMAIN_GRAPHICS_SDL; + } else if (xmlStrEqual(type, (xmlChar*)"vnc")) { + gtype = GVIR_TYPE_CONFIG_DOMAIN_GRAPHICS_VNC; + } else if (xmlStrEqual(type, (xmlChar*)"spice")) { + gtype = GVIR_TYPE_CONFIG_DOMAIN_GRAPHICS_SPICE; + } else if (xmlStrEqual(type, (xmlChar*)"rdp")) { + g_warning("Parsing of '%s' graphics nodes is unimplemented", type); + return NULL; + } else if (xmlStrEqual(type, (xmlChar*)"desktop")) { + g_warning("Parsing of '%s' graphics nodes is unimplemented", type); + return NULL; + } else { + g_warning("Unknown graphics node: %s", type); + return NULL; + } + xmlFree(type); + + return GVIR_CONFIG_DOMAIN_DEVICE(gvir_config_object_new_from_tree(gtype, doc, NULL, tree)); +} diff --git a/libvirt-gconfig/libvirt-gconfig-domain-interface.c b/libvirt-gconfig/libvirt-gconfig-domain-interface.c index 2d87040..0f427ff 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-interface.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-interface.c @@ -95,3 +95,41 @@ void gvir_config_domain_interface_set_model(GVirConfigDomainInterface *interface gvir_config_object_replace_child_with_attribute(GVIR_CONFIG_OBJECT(interface), "model", "type", model); } + +G_GNUC_INTERNAL GVirConfigDomainDevice * +gvir_config_domain_interface_new_from_tree(GVirConfigXmlDoc *doc, + xmlNodePtr tree) +{ + xmlChar *type; + GType gtype; + + type = gvir_config_xml_get_attribute_content(tree, "type"); + if (type == NULL) + return NULL; + + if (xmlStrEqual(type, (xmlChar*)"network")) { + gtype = GVIR_TYPE_CONFIG_DOMAIN_INTERFACE_NETWORK; + } else if (xmlStrEqual(type, (xmlChar*)"user")) { + gtype = GVIR_TYPE_CONFIG_DOMAIN_INTERFACE_USER; + } else if (xmlStrEqual(type, (xmlChar*)"bridge")) { + goto unimplemented; + } else if (xmlStrEqual(type, (xmlChar*)"direct")) { + goto unimplemented; + } else if (xmlStrEqual(type, (xmlChar*)"server")) { + goto unimplemented; + } else if (xmlStrEqual(type, (xmlChar*)"mcast")) { + goto unimplemented; + } else if (xmlStrEqual(type, (xmlChar*)"ethernet")) { + goto unimplemented; + } else { + g_warning("Unknown domain interface node: %s", type); + return NULL; + } + xmlFree(type); + + return GVIR_CONFIG_DOMAIN_DEVICE(gvir_config_object_new_from_tree(gtype, doc, NULL, tree)); + +unimplemented: + g_warning("Parsing of '%s' domain interface nodes is unimplemented", type); + return NULL; +} diff --git a/libvirt-gconfig/libvirt-gconfig-domain.c b/libvirt-gconfig/libvirt-gconfig-domain.c index ed48d57..e8a7c04 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain.c +++ b/libvirt-gconfig/libvirt-gconfig-domain.c @@ -360,3 +360,44 @@ void gvir_config_domain_add_device(GVirConfigDomain *domain, gvir_config_object_attach(devices_node, GVIR_CONFIG_OBJECT(device)); g_object_unref(G_OBJECT(devices_node)); } + +/** + * gvir_config_domain_get_devices: + * + * Gets the list of devices attached to @domain + * + * Returns: (element-type LibvirtGConfig.DomainDevice) (transfer full): + * a newly allocated #GList of #GVirConfigDomainDevice. + */ +GList *gvir_config_domain_get_devices(GVirConfigDomain *domain) +{ + xmlNodePtr it; + xmlNodePtr root; + GList *devices; + GVirConfigXmlDoc *doc; + + g_return_val_if_fail(GVIR_IS_CONFIG_DOMAIN(domain), NULL); + root = gvir_config_object_get_xml_node(GVIR_CONFIG_OBJECT(domain)); + g_return_val_if_fail(root != NULL, NULL); + + it = gvir_config_xml_get_element(root, "devices", NULL); + if (it == NULL) { + return NULL; + } + + g_object_get(G_OBJECT(domain), "doc", &doc, NULL); + devices = NULL; + for (it = it->children; it != NULL; it = it->next) { + GVirConfigDomainDevice *device; + + device = gvir_config_domain_device_new_from_tree(doc, it); + g_warn_if_fail(device != NULL); + if (device != NULL) + devices = g_list_append(devices, device); + } + if (doc != NULL) { + g_object_unref(G_OBJECT(doc)); + } + + return devices; +} diff --git a/libvirt-gconfig/libvirt-gconfig-domain.h b/libvirt-gconfig/libvirt-gconfig-domain.h index 2b631ea..f46cc35 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain.h +++ b/libvirt-gconfig/libvirt-gconfig-domain.h @@ -120,6 +120,7 @@ void gvir_config_domain_set_devices(GVirConfigDomain *domain, GList *devices); void gvir_config_domain_add_device(GVirConfigDomain *domain, GVirConfigDomainDevice *device); +GList *gvir_config_domain_get_devices(GVirConfigDomain *domain); void gvir_config_domain_set_lifecycle(GVirConfigDomain *domain, GVirConfigDomainLifecycleEvent event, GVirConfigDomainLifecycleAction action); diff --git a/libvirt-gconfig/libvirt-gconfig-private.h b/libvirt-gconfig/libvirt-gconfig-private.h index e985ebd..d4df030 100644 --- a/libvirt-gconfig/libvirt-gconfig-private.h +++ b/libvirt-gconfig/libvirt-gconfig-private.h @@ -23,6 +23,7 @@ #ifndef __LIBVIRT_GCONFIG_PRIVATE_H__ #define __LIBVIRT_GCONFIG_PRIVATE_H__ +#include <libvirt-gconfig/libvirt-gconfig-domain-device-private.h> #include <libvirt-gconfig/libvirt-gconfig-helpers-private.h> #include <libvirt-gconfig/libvirt-gconfig-object-private.h> #include <libvirt-gconfig/libvirt-gconfig-xml-doc.h> diff --git a/libvirt-gconfig/libvirt-gconfig.sym b/libvirt-gconfig/libvirt-gconfig.sym index 87afa9d..7200da5 100644 --- a/libvirt-gconfig/libvirt-gconfig.sym +++ b/libvirt-gconfig/libvirt-gconfig.sym @@ -14,6 +14,7 @@ LIBVIRT_GCONFIG_0.0.3 { gvir_config_domain_new; gvir_config_domain_new_from_xml; gvir_config_domain_set_clock; + gvir_config_domain_get_devices; gvir_config_domain_set_devices; gvir_config_domain_get_features; gvir_config_domain_set_features; -- 1.7.7.5
On Tue, Jan 3, 2012 at 12:11 PM, Christophe Fergeau <cfergeau@redhat.com> wrote:
There are some devices which we are not able to parse yet, in these cases a runtime warning is printed to tell what's going on.
Actually I see that most of the device types are not being handled. I would suggest we implement all the cases before merging this or otherwise we'll easily forget about this. Yes the warning won't let us forget but I don't think emitting a warning for each unimplemented type (by the library itself) is a good idea since these warnings are visible to users/app developers and they are useless for them. -- Regards, Zeeshan Ali (Khattak) FSF member#5124
On Tue, Jan 03, 2012 at 11:38:15PM +0200, Zeeshan Ali (Khattak) wrote:
On Tue, Jan 3, 2012 at 12:11 PM, Christophe Fergeau <cfergeau@redhat.com> wrote:
There are some devices which we are not able to parse yet, in these cases a runtime warning is printed to tell what's going on.
Actually I see that most of the device types are not being handled. I would suggest we implement all the cases before merging this or otherwise we'll easily forget about this. Yes the warning won't let us forget but I don't think emitting a warning for each unimplemented type (by the library itself) is a good idea since these warnings are visible to users/app developers and they are useless for them.
Agreed, the way we deal with the XML docs ensures we preserve any parts of the DOM we don't know about. IMHO we should only emit a warning if there is a bit of the DOM we don't expect and we are loosing that data during the requested manipulation. Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
On Tue, Jan 03, 2012 at 10:08:10PM +0000, Daniel P. Berrange wrote:
On Tue, Jan 03, 2012 at 11:38:15PM +0200, Zeeshan Ali (Khattak) wrote:
On Tue, Jan 3, 2012 at 12:11 PM, Christophe Fergeau <cfergeau@redhat.com> wrote:
There are some devices which we are not able to parse yet, in these cases a runtime warning is printed to tell what's going on.
Actually I see that most of the device types are not being handled. I would suggest we implement all the cases before merging this or otherwise we'll easily forget about this. Yes the warning won't let us forget but I don't think emitting a warning for each unimplemented type (by the library itself) is a good idea since these warnings are visible to users/app developers and they are useless for them.
Agreed, the way we deal with the XML docs ensures we preserve any parts of the DOM we don't know about. IMHO we should only emit a warning if there is a bit of the DOM we don't expect and we are loosing that data during the requested manipulation.
I'll change these warnings to g_debug then Christophe
On Wed, Jan 04, 2012 at 10:50:18AM +0400, Christophe Fergeau wrote:
On Tue, Jan 03, 2012 at 10:08:10PM +0000, Daniel P. Berrange wrote:
Agreed, the way we deal with the XML docs ensures we preserve any parts of the DOM we don't know about. IMHO we should only emit a warning if there is a bit of the DOM we don't expect and we are loosing that data during the requested manipulation.
I'll change these warnings to g_debug then
I just squashed the diff below in this patch: diff --git a/libvirt-gconfig/libvirt-gconfig-domain-device.c b/libvirt-gconfig/libvirt-gconfig-domain-device.c index a9ae8e8..8399da2 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-device.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-device.c @@ -98,7 +98,7 @@ gvir_config_domain_device_new_from_tree(GVirConfigXmlDoc *doc, } else if (xmlStrEqual(tree->name, (xmlChar*)"memballoon")) { type = GVIR_TYPE_CONFIG_DOMAIN_MEMBALLOON; } else { - g_warning("Unknown device node: %s", tree->name); + g_debug("Unknown device node: %s", tree->name); return NULL; } @@ -106,7 +106,6 @@ gvir_config_domain_device_new_from_tree(GVirConfigXmlDoc *doc, return GVIR_CONFIG_DOMAIN_DEVICE(gvir_config_object_new_from_tree(type, doc, NULL, tree)); unimplemented: - g_warning("Parsing of '%s' device nodes is unimplemented", - tree->name); + g_debug("Parsing of '%s' device nodes is unimplemented", tree->name); return NULL; } diff --git a/libvirt-gconfig/libvirt-gconfig-domain-graphics.c b/libvirt-gconfig/libvirt-gconfig-domain-graphics.c index 53d5e66..6caf7cf 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-graphics.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-graphics.c @@ -67,16 +67,18 @@ gvir_config_domain_graphics_new_from_tree(GVirConfigXmlDoc *doc, } else if (xmlStrEqual(type, (xmlChar*)"spice")) { gtype = GVIR_TYPE_CONFIG_DOMAIN_GRAPHICS_SPICE; } else if (xmlStrEqual(type, (xmlChar*)"rdp")) { - g_warning("Parsing of '%s' graphics nodes is unimplemented", type); - return NULL; + goto unimplemented; } else if (xmlStrEqual(type, (xmlChar*)"desktop")) { - g_warning("Parsing of '%s' graphics nodes is unimplemented", type); - return NULL; + goto unimplemented; } else { - g_warning("Unknown graphics node: %s", type); + g_debug("Unknown graphics node: %s", type); return NULL; } xmlFree(type); return GVIR_CONFIG_DOMAIN_DEVICE(gvir_config_object_new_from_tree(gtype, doc, NULL, tree)); + +unimplemented: + g_debug("Parsing of '%s' graphics nodes is unimplemented", type); + return NULL; } diff --git a/libvirt-gconfig/libvirt-gconfig-domain-interface.c b/libvirt-gconfig/libvirt-gconfig-domain-interface.c index 0f427ff..58e2dd9 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-interface.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-interface.c @@ -122,7 +122,7 @@ gvir_config_domain_interface_new_from_tree(GVirConfigXmlDoc *doc, } else if (xmlStrEqual(type, (xmlChar*)"ethernet")) { goto unimplemented; } else { - g_warning("Unknown domain interface node: %s", type); + g_debug("Unknown domain interface node: %s", type); return NULL; } xmlFree(type); @@ -130,6 +130,6 @@ gvir_config_domain_interface_new_from_tree(GVirConfigXmlDoc *doc, return GVIR_CONFIG_DOMAIN_DEVICE(gvir_config_object_new_from_tree(gtype, doc, NULL, tree)); unimplemented: - g_warning("Parsing of '%s' domain interface nodes is unimplemented", type); + g_debug("Parsing of '%s' domain interface nodes is unimplemented", type); return NULL; } diff --git a/libvirt-gconfig/libvirt-gconfig-domain.c b/libvirt-gconfig/libvirt-gconfig-domain.c index 26ffc87..d953826 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain.c +++ b/libvirt-gconfig/libvirt-gconfig-domain.c @@ -374,9 +374,10 @@ static gboolean add_one_device(xmlNodePtr node, gpointer opaque) GVirConfigDomainDevice *device; device = gvir_config_domain_device_new_from_tree(data->doc, node); - g_warn_if_fail(device != NULL); if (device != NULL) data->devices = g_list_append(data->devices, device); + else + g_debug("Failed to parse %s node", node->name); return TRUE; }
On Wed, Jan 4, 2012 at 8:50 AM, Christophe Fergeau <cfergeau@redhat.com> wrote:
On Tue, Jan 03, 2012 at 10:08:10PM +0000, Daniel P. Berrange wrote:
On Tue, Jan 03, 2012 at 11:38:15PM +0200, Zeeshan Ali (Khattak) wrote:
On Tue, Jan 3, 2012 at 12:11 PM, Christophe Fergeau <cfergeau@redhat.com> wrote:
There are some devices which we are not able to parse yet, in these cases a runtime warning is printed to tell what's going on.
Actually I see that most of the device types are not being handled. I would suggest we implement all the cases before merging this or otherwise we'll easily forget about this. Yes the warning won't let us forget but I don't think emitting a warning for each unimplemented type (by the library itself) is a good idea since these warnings are visible to users/app developers and they are useless for them.
Agreed, the way we deal with the XML docs ensures we preserve any parts of the DOM we don't know about. IMHO we should only emit a warning if there is a bit of the DOM we don't expect and we are loosing that data during the requested manipulation.
I'll change these warnings to g_debug then
OK, I'll put this on my TODO list then so we don't forget. :) -- Regards, Zeeshan Ali (Khattak) FSF member#5124
Hi, I was a bit too hasty in sending this, while the series is fine as is, I have made a few improvements to it, so I'll send a v2. Changes that will be in it: - handle 'sound' nodes - ignore blank xml nodes in get_devices to avoid not so useful warnings Christophe On Tue, Jan 03, 2012 at 02:11:20PM +0400, Christophe Fergeau wrote:
This header groups all libvirt-gconfig private headers. --- libvirt-gconfig/Makefile.am | 1 + .../libvirt-gconfig-domain-chardev-source-pty.c | 2 +- libvirt-gconfig/libvirt-gconfig-domain-chardev.c | 2 +- libvirt-gconfig/libvirt-gconfig-domain-clock.c | 3 +- libvirt-gconfig/libvirt-gconfig-domain-device.c | 1 + libvirt-gconfig/libvirt-gconfig-domain-disk.c | 3 +- libvirt-gconfig/libvirt-gconfig-domain-filesys.c | 3 +- .../libvirt-gconfig-domain-graphics-sdl.c | 3 +- .../libvirt-gconfig-domain-graphics-spice.c | 3 +- .../libvirt-gconfig-domain-graphics-vnc.c | 3 +- libvirt-gconfig/libvirt-gconfig-domain-input.c | 3 +- .../libvirt-gconfig-domain-interface-network.c | 3 +- .../libvirt-gconfig-domain-interface-user.c | 3 +- libvirt-gconfig/libvirt-gconfig-domain-interface.c | 2 +- .../libvirt-gconfig-domain-memballoon.c | 3 +- libvirt-gconfig/libvirt-gconfig-domain-os.c | 3 +- libvirt-gconfig/libvirt-gconfig-domain-seclabel.c | 3 +- libvirt-gconfig/libvirt-gconfig-domain-sound.c | 2 +- libvirt-gconfig/libvirt-gconfig-domain-video.c | 3 +- libvirt-gconfig/libvirt-gconfig-domain.c | 3 +- libvirt-gconfig/libvirt-gconfig-object.c | 4 +-- libvirt-gconfig/libvirt-gconfig-private.h | 30 ++++++++++++++++++++ .../libvirt-gconfig-storage-permissions.c | 2 +- .../libvirt-gconfig-storage-pool-source.c | 2 +- .../libvirt-gconfig-storage-pool-target.c | 2 +- libvirt-gconfig/libvirt-gconfig-storage-pool.c | 2 +- .../libvirt-gconfig-storage-vol-backing-store.c | 2 +- .../libvirt-gconfig-storage-vol-target.c | 2 +- libvirt-gconfig/libvirt-gconfig-storage-vol.c | 2 +- 29 files changed, 58 insertions(+), 42 deletions(-) create mode 100644 libvirt-gconfig/libvirt-gconfig-private.h
diff --git a/libvirt-gconfig/Makefile.am b/libvirt-gconfig/Makefile.am index 6342431..d9eb25a 100644 --- a/libvirt-gconfig/Makefile.am +++ b/libvirt-gconfig/Makefile.am @@ -51,6 +51,7 @@ GCONFIG_HEADER_FILES = \ libvirt-gconfig-storage-vol-backing-store.h \ libvirt-gconfig-storage-vol-target.h noinst_HEADERS = \ + libvirt-gconfig-private.h \ libvirt-gconfig-helpers-private.h \ libvirt-gconfig-object-private.h \ libvirt-gconfig-xml-doc.h diff --git a/libvirt-gconfig/libvirt-gconfig-domain-chardev-source-pty.c b/libvirt-gconfig/libvirt-gconfig-domain-chardev-source-pty.c index 1965444..e3c4c25 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-chardev-source-pty.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-chardev-source-pty.c @@ -23,7 +23,7 @@ #include <config.h>
#include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_DOMAIN_CHARDEV_SOURCE_PTY_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_CHARDEV_SOURCE_PTY, GVirConfigDomainChardevSourcePtyPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-chardev.c b/libvirt-gconfig/libvirt-gconfig-domain-chardev.c index 1359f8f..620773c 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-chardev.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-chardev.c @@ -23,7 +23,7 @@ #include <config.h>
#include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_DOMAIN_CHARDEV_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_CHARDEV, GVirConfigDomainChardevPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-clock.c b/libvirt-gconfig/libvirt-gconfig-domain-clock.c index 16d75d5..3cb8ffb 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-clock.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-clock.c @@ -23,8 +23,7 @@ #include <config.h>
#include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_DOMAIN_CLOCK_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_CLOCK, GVirConfigDomainClockPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-device.c b/libvirt-gconfig/libvirt-gconfig-domain-device.c index 980ae70..8fcf60f 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-device.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-device.c @@ -23,6 +23,7 @@ #include <config.h>
#include "libvirt-gconfig/libvirt-gconfig.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_DOMAIN_DEVICE_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_DEVICE, GVirConfigDomainDevicePrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-disk.c b/libvirt-gconfig/libvirt-gconfig-domain-disk.c index 18cdb1c..1f60213 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-disk.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-disk.c @@ -23,8 +23,7 @@ #include <config.h>
#include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_DOMAIN_DISK_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_DISK, GVirConfigDomainDiskPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-filesys.c b/libvirt-gconfig/libvirt-gconfig-domain-filesys.c index 6c3aa9d..ef942f8 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-filesys.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-filesys.c @@ -23,8 +23,7 @@ #include <config.h>
#include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_DOMAIN_FILESYS_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_FILESYS, GVirConfigDomainFilesysPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-graphics-sdl.c b/libvirt-gconfig/libvirt-gconfig-domain-graphics-sdl.c index c60e72b..1e33a1b 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-graphics-sdl.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-graphics-sdl.c @@ -23,8 +23,7 @@ #include <config.h>
#include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_DOMAIN_GRAPHICS_SDL_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_GRAPHICS_SDL, GVirConfigDomainGraphicsSdlPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-graphics-spice.c b/libvirt-gconfig/libvirt-gconfig-domain-graphics-spice.c index 321b914..7aa3e2c 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-graphics-spice.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-graphics-spice.c @@ -23,8 +23,7 @@ #include <config.h>
#include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_DOMAIN_GRAPHICS_SPICE_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_GRAPHICS_SPICE, GVirConfigDomainGraphicsSpicePrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-graphics-vnc.c b/libvirt-gconfig/libvirt-gconfig-domain-graphics-vnc.c index f7fd6db..b00a155 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-graphics-vnc.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-graphics-vnc.c @@ -23,8 +23,7 @@ #include <config.h>
#include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_DOMAIN_GRAPHICS_VNC_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_GRAPHICS_VNC, GVirConfigDomainGraphicsVncPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-input.c b/libvirt-gconfig/libvirt-gconfig-domain-input.c index 5bc70c8..bacc693 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-input.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-input.c @@ -23,8 +23,7 @@ #include <config.h>
#include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_DOMAIN_INPUT_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_INPUT, GVirConfigDomainInputPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-interface-network.c b/libvirt-gconfig/libvirt-gconfig-domain-interface-network.c index becddc8..6f24708 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-interface-network.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-interface-network.c @@ -25,8 +25,7 @@ #include <config.h>
#include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_DOMAIN_INTERFACE_NETWORK_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_INTERFACE_NETWORK, GVirConfigDomainInterfaceNetworkPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-interface-user.c b/libvirt-gconfig/libvirt-gconfig-domain-interface-user.c index 2c03af2..25770e0 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-interface-user.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-interface-user.c @@ -25,8 +25,7 @@ #include <config.h>
#include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_DOMAIN_INTERFACE_USER_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_INTERFACE_USER, GVirConfigDomainInterfaceUserPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-interface.c b/libvirt-gconfig/libvirt-gconfig-domain-interface.c index 834ce5d..2d87040 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-interface.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-interface.c @@ -25,7 +25,7 @@ #include <config.h>
#include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_DOMAIN_INTERFACE_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_INTERFACE, GVirConfigDomainInterfacePrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-memballoon.c b/libvirt-gconfig/libvirt-gconfig-domain-memballoon.c index ca74a72..9df7e52 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-memballoon.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-memballoon.c @@ -23,8 +23,7 @@ #include <config.h>
#include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_DOMAIN_MEMBALLOON_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_MEMBALLOON, GVirConfigDomainMemballoonPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-os.c b/libvirt-gconfig/libvirt-gconfig-domain-os.c index 2b0cdff..6818abf 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-os.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-os.c @@ -23,8 +23,7 @@ #include <config.h>
#include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_DOMAIN_OS_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_OS, GVirConfigDomainOsPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-seclabel.c b/libvirt-gconfig/libvirt-gconfig-domain-seclabel.c index 7537a78..2d69c6f 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-seclabel.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-seclabel.c @@ -23,8 +23,7 @@ #include <config.h>
#include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_DOMAIN_SECLABEL_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_SECLABEL, GVirConfigDomainSeclabelPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-sound.c b/libvirt-gconfig/libvirt-gconfig-domain-sound.c index 56149ca..abf9a14 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-sound.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-sound.c @@ -23,7 +23,7 @@ #include <config.h>
#include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_DOMAIN_SOUND_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_SOUND, GVirConfigDomainSoundPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain-video.c b/libvirt-gconfig/libvirt-gconfig-domain-video.c index bba8073..90c71da 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-video.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-video.c @@ -23,8 +23,7 @@ #include <config.h>
#include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_DOMAIN_VIDEO_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN_VIDEO, GVirConfigDomainVideoPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-domain.c b/libvirt-gconfig/libvirt-gconfig-domain.c index c5df963..ed48d57 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain.c +++ b/libvirt-gconfig/libvirt-gconfig-domain.c @@ -24,8 +24,7 @@ #include <config.h>
#include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_DOMAIN_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_DOMAIN, GVirConfigDomainPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-object.c b/libvirt-gconfig/libvirt-gconfig-object.c index cf71264..ce6cd21 100644 --- a/libvirt-gconfig/libvirt-gconfig-object.c +++ b/libvirt-gconfig/libvirt-gconfig-object.c @@ -28,9 +28,7 @@ #include <libxml/relaxng.h>
#include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-helpers-private.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" -#include "libvirt-gconfig/libvirt-gconfig-xml-doc.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_OBJECT_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_OBJECT, GVirConfigObjectPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-private.h b/libvirt-gconfig/libvirt-gconfig-private.h new file mode 100644 index 0000000..e985ebd --- /dev/null +++ b/libvirt-gconfig/libvirt-gconfig-private.h @@ -0,0 +1,30 @@ +/* + * libvirt-gconfig-private.h: libvirt domain device configuration + * + * Copyright (C) 2011 Red Hat, Inc. + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA + * + * Author: Christophe Fergeau <cfergeau@gmail.com> + */ + +#ifndef __LIBVIRT_GCONFIG_PRIVATE_H__ +#define __LIBVIRT_GCONFIG_PRIVATE_H__ + +#include <libvirt-gconfig/libvirt-gconfig-helpers-private.h> +#include <libvirt-gconfig/libvirt-gconfig-object-private.h> +#include <libvirt-gconfig/libvirt-gconfig-xml-doc.h> + +#endif /* __LIBVIRT_GCONFIG_PRIVATE_H__ */ diff --git a/libvirt-gconfig/libvirt-gconfig-storage-permissions.c b/libvirt-gconfig/libvirt-gconfig-storage-permissions.c index c5f9f49..49e28ac 100644 --- a/libvirt-gconfig/libvirt-gconfig-storage-permissions.c +++ b/libvirt-gconfig/libvirt-gconfig-storage-permissions.c @@ -23,7 +23,7 @@ #include <config.h>
#include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_STORAGE_PERMISSIONS_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_STORAGE_PERMISSIONS, GVirConfigStoragePermissionsPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-storage-pool-source.c b/libvirt-gconfig/libvirt-gconfig-storage-pool-source.c index b2223a2..fafb755 100644 --- a/libvirt-gconfig/libvirt-gconfig-storage-pool-source.c +++ b/libvirt-gconfig/libvirt-gconfig-storage-pool-source.c @@ -23,7 +23,7 @@ #include <config.h>
#include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_STORAGE_POOL_SOURCE_GET_PRIVATE(obj) \ diff --git a/libvirt-gconfig/libvirt-gconfig-storage-pool-target.c b/libvirt-gconfig/libvirt-gconfig-storage-pool-target.c index a5af381..4ce5a12 100644 --- a/libvirt-gconfig/libvirt-gconfig-storage-pool-target.c +++ b/libvirt-gconfig/libvirt-gconfig-storage-pool-target.c @@ -23,7 +23,7 @@ #include <config.h>
#include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_STORAGE_POOL_TARGET_GET_PRIVATE(obj) \ diff --git a/libvirt-gconfig/libvirt-gconfig-storage-pool.c b/libvirt-gconfig/libvirt-gconfig-storage-pool.c index 95609bb..c22e160 100644 --- a/libvirt-gconfig/libvirt-gconfig-storage-pool.c +++ b/libvirt-gconfig/libvirt-gconfig-storage-pool.c @@ -25,7 +25,7 @@ #include <config.h>
#include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_STORAGE_POOL_GET_PRIVATE(obj) \ (G_TYPE_INSTANCE_GET_PRIVATE((obj), GVIR_TYPE_CONFIG_STORAGE_POOL, GVirConfigStoragePoolPrivate)) diff --git a/libvirt-gconfig/libvirt-gconfig-storage-vol-backing-store.c b/libvirt-gconfig/libvirt-gconfig-storage-vol-backing-store.c index 2530ed3..1791ba0 100644 --- a/libvirt-gconfig/libvirt-gconfig-storage-vol-backing-store.c +++ b/libvirt-gconfig/libvirt-gconfig-storage-vol-backing-store.c @@ -23,7 +23,7 @@ #include <config.h>
#include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_STORAGE_VOL_BACKING_STORE_GET_PRIVATE(obj) \ diff --git a/libvirt-gconfig/libvirt-gconfig-storage-vol-target.c b/libvirt-gconfig/libvirt-gconfig-storage-vol-target.c index 6a3bb0c..373f684 100644 --- a/libvirt-gconfig/libvirt-gconfig-storage-vol-target.c +++ b/libvirt-gconfig/libvirt-gconfig-storage-vol-target.c @@ -23,7 +23,7 @@ #include <config.h>
#include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_STORAGE_VOL_TARGET_GET_PRIVATE(obj) \ diff --git a/libvirt-gconfig/libvirt-gconfig-storage-vol.c b/libvirt-gconfig/libvirt-gconfig-storage-vol.c index 079a5fd..1864447 100644 --- a/libvirt-gconfig/libvirt-gconfig-storage-vol.c +++ b/libvirt-gconfig/libvirt-gconfig-storage-vol.c @@ -24,7 +24,7 @@ #include <config.h>
#include "libvirt-gconfig/libvirt-gconfig.h" -#include "libvirt-gconfig/libvirt-gconfig-object-private.h" +#include "libvirt-gconfig/libvirt-gconfig-private.h"
#define GVIR_CONFIG_STORAGE_VOL_GET_PRIVATE(obj) \ -- 1.7.7.5
-- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
participants (3)
-
Christophe Fergeau -
Daniel P. Berrange -
Zeeshan Ali (Khattak)