Re: small fix for nm-openvpn-service: could not find the openvpn binary

2009-10-07 Thread Dan Williams
On Wed, 2009-10-07 at 22:01 +0800, Juvenn Woo wrote:
> I compiled openvpn-2.1_rc13 from source, that installed the openvpn
> binary only to /usr/local/sbin, and neither /usr/sbin nor /sbin. While
> network-manager-openvpn only includes /usr/sbin/openvpn and
> /sbin/openvpn as binary paths to search. The network manager applet
> then complains of "Could not find the openvpn binary", every time I
> try to connect. Afte a little look into source of
> nm-openvpn-service.c, I came up a small fix:

Thanks, change pushed to stable and master.

Dan

> diff --git a/src/nm-openvpn-service.c b/src/nm-openvpn-service.c
> index 857d539..c4da0c3 100644
> --- a/src/nm-openvpn-service.c
> +++ b/src/nm-openvpn-service.c
> @@ -577,6 +577,7 @@ nm_find_openvpn (void)
> static const char *openvpn_binary_paths[] = {
> "/usr/sbin/openvpn",
> "/sbin/openvpn",
> +   "/usr/local/sbin/openvpn",
> NULL
> };
> const char  **openvpn_binary = openvpn_binary_paths;
> 
>  Hope this will make life easier!
> --
> Juvenn Woo
> ___
> NetworkManager-list mailing list
> NetworkManager-list@gnome.org
> http://mail.gnome.org/mailman/listinfo/networkmanager-list

___
NetworkManager-list mailing list
NetworkManager-list@gnome.org
http://mail.gnome.org/mailman/listinfo/networkmanager-list


small fix for nm-openvpn-service: could not find the openvpn binary

2009-10-07 Thread Juvenn Woo
I compiled openvpn-2.1_rc13 from source, that installed the openvpn
binary only to /usr/local/sbin, and neither /usr/sbin nor /sbin. While
network-manager-openvpn only includes /usr/sbin/openvpn and
/sbin/openvpn as binary paths to search. The network manager applet
then complains of "Could not find the openvpn binary", every time I
try to connect. Afte a little look into source of
nm-openvpn-service.c, I came up a small fix:

diff --git a/src/nm-openvpn-service.c b/src/nm-openvpn-service.c
index 857d539..c4da0c3 100644
--- a/src/nm-openvpn-service.c
+++ b/src/nm-openvpn-service.c
@@ -577,6 +577,7 @@ nm_find_openvpn (void)
static const char *openvpn_binary_paths[] = {
"/usr/sbin/openvpn",
"/sbin/openvpn",
+   "/usr/local/sbin/openvpn",
NULL
};
const char  **openvpn_binary = openvpn_binary_paths;

 Hope this will make life easier!
--
Juvenn Woo
___
NetworkManager-list mailing list
NetworkManager-list@gnome.org
http://mail.gnome.org/mailman/listinfo/networkmanager-list