Re: [UPDATE] Re: [PATCH] moving vpn helpers to libexedir / vpn libs to $(libdir)/NetworkManager

From: Michael Biebl <biebl debian org>
To: Dan Williams <dcbw redhat com>, network manager <networkmanager-list gnome org>
Subject: Re: [UPDATE] Re: [PATCH] moving vpn helpers to libexedir / vpn libs to $(libdir)/NetworkManager
Date: Mon, 21 Jul 2008 07:09:21 +0200

Dan Williams wrote:

On Mon, 2008-07-21 at 02:26 +0200, Michael Biebl wrote:

Please review and apply.


Seems the patch didn't make it?


Argh, right. Now attached.

Michael
--
Why is it that all of the instruments seeking intelligent life in the
universe are pointed away from Earth?

Index: vpn-daemons/pptp/src/Makefile.am
===================================================================
--- vpn-daemons/pptp/src/Makefile.am	(Revision 3838)
+++ vpn-daemons/pptp/src/Makefile.am	(Arbeitskopie)
@@ -4,7 +4,6 @@
 	$(DBUS_GLIB_CFLAGS)						\
 	$(GTHREAD_CFLAGS)					\
 	$(NM_UTILS_CFLAGS)				\
-	-Wall							\
 	-DDBUS_API_SUBJECT_TO_CHANGE				\
 	-DG_DISABLE_DEPRECATED					\
 	-DBINDIR=\"$(bindir)\"					\
@@ -15,10 +14,9 @@
 	-DLIBEXECDIR=\""$(libexecdir)"\"			\
 	-DLOCALSTATEDIR=\""$(localstatedir)"\"		 	\
 	-DDATADIR=\"$(datadir)\"				\
-	-DPLUGINDIR=\"$(PPPD_PLUGIN_DIR)\" \
-	-fPIC
+	-DPLUGINDIR=\"$(PPPD_PLUGIN_DIR)\" 
 
-bin_PROGRAMS = nm-pptp-service
+libexec_PROGRAMS = nm-pptp-service
 
 nm_pptp_service_SOURCES = \
 				nm-pptp-service.c \
Index: vpn-daemons/pptp/properties/Makefile.am
===================================================================
--- vpn-daemons/pptp/properties/Makefile.am	(Revision 3838)
+++ vpn-daemons/pptp/properties/Makefile.am	(Arbeitskopie)
@@ -1,4 +1,5 @@
-lib_LTLIBRARIES = libnm-pptp-properties.la
+plugindir = $(libdir)/NetworkManager
+plugin_LTLIBRARIES = libnm-pptp-properties.la
 
 libnm_pptp_properties_la_SOURCES = \
 	nm-pptp.c \
Index: vpn-daemons/pptp/Makefile.am
===================================================================
--- vpn-daemons/pptp/Makefile.am	(Revision 3838)
+++ vpn-daemons/pptp/Makefile.am	(Arbeitskopie)
@@ -22,10 +22,8 @@
 icon_DATA = gnome-mime-application-x-pptp-settings.png
 endif
 
-nm-pptp-service.name: $(srcdir)/nm-pptp-service.name.in Makefile
-	sed	-e 's|[ ]BINDIR[@]|$(bindir)|g' \
-		< $(srcdir)/nm-pptp-service.name.in \
-		> nm-pptp-service.name
+nm-pptp-service.name: $(srcdir)/nm-pptp-service.name.in
+	sed -e 's|[ ]LIBEXECDIR[@]|$(libexecdir)|g' $< >$@
 
 EXTRA_DIST = nm-pptp-service.name.in \
              $(dbusservice_DATA)  \
Index: vpn-daemons/pptp/nm-pptp-service.name.in
===================================================================
--- vpn-daemons/pptp/nm-pptp-service.name.in	(Revision 3838)
+++ vpn-daemons/pptp/nm-pptp-service.name.in	(Arbeitskopie)
@@ -1,7 +1,7 @@
 [VPN Connection]
 name=pptp
 service=org.freedesktop.NetworkManager.pptp
-program= BINDIR@/nm-pptp-service
+program= LIBEXECDIR@/nm-pptp-service
 
 [GNOME]
 auth-dialog=nm-pptp-auth-dialog
Index: vpn-daemons/openvpn/nm-openvpn-service.name.in
===================================================================
--- vpn-daemons/openvpn/nm-openvpn-service.name.in	(Revision 3838)
+++ vpn-daemons/openvpn/nm-openvpn-service.name.in	(Arbeitskopie)
@@ -1,7 +1,7 @@
 [VPN Connection]
 name=openvpn
 service=org.freedesktop.NetworkManager.openvpn
