From 4be4b989f22355cc548f7afad76503b3394b5744 Mon Sep 17 00:00:00 2001 From: Kali Kaneko Date: Fri, 8 Dec 2017 17:58:28 +0100 Subject: [docs] add comment about python interpreter --- src/leap/bitmask/vpn/launchers/linux.py | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'src/leap/bitmask') diff --git a/src/leap/bitmask/vpn/launchers/linux.py b/src/leap/bitmask/vpn/launchers/linux.py index b6bf278e..0e145e50 100644 --- a/src/leap/bitmask/vpn/launchers/linux.py +++ b/src/leap/bitmask/vpn/launchers/linux.py @@ -110,6 +110,11 @@ class LinuxVPNLauncher(VPNLauncher): return 'bitmask-root' def _version(self, bitmask_root): + # FIXME this, as a couple of other calls in the vpn modules, relies + # on having a python executable in the path. Even all modern + # default distros provide that, we should not rely on it. + # At least, we should be ready to do error handling if the binary + # is not found. out = subprocess.check_output(['python', bitmask_root, "version"]) return int(out) -- cgit v1.2.3