Re: [libvirt-glib] gobject: Fix GEnum generation through glib-mkenums

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jun 3, 2014 at 10:16 AM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote:
> We were only passing libvirt-gobject-domain.h and
> libvirt-gobject-connection.h through glib-mkenums, which causes it to
> only generate GEnum information for enums found in these headers.
> We want to do that for all enums defined in installed headers, so
> passing all headers listed in libvirt_gobject_1_0_la_HEADERS is more
> appropriate.

ACK if your patch doesn't introduce any warnings/errors.

> ---
>  libvirt-gobject/Makefile.am         | 4 ++--
>  libvirt-gobject/libvirt-gobject.sym | 5 +++++
>  2 files changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/libvirt-gobject/Makefile.am b/libvirt-gobject/Makefile.am
> index 516c10f..7163c7d 100644
> --- a/libvirt-gobject/Makefile.am
> +++ b/libvirt-gobject/Makefile.am
> @@ -93,7 +93,7 @@ libvirt_gobject_1_0_la_LDFLAGS = \
>                         -Wl,--version-script=$(srcdir)/libvirt-gobject.sym \
>                         -version-info $(LIBVIRT_GLIB_VERSION_INFO)
>
> -libvirt-gobject-enums.c: libvirt-gobject-domain.h libvirt-gobject-connection.h
> +libvirt-gobject-enums.c: $(libvirt_gobject_1_0_la_HEADERS)
>         $(AM_V_GEN)glib-mkenums \
>                          --fhead "#include \"libvirt-gobject/libvirt-gobject.h\"\n\n" \
>                          --vhead "static const G@Type@Value _@enum_name@_values[] = {" \
> @@ -106,7 +106,7 @@ libvirt-gobject-enums.c: libvirt-gobject-domain.h libvirt-gobject-connection.h
>                          --vtail "  return type;\n}\n\n" \
>                  $^ | sed -e 's/g_vir/gvir/g' > $@
>
> -libvirt-gobject-enums.h: libvirt-gobject-domain.h libvirt-gobject-connection.h
> +libvirt-gobject-enums.h: $(libvirt_gobject_1_0_la_HEADERS)
>         $(AM_V_GEN)glib-mkenums    --fhead "#ifndef __LIBVIRT_GOBJECT_ENUMS_H__\n" \
>                          --fhead "#define __LIBVIRT_GOBJECT_ENUMS_H__\n\n" \
>                          --fhead "G_BEGIN_DECLS\n\n" \
> diff --git a/libvirt-gobject/libvirt-gobject.sym b/libvirt-gobject/libvirt-gobject.sym
> index b781cc6..4e856e7 100644
> --- a/libvirt-gobject/libvirt-gobject.sym
> +++ b/libvirt-gobject/libvirt-gobject.sym
> @@ -237,6 +237,11 @@ LIBVIRT_GOBJECT_0.1.5 {
>  LIBVIRT_GOBJECT_0.1.9 {
>    global:
>         gvir_domain_snapshot_delete;
> +       gvir_domain_snapshot_delete_flags_get_type;
> +       gvir_storage_pool_state_get_type;
> +       gvir_storage_vol_resize_flags_get_type;
> +       gvir_storage_vol_type_get_type;
> +       gvir_stream_io_condition_get_type;

Are these related? Shouldn't this change be split?

Regards,

Zeeshan Ali (Khattak)
________________________________________
Befriend GNOME: http://www.gnome.org/friends/

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]