2008-11-25 Dan Williams <dcbw@redhat.com>
authorDan Williams <dcbw@redhat.com>
Tue, 25 Nov 2008 22:42:21 +0000 (22:42 +0000)
committerDan Williams <dcbw@redhat.com>
Tue, 25 Nov 2008 22:42:21 +0000 (22:42 +0000)
* system-settings/plugins/ifcfg-rh
- Rename 'ifcfg-fedora' to 'ifcfg-rh'; it's not just used on Fedora

* system-settings/src/main.c
- (load_plugins): transparently handle ifcfg-fedora as ifcfg-rh

git-svn-id: http://svn-archive.gnome.org/svn/NetworkManager/trunk@4337 4912f4e0-d625-0410-9fb7-b9a5a253dbdc

14 files changed:
ChangeLog
configure.in
system-settings/plugins/Makefile.am
system-settings/plugins/ifcfg-rh/Makefile.am [moved from system-settings/plugins/ifcfg-fedora/Makefile.am with 63% similarity]
system-settings/plugins/ifcfg-rh/common.h [moved from system-settings/plugins/ifcfg-fedora/common.h with 96% similarity]
system-settings/plugins/ifcfg-rh/nm-ifcfg-connection.c [moved from system-settings/plugins/ifcfg-fedora/nm-ifcfg-connection.c with 100% similarity]
system-settings/plugins/ifcfg-rh/nm-ifcfg-connection.h [moved from system-settings/plugins/ifcfg-fedora/nm-ifcfg-connection.h with 100% similarity]
system-settings/plugins/ifcfg-rh/plugin.c [moved from system-settings/plugins/ifcfg-fedora/plugin.c with 100% similarity]
system-settings/plugins/ifcfg-rh/plugin.h [moved from system-settings/plugins/ifcfg-fedora/plugin.h with 100% similarity]
system-settings/plugins/ifcfg-rh/reader.c [moved from system-settings/plugins/ifcfg-fedora/reader.c with 100% similarity]
system-settings/plugins/ifcfg-rh/reader.h [moved from system-settings/plugins/ifcfg-fedora/reader.h with 100% similarity]
system-settings/plugins/ifcfg-rh/shvar.c [moved from system-settings/plugins/ifcfg-fedora/shvar.c with 100% similarity]
system-settings/plugins/ifcfg-rh/shvar.h [moved from system-settings/plugins/ifcfg-fedora/shvar.h with 100% similarity]
system-settings/src/main.c

index 46211c7..2904e1d 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2008-11-25  Dan Williams  <dcbw@redhat.com>
+
+       * system-settings/plugins/ifcfg-rh
+               - Rename 'ifcfg-fedora' to 'ifcfg-rh'; it's not just used on Fedora
+
+       * system-settings/src/main.c
+               - (load_plugins): transparently handle ifcfg-fedora as ifcfg-rh
+
 2008-11-25  Dan Williams  <dcbw@redhat.com>
 
        Patch from Tambet Ingo <tambet@gmail.com>
index b11cc10..8f6f425 100644 (file)
@@ -501,7 +501,7 @@ system-settings/Makefile
 system-settings/src/Makefile
 system-settings/plugins/Makefile
 system-settings/plugins/ifupdown/Makefile
-system-settings/plugins/ifcfg-fedora/Makefile
+system-settings/plugins/ifcfg-rh/Makefile
 system-settings/plugins/ifcfg-suse/Makefile
 system-settings/plugins/keyfile/Makefile
 test/Makefile
index 84a9825..94f7560 100644 (file)
@@ -1,7 +1,7 @@
 SUBDIRS=keyfile
 
 if TARGET_REDHAT
-SUBDIRS+=ifcfg-fedora
+SUBDIRS+=ifcfg-rh
 endif
 
 if TARGET_SUSE
@@ -9,7 +9,7 @@ SUBDIRS+=ifcfg-suse
 endif
 
 if TARGET_MANDRIVA
-SUBDIRS+=ifcfg-fedora
+SUBDIRS+=ifcfg-rh
 endif
 
 if TARGET_DEBIAN
@@ -1,7 +1,7 @@
 
-pkglib_LTLIBRARIES = libnm-settings-plugin-ifcfg-fedora.la
+pkglib_LTLIBRARIES = libnm-settings-plugin-ifcfg-rh.la
 
-libnm_settings_plugin_ifcfg_fedora_la_SOURCES = \
+libnm_settings_plugin_ifcfg_rh_la_SOURCES = \
        shvar.c \
        shvar.h \
        plugin.c \
@@ -12,7 +12,7 @@ libnm_settings_plugin_ifcfg_fedora_la_SOURCES = \
        reader.h \
        common.h
 
