From b138c5e11476e46f3b84cedf3601e302c5a27630 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tom=C3=A1s=20Touceda?= Date: Wed, 10 Jul 2013 16:20:09 -0300 Subject: Make the install_path relative to the launcher path instead of absolute Also, add quotes to the up/down script lines in case the path has spaces --- changes/bug_osx_bundle | 2 ++ src/leap/services/eip/vpnlaunchers.py | 9 +++++---- 2 files changed, 7 insertions(+), 4 deletions(-) create mode 100644 changes/bug_osx_bundle diff --git a/changes/bug_osx_bundle b/changes/bug_osx_bundle new file mode 100644 index 00000000..d059468d --- /dev/null +++ b/changes/bug_osx_bundle @@ -0,0 +1,2 @@ + o OSX: Make the install_path relative to the launcher path instead + of absolute. \ No newline at end of file diff --git a/src/leap/services/eip/vpnlaunchers.py b/src/leap/services/eip/vpnlaunchers.py index a1f7e4cf..116a8790 100644 --- a/src/leap/services/eip/vpnlaunchers.py +++ b/src/leap/services/eip/vpnlaunchers.py @@ -472,8 +472,8 @@ class DarwinVPNLauncher(VPNLauncher): INSTALL_MSG = ("\"LEAP needs administrative privileges to install " "missing scripts and fix permissions.\"") - INSTALL_PATH = "/Applications/LEAP Client.app" - INSTALL_PATH_ESCAPED = "/Applications/LEAP\ Client.app" + INSTALL_PATH = os.path.realpath(os.getcwd() + "/../../") + INSTALL_PATH_ESCAPED = os.path.realpath(os.getcwd() + "/../../") OPENVPN_BIN = 'openvpn.leap' OPENVPN_PATH = "%s/Contents/Resources/openvpn" % (INSTALL_PATH,) OPENVPN_PATH_ESCAPED = "%s/Contents/Resources/openvpn" % ( @@ -620,12 +620,13 @@ class DarwinVPNLauncher(VPNLauncher): if _has_updown_scripts(self.UP_SCRIPT): args += [ - '--up', self.UP_SCRIPT, + '--up', '\"%s\"' % (self.UP_SCRIPT,), ] if _has_updown_scripts(self.DOWN_SCRIPT): args += [ - '--down', self.DOWN_SCRIPT] + '--down', '\"%s\"' % (self.DOWN_SCRIPT,) + ] # should have the down script too if _has_updown_scripts(self.OPENVPN_DOWN_PLUGIN): -- cgit v1.2.3