ruby-libvirt 0.4.0 → 0.5.0

data/ext/libvirt/interface.h

@@ -1,6 +1,8 @@
 #ifndef INTERFACE_H
 #define INTERFACE_H
 
-void init_interface();
+void ruby_libvirt_interface_init(void);
+
+VALUE ruby_libvirt_interface_new(virInterfacePtr i, VALUE conn);
 
 #endif

data/ext/libvirt/network.c

@@ -2,6 +2,7 @@
  * network.c: virNetwork methods
  *
  * Copyright (C) 2007,2010 Red Hat Inc.
+ * Copyright (C) 2013 Chris Lalancette <clalancette@gmail.com>
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Lesser General Public
@@ -28,124 +29,156 @@
 #if HAVE_TYPE_VIRNETWORKPTR
 static VALUE c_network;
 
-static void network_free(void *d) {
-    generic_free(Network, d);
+static void network_free(void *d)
+{
+    ruby_libvirt_free_struct(Network, d);
 }
 
-static virNetworkPtr network_get(VALUE s) {
-    generic_get(Network, s);
+static virNetworkPtr network_get(VALUE n)
+{
+    ruby_libvirt_get_struct(Network, n);
 }
 
-VALUE network_new(virNetworkPtr n, VALUE conn) {
-    return generic_new(c_network, n, conn, network_free);
+VALUE ruby_libvirt_network_new(virNetworkPtr n, VALUE conn)
+{
+    return ruby_libvirt_new_class(c_network, n, conn, network_free);
 }
 
 /*
  * call-seq:
  *   net.undefine -> nil
  *
- * Call +virNetworkUndefine+[http://www.libvirt.org/html/libvirt-libvirt.html#virNetworkUndefine]
+ * Call virNetworkUndefine[http://www.libvirt.org/html/libvirt-libvirt.html#virNetworkUndefine]
  * to undefine this network.
  */
-static VALUE libvirt_netw_undefine(VALUE s) {
-    gen_call_void(virNetworkUndefine, conn(s), network_get(s));
+static VALUE libvirt_network_undefine(VALUE n)
+{
+    ruby_libvirt_generate_call_nil(virNetworkUndefine,
+                                   ruby_libvirt_connect_get(n),
+                                   network_get(n));
 }
 
 /*
  * call-seq:
  *   net.create -> nil
  *
- * Call +virNetworkCreate+[http://www.libvirt.org/html/libvirt-libvirt.html#virNetworkCreate]
+ * Call virNetworkCreate[http://www.libvirt.org/html/libvirt-libvirt.html#virNetworkCreate]
  * to start this network.
  */
-static VALUE libvirt_netw_create(VALUE s) {
-    gen_call_void(virNetworkCreate, conn(s), network_get(s));
+static VALUE libvirt_network_create(VALUE n)
+{
+    ruby_libvirt_generate_call_nil(virNetworkCreate,
+                                   ruby_libvirt_connect_get(n),
+                                   network_get(n));
+}
+
+#if HAVE_VIRNETWORKUPDATE
+/*
+ * call-seq:
+ *   net.update -> nil
+ *
+ * Call virNetworkUpdate[http://www.libvirt.org/html/libvirt-libvirt.html#virNetworkUpdate]
+ * to update this network.
+ */
+static VALUE libvirt_network_update(VALUE n, VALUE command, VALUE section,
+                                    VALUE index, VALUE xml, VALUE flags)
+{
+    ruby_libvirt_generate_call_nil(virNetworkUpdate,
+                                   ruby_libvirt_connect_get(n),
+                                   network_get(n), NUM2UINT(command),
+                                   NUM2UINT(section), NUM2INT(index),
+                                   StringValuePtr(xml), NUM2UINT(flags));
 }
+#endif
 
 /*
  * call-seq:
  *   net.destroy -> nil
  *
- * Call +virNetworkDestroy+[http://www.libvirt.org/html/libvirt-libvirt.html#virNetworkDestroy]
+ * Call virNetworkDestroy[http://www.libvirt.org/html/libvirt-libvirt.html#virNetworkDestroy]
  * to shutdown this network.
  */
-static VALUE libvirt_netw_destroy(VALUE s) {
-    gen_call_void(virNetworkDestroy, conn(s), network_get(s));
+static VALUE libvirt_network_destroy(VALUE n)
+{
+    ruby_libvirt_generate_call_nil(virNetworkDestroy,
+                                   ruby_libvirt_connect_get(n),
+                                   network_get(n));
 }
 
 /*
  * call-seq:
  *   net.name -> string
  *
- * Call +virNetworkGetName+[http://www.libvirt.org/html/libvirt-libvirt.html#virNetworkGetName]
+ * Call virNetworkGetName[http://www.libvirt.org/html/libvirt-libvirt.html#virNetworkGetName]
  * to retrieve the name of this network.
  */
-static VALUE libvirt_netw_name(VALUE s) {
-    gen_call_string(virNetworkGetName, conn(s), 0, network_get(s));
+static VALUE libvirt_network_name(VALUE n)
+{
+    ruby_libvirt_generate_call_string(virNetworkGetName,
+                                      ruby_libvirt_connect_get(n), 0,
+                                      network_get(n));
 }
 
 /*
  * call-seq:
  *   net.uuid -> string
  *
- * Call +virNetworkGetUUIDString+[http://www.libvirt.org/html/libvirt-libvirt.html#virNetworkGetUUIDString]
+ * Call virNetworkGetUUIDString[http://www.libvirt.org/html/libvirt-libvirt.html#virNetworkGetUUIDString]
  * to retrieve the UUID of this network.
  */
-static VALUE libvirt_netw_uuid(VALUE s) {
-    virNetworkPtr netw = network_get(s);
-    char uuid[VIR_UUID_STRING_BUFLEN];
-    int r;
-
-    r = virNetworkGetUUIDString(netw, uuid);
-    _E(r < 0, create_error(e_RetrieveError, "virNetworkGetUUIDString",
-                           conn(s)));
-
-    return rb_str_new2((char *) uuid);
+static VALUE libvirt_network_uuid(VALUE n)
+{
+    ruby_libvirt_generate_uuid(virNetworkGetUUIDString,
+                               ruby_libvirt_connect_get(n), network_get(n));
 }
 
 /*
  * call-seq:
  *   net.xml_desc(flags=0) -> string
  *
- * Call +virNetworkGetXMLDesc+[http://www.libvirt.org/html/libvirt-libvirt.html#virNetworkGetXMLDesc]
+ * Call virNetworkGetXMLDesc[http://www.libvirt.org/html/libvirt-libvirt.html#virNetworkGetXMLDesc]
  * to retrieve the XML for this network.
  */
-static VALUE libvirt_netw_xml_desc(int argc, VALUE *argv, VALUE s) {
+static VALUE libvirt_network_xml_desc(int argc, VALUE *argv, VALUE n)
+{
     VALUE flags;
 
     rb_scan_args(argc, argv, "01", &flags);
 
-    if (NIL_P(flags))
-        flags = INT2NUM(0);
-
-    gen_call_string(virNetworkGetXMLDesc, conn(s), 1, network_get(s),
-                    NUM2UINT(flags));
+    ruby_libvirt_generate_call_string(virNetworkGetXMLDesc,
+                                      ruby_libvirt_connect_get(n), 1,
+                                      network_get(n),
+                                      ruby_libvirt_value_to_uint(flags));
 }
 
 /*
  * call-seq:
  *   net.bridge_name -> string
  *
- * Call +virNetworkGetBridgeName+[http://www.libvirt.org/html/libvirt-libvirt.html#virNetworkGetBridgeName]
+ * Call virNetworkGetBridgeName[http://www.libvirt.org/html/libvirt-libvirt.html#virNetworkGetBridgeName]
  * to retrieve the bridge name for this network.
  */
-static VALUE libvirt_netw_bridge_name(VALUE s) {
-    gen_call_string(virNetworkGetBridgeName, conn(s), 1, network_get(s));
+static VALUE libvirt_network_bridge_name(VALUE n)
+{
+    ruby_libvirt_generate_call_string(virNetworkGetBridgeName,
+                                      ruby_libvirt_connect_get(n),
+                                      1, network_get(n));
 }
 
 /*
  * call-seq:
  *   net.autostart? -> [true|false]
  *
- * Call +virNetworkGetAutostart+[http://www.libvirt.org/html/libvirt-libvirt.html#virNetworkGetAutostart]
+ * Call virNetworkGetAutostart[http://www.libvirt.org/html/libvirt-libvirt.html#virNetworkGetAutostart]
  * to determine if this network will be autostarted when libvirtd starts.
  */
-static VALUE libvirt_netw_autostart(VALUE s){
-    virNetworkPtr netw = network_get(s);
+static VALUE libvirt_network_autostart(VALUE n)
+{
     int r, autostart;
 
-    r = virNetworkGetAutostart(netw, &autostart);
-    _E(r < 0, create_error(e_RetrieveError, "virNetworkAutostart", conn(s)));
+    r = virNetworkGetAutostart(network_get(n), &autostart);
+    ruby_libvirt_raise_error_if(r < 0, e_RetrieveError, "virNetworkAutostart",
+                                ruby_libvirt_connect_get(n));
 
     return autostart ? Qtrue : Qfalse;
 }
@@ -154,27 +187,31 @@ static VALUE libvirt_netw_autostart(VALUE s){
  * call-seq:
  *   net.autostart = [true|false]
  *
- * Call +virNetworkSetAutostart+[http://www.libvirt.org/html/libvirt-libvirt.html#virNetworkSetAutostart]
+ * Call virNetworkSetAutostart[http://www.libvirt.org/html/libvirt-libvirt.html#virNetworkSetAutostart]
  * to set this network to be autostarted when libvirtd starts.
  */
-static VALUE libvirt_netw_autostart_set(VALUE s, VALUE autostart) {
-    if (autostart != Qtrue && autostart != Qfalse)
+static VALUE libvirt_network_autostart_equal(VALUE n, VALUE autostart)
+{
+    if (autostart != Qtrue && autostart != Qfalse) {
         rb_raise(rb_eTypeError,
                  "wrong argument type (expected TrueClass or FalseClass)");
+    }
 
-    gen_call_void(virNetworkSetAutostart, conn(s), network_get(s),
-                  RTEST(autostart) ? 1 : 0);
+    ruby_libvirt_generate_call_nil(virNetworkSetAutostart,
+                                   ruby_libvirt_connect_get(n),
+                                   network_get(n), RTEST(autostart) ? 1 : 0);
 }
 
 /*
  * call-seq:
  *   net.free -> nil
  *
- * Call +virNetworkFree+[http://www.libvirt.org/html/libvirt-libvirt.html#virNetworkFree]
+ * Call virNetworkFree[http://www.libvirt.org/html/libvirt-libvirt.html#virNetworkFree]
  * to free this network.  The object will no longer be valid after this call.
  */
-static VALUE libvirt_netw_free(VALUE s) {
-    gen_call_free(Network, s);
+static VALUE libvirt_network_free(VALUE n)
+{
+    ruby_libvirt_generate_call_free(Network, n);
 }
 
 #if HAVE_VIRNETWORKISACTIVE
@@ -182,11 +219,14 @@ static VALUE libvirt_netw_free(VALUE s) {
  * call-seq:
  *   net.active? -> [true|false]
  *
- * Call +virNetworkIsActive+[http://www.libvirt.org/html/libvirt-libvirt.html#virNetworkIsActive]
+ * Call virNetworkIsActive[http://www.libvirt.org/html/libvirt-libvirt.html#virNetworkIsActive]
  * to determine if this network is currently active.
  */
-static VALUE libvirt_netw_active_p(VALUE s) {
-    gen_call_truefalse(virNetworkIsActive, conn(s), network_get(s));
+static VALUE libvirt_network_active_p(VALUE n)
+{
+    ruby_libvirt_generate_call_truefalse(virNetworkIsActive,
+                                         ruby_libvirt_connect_get(n),
+                                         network_get(n));
 }
 #endif
 
@@ -195,11 +235,14 @@ static VALUE libvirt_netw_active_p(VALUE s) {
  * call-seq:
  *   net.persistent? -> [true|false]
  *
- * Call +virNetworkIsPersistent+[http://www.libvirt.org/html/libvirt-libvirt.html#virNetworkIsPersistent]
+ * Call virNetworkIsPersistent[http://www.libvirt.org/html/libvirt-libvirt.html#virNetworkIsPersistent]
  * to determine if this network is persistent.
  */
-static VALUE libvirt_netw_persistent_p(VALUE s) {
-    gen_call_truefalse(virNetworkIsPersistent, conn(s), network_get(s));
+static VALUE libvirt_network_persistent_p(VALUE n)
+{
+    ruby_libvirt_generate_call_truefalse(virNetworkIsPersistent,
+                                         ruby_libvirt_connect_get(n),
+                                         network_get(n));
 }
 #endif
 
@@ -208,28 +251,204 @@ static VALUE libvirt_netw_persistent_p(VALUE s) {
 /*
  * Class Libvirt::Network
  */
-void init_network()
+void ruby_libvirt_network_init(void)
 {
 #if HAVE_TYPE_VIRNETWORKPTR
     c_network = rb_define_class_under(m_libvirt, "Network", rb_cObject);
     rb_define_attr(c_network, "connection", 1, 0);
 
-    rb_define_method(c_network, "undefine", libvirt_netw_undefine, 0);
-    rb_define_method(c_network, "create", libvirt_netw_create, 0);
-    rb_define_method(c_network, "destroy", libvirt_netw_destroy, 0);
-    rb_define_method(c_network, "name", libvirt_netw_name, 0);
-    rb_define_method(c_network, "uuid", libvirt_netw_uuid, 0);
-    rb_define_method(c_network, "xml_desc", libvirt_netw_xml_desc, -1);
-    rb_define_method(c_network, "bridge_name", libvirt_netw_bridge_name, 0);
-    rb_define_method(c_network, "autostart", libvirt_netw_autostart, 0);
-    rb_define_method(c_network, "autostart?", libvirt_netw_autostart, 0);
-    rb_define_method(c_network, "autostart=", libvirt_netw_autostart_set, 1);
-    rb_define_method(c_network, "free", libvirt_netw_free, 0);
+    rb_define_method(c_network, "undefine", libvirt_network_undefine, 0);
+    rb_define_method(c_network, "create", libvirt_network_create, 0);
+#if HAVE_VIRNETWORKUPDATE
+    rb_define_method(c_network, "update", libvirt_network_update, 5);
+#endif
+    rb_define_method(c_network, "destroy", libvirt_network_destroy, 0);
+    rb_define_method(c_network, "name", libvirt_network_name, 0);
+    rb_define_method(c_network, "uuid", libvirt_network_uuid, 0);
+    rb_define_method(c_network, "xml_desc", libvirt_network_xml_desc, -1);
+    rb_define_method(c_network, "bridge_name", libvirt_network_bridge_name, 0);
+    rb_define_method(c_network, "autostart", libvirt_network_autostart, 0);
+    rb_define_method(c_network, "autostart?", libvirt_network_autostart, 0);
+    rb_define_method(c_network, "autostart=", libvirt_network_autostart_equal,
+                     1);
+    rb_define_method(c_network, "free", libvirt_network_free, 0);
 #if HAVE_VIRNETWORKISACTIVE
-    rb_define_method(c_network, "active?", libvirt_netw_active_p, 0);
+    rb_define_method(c_network, "active?", libvirt_network_active_p, 0);
 #endif
 #if HAVE_VIRNETWORKISPERSISTENT
-    rb_define_method(c_network, "persistent?", libvirt_netw_persistent_p, 0);
+    rb_define_method(c_network, "persistent?", libvirt_network_persistent_p, 0);
+#endif
+#if HAVE_CONST_VIR_NETWORK_UPDATE_COMMAND_NONE
+    /* Ideally we would just have the "UPDATE_COMMAND_NONE" constant.
+     * Unfortunately we screwed up long ago, and we have to
+     * leave "NETWORK_UPDATE_COMMAND_NONE" for backwards compatibility.
+     */
+    rb_define_const(c_network, "UPDATE_COMMAND_NONE",
+                    INT2NUM(VIR_NETWORK_UPDATE_COMMAND_NONE));
+    rb_define_const(c_network, "NETWORK_UPDATE_COMMAND_NONE",
+                    INT2NUM(VIR_NETWORK_UPDATE_COMMAND_NONE));
+    /* Ideally we would just have the "UPDATE_COMMAND_MODIFY" constant.
+     * Unfortunately we screwed up long ago, and we have to
+     * leave "NETWORK_UPDATE_COMMAND_MODIFY" for backwards compatibility.
+     */
+    rb_define_const(c_network, "UPDATE_COMMAND_MODIFY",
+                    INT2NUM(VIR_NETWORK_UPDATE_COMMAND_MODIFY));
+    rb_define_const(c_network, "NETWORK_UPDATE_COMMAND_MODIFY",
+                    INT2NUM(VIR_NETWORK_UPDATE_COMMAND_MODIFY));
+    /* Ideally we would just have the "UPDATE_COMMAND_ADD_LAST" constant.
+     * Unfortunately we screwed up long ago, and we have to
+     * leave "NETWORK_UPDATE_COMMAND_ADD_LAST" for backwards compatibility.
+     */
+    rb_define_const(c_network, "UPDATE_COMMAND_ADD_LAST",
+                    INT2NUM(VIR_NETWORK_UPDATE_COMMAND_ADD_LAST));
+    rb_define_const(c_network, "NETWORK_UPDATE_COMMAND_ADD_LAST",
+                    INT2NUM(VIR_NETWORK_UPDATE_COMMAND_ADD_LAST));
+    /* Ideally we would just have the "UPDATE_COMMAND_ADD_FIRST" constant.
+     * Unfortunately we screwed up long ago, and we have to
+     * leave "NETWORK_UPDATE_COMMAND_ADD_FIRST" for backwards compatibility.
+     */
+    rb_define_const(c_network, "UPDATE_COMMAND_ADD_FIRST",
+                    INT2NUM(VIR_NETWORK_UPDATE_COMMAND_ADD_FIRST));
+    rb_define_const(c_network, "NETWORK_UPDATE_COMMAND_ADD_FIRST",
+                    INT2NUM(VIR_NETWORK_UPDATE_COMMAND_ADD_FIRST));
+    /* Ideally we would just have the "SECTION_NONE" constant.
+     * Unfortunately we screwed up long ago, and we have to
+     * leave "NETWORK_SECTION_NONE" for backwards compatibility.
+     */
+    rb_define_const(c_network, "SECTION_NONE",
+                    INT2NUM(VIR_NETWORK_SECTION_NONE));
+    rb_define_const(c_network, "NETWORK_SECTION_NONE",
+                    INT2NUM(VIR_NETWORK_SECTION_NONE));
+    /* Ideally we would just have the "SECTION_BRIDGE" constant.
+     * Unfortunately we screwed up long ago, and we have to
+     * leave "NETWORK_SECTION_BRIDGE" for backwards compatibility.
+     */
+    rb_define_const(c_network, "SECTION_BRIDGE",
+                    INT2NUM(VIR_NETWORK_SECTION_BRIDGE));
+    rb_define_const(c_network, "NETWORK_SECTION_BRIDGE",
+                    INT2NUM(VIR_NETWORK_SECTION_BRIDGE));
+    /* Ideally we would just have the "SECTION_DOMAIN" constant.
+     * Unfortunately we screwed up long ago, and we have to
+     * leave "NETWORK_SECTION_DOMAIN" for backwards compatibility.
+     */
+    rb_define_const(c_network, "SECTION_DOMAIN",
+                    INT2NUM(VIR_NETWORK_SECTION_DOMAIN));
+    rb_define_const(c_network, "NETWORK_SECTION_DOMAIN",
+                    INT2NUM(VIR_NETWORK_SECTION_DOMAIN));
+    /* Ideally we would just have the "SECTION_IP" constant.
+     * Unfortunately we screwed up long ago, and we have to
+     * leave "NETWORK_SECTION_IP" for backwards compatibility.
+     */
+    rb_define_const(c_network, "SECTION_IP",
+                    INT2NUM(VIR_NETWORK_SECTION_IP));
+    rb_define_const(c_network, "NETWORK_SECTION_IP",
+                    INT2NUM(VIR_NETWORK_SECTION_IP));
+    /* Ideally we would just have the "SECTION_IP_DHCP_HOST" constant.
+     * Unfortunately we screwed up long ago, and we have to
+     * leave "NETWORK_SECTION_IP_DHCP_HOST" for backwards compatibility.
+     */
+    rb_define_const(c_network, "SECTION_IP_DHCP_HOST",
+                    INT2NUM(VIR_NETWORK_SECTION_IP_DHCP_HOST));
+    rb_define_const(c_network, "NETWORK_SECTION_IP_DHCP_HOST",
+                    INT2NUM(VIR_NETWORK_SECTION_IP_DHCP_HOST));
+    /* Ideally we would just have the "SECTION_IP_DHCP_RANGE" constant.
+     * Unfortunately we screwed up long ago, and we have to
+     * leave "NETWORK_SECTION_IP_DHCP_RANGE" for backwards compatibility.
+     */
+    rb_define_const(c_network, "SECTION_IP_DHCP_RANGE",
+                    INT2NUM(VIR_NETWORK_SECTION_IP_DHCP_RANGE));
+    rb_define_const(c_network, "NETWORK_SECTION_IP_DHCP_RANGE",
+                    INT2NUM(VIR_NETWORK_SECTION_IP_DHCP_RANGE));
+    /* Ideally we would just have the "SECTION_FORWARD" constant.
+     * Unfortunately we screwed up long ago, and we have to
+     * leave "NETWORK_SECTION_FORWARD" for backwards compatibility.
+     */
+    rb_define_const(c_network, "SECTION_FORWARD",
+                    INT2NUM(VIR_NETWORK_SECTION_FORWARD));
+    rb_define_const(c_network, "NETWORK_SECTION_FORWARD",
+                    INT2NUM(VIR_NETWORK_SECTION_FORWARD));
+    /* Ideally we would just have the "SECTION_FORWARD_INTERFACE" constant.
+     * Unfortunately we screwed up long ago, and we have to
+     * leave "NETWORK_SECTION_FORWARD_INTERFACE" for backwards compatibility.
+     */
+    rb_define_const(c_network, "SECTION_FORWARD_INTERFACE",
+                    INT2NUM(VIR_NETWORK_SECTION_FORWARD_INTERFACE));
+    rb_define_const(c_network, "NETWORK_SECTION_FORWARD_INTERFACE",
+                    INT2NUM(VIR_NETWORK_SECTION_FORWARD_INTERFACE));
+    /* Ideally we would just have the "SECTION_FORWARD_PF" constant.
+     * Unfortunately we screwed up long ago, and we have to
+     * leave "NETWORK_SECTION_FORWARD_PF" for backwards compatibility.
+     */
+    rb_define_const(c_network, "SECTION_FORWARD_PF",
+                    INT2NUM(VIR_NETWORK_SECTION_FORWARD_PF));
+    rb_define_const(c_network, "NETWORK_SECTION_FORWARD_PF",
+                    INT2NUM(VIR_NETWORK_SECTION_FORWARD_PF));
+    /* Ideally we would just have the "SECTION_PORTGROUP" constant.
+     * Unfortunately we screwed up long ago, and we have to
+     * leave "NETWORK_SECTION_PORTGROUP" for backwards compatibility.
+     */
+    rb_define_const(c_network, "SECTION_PORTGROUP",
+                    INT2NUM(VIR_NETWORK_SECTION_PORTGROUP));
+    rb_define_const(c_network, "NETWORK_SECTION_PORTGROUP",
+                    INT2NUM(VIR_NETWORK_SECTION_PORTGROUP));
+    /* Ideally we would just have the "SECTION_DNS_HOST" constant.
+     * Unfortunately we screwed up long ago, and we have to
+     * leave "NETWORK_SECTION_DNS_HOST" for backwards compatibility.
+     */
+    rb_define_const(c_network, "SECTION_DNS_HOST",
+                    INT2NUM(VIR_NETWORK_SECTION_DNS_HOST));
+    rb_define_const(c_network, "NETWORK_SECTION_DNS_HOST",
+                    INT2NUM(VIR_NETWORK_SECTION_DNS_HOST));
+    /* Ideally we would just have the "SECTION_DNS_TXT" constant.
+     * Unfortunately we screwed up long ago, and we have to
+     * leave "NETWORK_SECTION_DNS_TXT" for backwards compatibility.
+     */
+    rb_define_const(c_network, "SECTION_DNS_TXT",
+                    INT2NUM(VIR_NETWORK_SECTION_DNS_TXT));
+    rb_define_const(c_network, "NETWORK_SECTION_DNS_TXT",
+                    INT2NUM(VIR_NETWORK_SECTION_DNS_TXT));
+    /* Ideally we would just have the "SECTION_DNS_SRV" constant.
+     * Unfortunately we screwed up long ago, and we have to
+     * leave "NETWORK_SECTION_DNS_SRV" for backwards compatibility.
+     */
+    rb_define_const(c_network, "SECTION_DNS_SRV",
+                    INT2NUM(VIR_NETWORK_SECTION_DNS_SRV));
+    rb_define_const(c_network, "NETWORK_SECTION_DNS_SRV",
+                    INT2NUM(VIR_NETWORK_SECTION_DNS_SRV));
+    /* Ideally we would just have the "UPDATE_AFFECT_CURRENT" constant.
+     * Unfortunately we screwed up long ago, and we have to
+     * leave "NETWORK_UPDATE_AFFECT_CURRENT" for backwards compatibility.
+     */
+    rb_define_const(c_network, "UPDATE_AFFECT_CURRENT",
+                    INT2NUM(VIR_NETWORK_UPDATE_AFFECT_CURRENT));
+    rb_define_const(c_network, "NETWORK_UPDATE_AFFECT_CURRENT",
+                    INT2NUM(VIR_NETWORK_UPDATE_AFFECT_CURRENT));
+    /* Ideally we would just have the "UPDATE_AFFECT_LIVE" constant.
+     * Unfortunately we screwed up long ago, and we have to
+     * leave "NETWORK_UPDATE_AFFECT_LIVE" for backwards compatibility.
+     */
+    rb_define_const(c_network, "UPDATE_AFFECT_LIVE",
+                    INT2NUM(VIR_NETWORK_UPDATE_AFFECT_LIVE));
+    rb_define_const(c_network, "NETWORK_UPDATE_AFFECT_LIVE",
+                    INT2NUM(VIR_NETWORK_UPDATE_AFFECT_LIVE));
+    /* Ideally we would just have the "UPDATE_AFFECT_CONFIG" constant.
+     * Unfortunately we screwed up long ago, and we have to
+     * leave "NETWORK_UPDATE_AFFECT_CONFIG" for backwards compatibility.
+     */
+    rb_define_const(c_network, "UPDATE_AFFECT_CONFIG",
+                    INT2NUM(VIR_NETWORK_UPDATE_AFFECT_CONFIG));
+    rb_define_const(c_network, "NETWORK_UPDATE_AFFECT_CONFIG",
+                    INT2NUM(VIR_NETWORK_UPDATE_AFFECT_CONFIG));
 #endif
+
+#if HAVE_CONST_VIR_NETWORK_XML_INACTIVE
+    rb_define_const(c_network, "XML_INACTIVE",
+                    INT2NUM(VIR_NETWORK_XML_INACTIVE));
+#endif
+#if HAVE_CONST_VIR_NETWORK_UPDATE_COMMAND_DELETE
+    rb_define_const(c_network, "UPDATE_COMMAND_DELETE",
+                    INT2NUM(VIR_NETWORK_UPDATE_COMMAND_DELETE));
+#endif
+
 #endif
 }