-libnm_settings_plugin_ifcfg_fedora_la_CPPFLAGS = \
+libnm_settings_plugin_ifcfg_rh_la_CPPFLAGS = \
        $(GLIB_CFLAGS) \
        $(GMODULE_CFLAGS) \
        $(DBUS_CFLAGS) \
@@ -24,8 +24,8 @@ libnm_settings_plugin_ifcfg_fedora_la_CPPFLAGS = \
        -I$(top_builddir)/marshallers \
        -DSYSCONFDIR=\"$(sysconfdir)\"
 
-libnm_settings_plugin_ifcfg_fedora_la_LDFLAGS = -module -avoid-version
-libnm_settings_plugin_ifcfg_fedora_la_LIBADD = \
+libnm_settings_plugin_ifcfg_rh_la_LDFLAGS = -module -avoid-version
+libnm_settings_plugin_ifcfg_rh_la_LIBADD = \
        $(GLIB_LIBS) \
        $(GMODULE_LIBS) \
        $(top_builddir)/libnm-util/libnm-util.la \
@@ -33,10 +33,10 @@ libnm_settings_plugin_ifcfg_fedora_la_LIBADD = \
        $(top_builddir)/marshallers/libmarshallers.la
 
 if NO_GIO
-libnm_settings_plugin_ifcfg_fedora_la_LIBADD += \
+libnm_settings_plugin_ifcfg_rh_la_LIBADD += \
        $(top_builddir)/gfilemonitor/libgfilemonitor.la
 else
-libnm_settings_plugin_ifcfg_fedora_la_LIBADD += \
+libnm_settings_plugin_ifcfg_rh_la_LIBADD += \
        $(GIO_LIBS)
 endif
 
similarity index 96%
rename from system-settings/plugins/ifcfg-fedora/common.h
rename to system-settings/plugins/ifcfg-rh/common.h
index 2980ed3..812bebc 100644 (file)
@@ -28,7 +28,7 @@
 #define ORIG_TAG ".orig"
 #define REJ_TAG ".rej"
 
-#define IFCFG_PLUGIN_NAME "ifcfg-fedora"
+#define IFCFG_PLUGIN_NAME "ifcfg-rh"
 #define IFCFG_PLUGIN_INFO "(c) 2007 - 2008 Red Hat, Inc.  To report bugs please use the NetworkManager mailing list."
 
 #include <glib.h>
index 9286486..9162ad2 100644 (file)
@@ -111,31 +111,35 @@ load_plugins (Application *app, const char *plugins, GError **error)
 {
        GSList *list = NULL;
        char **plist;
-       char **pname;
+       char **iter;
 
        plist = g_strsplit (plugins, ",", 0);
        if (!plist)
                return FALSE;
 
-       for (pname = plist; *pname; pname++) {
+       for (iter = plist; *iter; iter++) {
                GModule *plugin;
-               char *full_name;
-               char *path;
+               char *full_name, *path;
+               const char *pname = *iter;
                GObject *obj;
                GObject * (*factory_func) (void);
 
-               obj = find_plugin (list, *pname);
+               /* ifcfg-fedora was renamed ifcfg-rh; handle old configs here */
+               if (!strcmp (pname, "ifcfg-fedora"))
+                       pname = "ifcfg-rh";
+
+               obj = find_plugin (list, pname);
                if (obj)
                        continue;
 
-               full_name = g_strdup_printf ("nm-settings-plugin-%s", *pname);
+               full_name = g_strdup_printf ("nm-settings-plugin-%s", pname);
                path = g_module_build_path (PLUGINDIR, full_name);
 
                plugin = g_module_open (path, G_MODULE_BIND_LOCAL);
                if (!plugin) {
                        g_set_error (error, plugins_error_quark (), 0,
                                     "Could not load plugin '%s': %s",
-                                    *pname, g_module_error ());
+                                    pname, g_module_error ());
                        g_free (full_name);
                        g_free (path);
                        break;
@@ -147,7 +151,7 @@ load_plugins (Application *app, const char *plugins, GError **error)
                if (!g_module_symbol (plugin, "nm_system_config_factory", (gpointer) (&factory_func))) {
                        g_set_error (error, plugins_error_quark (), 0,
                                     "Could not find plugin '%s' factory function.",
-                                    *pname);
+                                    pname);
                        break;
                }
 
@@ -155,7 +159,7 @@ load_plugins (Application *app, const char *plugins, GError **error)
                if (!obj || !NM_IS_SYSTEM_CONFIG_INTERFACE (obj)) {
                        g_set_error (error, plugins_error_quark (), 0,
                                     "Plugin '%s' returned invalid system config object.",
-                                    *pname);
+                                    pname);
                        break;
                }