-program= BINDIR@/nm-openvpn-service
+program= LIBEXECDIR@/nm-openvpn-service
 
 [GNOME]
 auth-dialog=nm-openvpn-auth-dialog
Index: vpn-daemons/openvpn/src/Makefile.am
===================================================================
--- vpn-daemons/openvpn/src/Makefile.am	(Revision 3838)
+++ vpn-daemons/openvpn/src/Makefile.am	(Arbeitskopie)
@@ -1,7 +1,6 @@
 AM_CPPFLAGS =							\
 	$(DBUS_CFLAGS)						\
 	$(NETWORK_MANAGER_CFLAGS)				\
-	-Wall							\
 	-DDBUS_API_SUBJECT_TO_CHANGE				\
 	-DG_DISABLE_DEPRECATED					\
 	-DBINDIR=\"$(bindir)\"					\
@@ -13,7 +12,7 @@
 	-DLOCALSTATEDIR=\""$(localstatedir)"\"		 	\
 	-DDATADIR=\"$(datadir)\"
 
-bin_PROGRAMS = nm-openvpn-service nm-openvpn-service-openvpn-helper
+libexec_PROGRAMS = nm-openvpn-service nm-openvpn-service-openvpn-helper
 
 nm_openvpn_service_SOURCES =				\
 				nm-openvpn-service.c	\
Index: vpn-daemons/openvpn/src/nm-openvpn-service.c
===================================================================
--- vpn-daemons/openvpn/src/nm-openvpn-service.c	(Revision 3838)
+++ vpn-daemons/openvpn/src/nm-openvpn-service.c	(Arbeitskopie)
@@ -53,7 +53,7 @@
 #include "nm-openvpn-service.h"
 #include "nm-utils.h"
 
-#define NM_OPENVPN_HELPER_PATH		BINDIR"/nm-openvpn-service-openvpn-helper"
+#define NM_OPENVPN_HELPER_PATH		LIBEXECDIR"/nm-openvpn-service-openvpn-helper"
 
 G_DEFINE_TYPE (NMOpenvpnPlugin, nm_openvpn_plugin, NM_TYPE_VPN_PLUGIN)
 
Index: vpn-daemons/openvpn/properties/Makefile.am
===================================================================
--- vpn-daemons/openvpn/properties/Makefile.am	(Revision 3838)
+++ vpn-daemons/openvpn/properties/Makefile.am	(Arbeitskopie)
@@ -1,7 +1,6 @@
+plugindir = $(libdir)/NetworkManager
+plugin_LTLIBRARIES = libnm-openvpn-properties.la
 
-
-lib_LTLIBRARIES = libnm-openvpn-properties.la
-
 libnm_openvpn_properties_la_SOURCES = \
 	nm-openvpn.c \
 	nm-openvpn.h \
Index: vpn-daemons/openvpn/Makefile.am
===================================================================
--- vpn-daemons/openvpn/Makefile.am	(Revision 3838)
+++ vpn-daemons/openvpn/Makefile.am	(Arbeitskopie)
@@ -22,10 +22,8 @@
 icon_DATA = gnome-mime-application-x-openvpn-settings.png
 endif
 
-nm-openvpn-service.name: $(srcdir)/nm-openvpn-service.name.in Makefile
-	sed	-e 's|[ ]BINDIR[@]|$(bindir)|g' \
-		< $(srcdir)/nm-openvpn-service.name.in \
-		> nm-openvpn-service.name
+nm-openvpn-service.name: $(srcdir)/nm-openvpn-service.name.in
+	sed -e 's|[ ]LIBEXECDIR[@]|$(libexecdir)|g' $< >$@
 
 EXTRA_DIST = nm-openvpn-service.name.in \
              $(dbusservice_DATA)  \
Index: vpn-daemons/vpnc/src/Makefile.am
===================================================================
--- vpn-daemons/vpnc/src/Makefile.am	(Revision 3838)
+++ vpn-daemons/vpnc/src/Makefile.am	(Arbeitskopie)
@@ -4,7 +4,6 @@
 	$(DBUS_CFLAGS) \
 	$(GTHREAD_CFLAGS) \
 	$(NM_UTILS_CFLAGS) \
-	-Wall \
 	-DDBUS_API_SUBJECT_TO_CHANGE \
 	-DG_DISABLE_DEPRECATED \
 	-DBINDIR=\"$(bindir)\" \
@@ -16,7 +15,7 @@
 	-DLOCALSTATEDIR=\""$(localstatedir)"\" \
 	-DDATADIR=\"$(datadir)\"
 
