Reduce list of dependencies by porting auth_dialog to plain gtk/gconf. Patch attached. -- Why is it that all of the instruments seeking intelligent life in the universe are pointed away from Earth?
From 1fdd75f4d0bd7fdc7b2b450d07620cd51dc609b9 Mon Sep 17 00:00:00 2001
From: Michael Biebl <biebl debian org>
Date: Thu, 18 Dec 2008 03:03:26 +0100
Subject: [PATCH] Remove libgnomeui dependency. Use plain gtk/gconf.
---
auth-dialog/Makefile.am | 4 ++--
auth-dialog/main.c | 25 ++++++++++---------------
configure.in | 4 ----
3 files changed, 12 insertions(+), 21 deletions(-)
diff --git a/auth-dialog/Makefile.am b/auth-dialog/Makefile.am
index 28aaf7f..f1e5401 100644
--- a/auth-dialog/Makefile.am
+++ b/auth-dialog/Makefile.am
@@ -6,7 +6,7 @@ nm_pptp_auth_dialog_CPPFLAGS = \
$(NM_UTILS_CFLAGS) \
$(GTHREAD_CFLAGS) \
$(GTK_CFLAGS) \
- $(LIBGNOMEUI_CFLAGS) \
+ $(GCONF_CFLAGS) \
$(GNOMEKEYRING_CFLAGS) \
-DICONDIR=\""$(datadir)/pixmaps"\" \
-DGLADEDIR=\""$(gladedir)"\" \
@@ -27,7 +27,7 @@ nm_pptp_auth_dialog_SOURCES = \
nm_pptp_auth_dialog_LDADD = \
$(GTK_LIBS) \
- $(LIBGNOMEUI_LIBS) \
+ $(GCONF_LIBS) \
$(top_builddir)/common-gnome/libnm-pptp-common-gnome.la \
$(NULL)
diff --git a/auth-dialog/main.c b/auth-dialog/main.c
index 6662f24..7b4e9a9 100644
--- a/auth-dialog/main.c
+++ b/auth-dialog/main.c
@@ -25,9 +25,9 @@
#endif
#include <string.h>
+#include <stdlib.h>
#include <glib/gi18n.h>
#include <gtk/gtk.h>
-#include <libgnomeui/libgnomeui.h>
#include <nm-setting-vpn.h>
@@ -128,9 +128,8 @@ main (int argc, char *argv[])
gchar *vpn_service = NULL;
char *password = NULL;
char buf[1];
- int ret, exit_status = 1;
+ int ret;
GOptionContext *context;
- GnomeProgram *program;
GOptionEntry entries[] = {
{ "reprompt", 'r', 0, G_OPTION_ARG_NONE, &retry, "Reprompt for passwords", NULL},
{ "uuid", 'u', 0, G_OPTION_ARG_STRING, &vpn_uuid, "UUID of VPN connection", NULL},
@@ -143,27 +142,26 @@ main (int argc, char *argv[])
bind_textdomain_codeset (GETTEXT_PACKAGE, "UTF-8");
textdomain (GETTEXT_PACKAGE);
+ gtk_init (&argc, &argv);
+
context = g_option_context_new ("- pptp auth dialog");
g_option_context_add_main_entries (context, entries, GETTEXT_PACKAGE);
+ g_option_context_parse (context, &argc, &argv, NULL);
+ g_option_context_free (context);
- program = gnome_program_init ("nm-pptp-auth-dialog", VERSION,
- LIBGNOMEUI_MODULE,
- argc, argv,
- GNOME_PARAM_GOPTION_CONTEXT, context,
- GNOME_PARAM_NONE);
if (vpn_uuid == NULL || vpn_name == NULL || vpn_service == NULL) {
fprintf (stderr, "Have to supply UUID, name, and service\n");
- goto out;
+ return EXIT_FAILURE;
}
if (strcmp (vpn_service, NM_DBUS_SERVICE_PPTP) != 0) {
fprintf (stderr, "This dialog only works with the '%s' service\n", NM_DBUS_SERVICE_PPTP);
- goto out;
+ return EXIT_FAILURE;
}
if (!get_secrets (vpn_uuid, vpn_name, vpn_service, retry, &password))
- goto out;
+ return EXIT_FAILURE;
/* dump the passwords to stdout */
printf ("%s\n%s\n", NM_PPTP_KEY_PASSWORD, password);
@@ -173,7 +171,6 @@ main (int argc, char *argv[])
memset (password, 0, strlen (password));
gnome_keyring_memory_free (password);
}
- exit_status = 0;
/* for good measure, flush stdout since Kansas is going Bye-Bye */
fflush (stdout);
@@ -181,7 +178,5 @@ main (int argc, char *argv[])
/* wait for data on stdin */
ret = fread (buf, sizeof (char), sizeof (buf), stdin);
-out:
- g_object_unref (program);
- return exit_status;
+ return EXIT_SUCCESS;
}
diff --git a/configure.in b/configure.in
index 85719e4..174372e 100644
--- a/configure.in
+++ b/configure.in
@@ -85,10 +85,6 @@ if test x"$with_gnome" != xno; then
AC_SUBST(GLADE_CFLAGS)
AC_SUBST(GLADE_LIBS)
- PKG_CHECK_MODULES(LIBGNOMEUI, libgnomeui-2.0)
- AC_SUBST(LIBGNOMEUI_CFLAGS)
- AC_SUBST(LIBGNOMEUI_LIBS)
-
PKG_CHECK_MODULES(GCONF, gconf-2.0)
AC_SUBST(GCONF_CFLAGS)
AC_SUBST(GCONF_LIBS)
--
1.6.0.4
Attachment:
signature.asc
Description: OpenPGP digital signature