diff options
author | Kali Kaneko <kali@leap.se> | 2017-12-08 17:58:28 +0100 |
---|---|---|
committer | Kali Kaneko <kali@leap.se> | 2017-12-08 18:12:03 +0100 |
commit | 4be4b989f22355cc548f7afad76503b3394b5744 (patch) | |
tree | 9f58dd451bfbe0dc4149bfe6fa0a3fb9ea390729 /src/leap/bitmask/vpn/launchers/linux.py | |
parent | 13af5d9f4abc6a0dd5aef91e076ed2bb90779d4f (diff) |
[docs] add comment about python interpreter
Diffstat (limited to 'src/leap/bitmask/vpn/launchers/linux.py')
-rw-r--r-- | src/leap/bitmask/vpn/launchers/linux.py | 5 |
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) |