-bin_PROGRAMS = nm-vpnc-service nm-vpnc-service-vpnc-helper
+libexec_PROGRAMS = nm-vpnc-service nm-vpnc-service-vpnc-helper
 
 nm_vpnc_service_SOURCES = \
 				nm-vpnc-service.c \
Index: vpn-daemons/vpnc/src/nm-vpnc-service.c
===================================================================
--- vpn-daemons/vpnc/src/nm-vpnc-service.c	(Revision 3838)
+++ vpn-daemons/vpnc/src/nm-vpnc-service.c	(Arbeitskopie)
@@ -30,7 +30,7 @@
 	NULL
 };
 
-#define NM_VPNC_HELPER_PATH		BINDIR"/nm-vpnc-service-vpnc-helper"
+#define NM_VPNC_HELPER_PATH		LIBEXECDIR"/nm-vpnc-service-vpnc-helper"
 #define NM_VPNC_UDP_ENCAPSULATION_PORT	0 /* random port */
 
 typedef struct {
Index: vpn-daemons/vpnc/properties/Makefile.am
===================================================================
--- vpn-daemons/vpnc/properties/Makefile.am	(Revision 3838)
+++ vpn-daemons/vpnc/properties/Makefile.am	(Arbeitskopie)
@@ -1,7 +1,6 @@
+plugindir = $(libdir)/NetworkManager
+plugin_LTLIBRARIES = libnm-vpnc-properties.la
 
-
-lib_LTLIBRARIES = libnm-vpnc-properties.la
-
 libnm_vpnc_properties_la_SOURCES = \
 	pcf-file.c \
 	pcf-file.h \
Index: vpn-daemons/vpnc/nm-vpnc-service.name.in
===================================================================
--- vpn-daemons/vpnc/nm-vpnc-service.name.in	(Revision 3838)
+++ vpn-daemons/vpnc/nm-vpnc-service.name.in	(Arbeitskopie)
@@ -1,7 +1,7 @@
 [VPN Connection]
 name=vpnc
 service=org.freedesktop.NetworkManager.vpnc
-program= BINDIR@/nm-vpnc-service
+program= LIBEXECDIR@/nm-vpnc-service
 
 [GNOME]
 auth-dialog=nm-vpnc-auth-dialog
Index: vpn-daemons/vpnc/Makefile.am
===================================================================
--- vpn-daemons/vpnc/Makefile.am	(Revision 3838)
+++ vpn-daemons/vpnc/Makefile.am	(Arbeitskopie)
@@ -22,10 +22,8 @@
 icon_DATA = gnome-mime-application-x-cisco-vpn-settings.png
 endif
 
-nm-vpnc-service.name: $(srcdir)/nm-vpnc-service.name.in Makefile
-	sed	-e 's|[ ]BINDIR[@]|$(bindir)|g' \
-		< $(srcdir)/nm-vpnc-service.name.in \
-		> nm-vpnc-service.name
+nm-vpnc-service.name: $(srcdir)/nm-vpnc-service.name.in
+	sed -e 's|[ ]LIBEXECDIR[@]|$(libexecdir)|g' $< >$@
 
 EXTRA_DIST = nm-vpnc-service.name.in \
              $(dbusservice_DATA)  \
Index: marshallers/Makefile.am
===================================================================
--- marshallers/Makefile.am	(Revision 3838)
+++ marshallers/Makefile.am	(Arbeitskopie)
@@ -9,8 +9,6 @@
 
 libmarshallers_la_CPPFLAGS = \
 	$(GTHREAD_CFLAGS) \
-	-g \
-	-Wall \
 	-DG_DISABLE_DEPRECATED

Attachment: signature.asc
Description: OpenPGP digital signature

Follow-Ups:
- Re: [UPDATE] Re: [PATCH] moving vpn helpers to libexedir / vpn libs to $(libdir)/NetworkManager
  - From: Michael Biebl

References:
- [PATCH] moving vpn helpers to libexedir / vpn libs to $(libdir)/NetworkManager
  - From: Michael Biebl
- [UPDATE] Re: [PATCH] moving vpn helpers to libexedir / vpn libs to $(libdir)/NetworkManager
  - From: Michael Biebl
- Re: [UPDATE] Re: [PATCH] moving vpn helpers to libexedir / vpn libs to $(libdir)/NetworkManager
  - From: Dan Williams

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