summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKali Kaneko <kali@leap.se>2017-12-08 17:58:28 +0100
committerKali Kaneko <kali@leap.se>2017-12-08 18:12:03 +0100
commit4be4b989f22355cc548f7afad76503b3394b5744 (patch)
tree9f58dd451bfbe0dc4149bfe6fa0a3fb9ea390729
parent13af5d9f4abc6a0dd5aef91e076ed2bb90779d4f (diff)
[docs] add comment about python interpreter
-rw-r--r--src/leap/bitmask/vpn/launchers/linux.py5
1 files changed, 5 insertions, 0 deletions
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)