arno/gns3-server
1
0
Fork 0
mirror of https://github.com/GNS3/gns3-server synced 2024-11-28 03:08:14 +00:00
Code Issues Releases Wiki Activity

Workaround for exception when using gns3server.utils.parse_version. Fixes #513.

Browse Source
This commit is contained in:
grossmj 2016-05-05 10:15:51 -06:00
parent 56c153af79
commit 16b5bb8810
1 changed files with 8 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
gns3server/modules/vmware/__init__.py
View File

@ -153,11 +153,11 @@ class VMware(BaseManager):
if player_version < 6: if player_version < 6:
raise VMwareError("Using VMware Player requires version 6 or above") raise VMwareError("Using VMware Player requires version 6 or above")
elif player_version == 6: elif player_version == 6:
yield from self.check_vmrun_version(minimum_required_version="1.13") yield from self.check_vmrun_version(minimum_required_version="1.13.0")
elif player_version == 7: elif player_version == 7:
yield from self.check_vmrun_version(minimum_required_version="1.14") yield from self.check_vmrun_version(minimum_required_version="1.14.0")
elif player_version >= 12: elif player_version >= 12:
yield from self.check_vmrun_version(minimum_required_version="1.15") yield from self.check_vmrun_version(minimum_required_version="1.15.0")
@asyncio.coroutine @asyncio.coroutine
def _check_vmware_workstation_requirements(self, ws_version): def _check_vmware_workstation_requirements(self, ws_version):
@ -175,11 +175,11 @@ class VMware(BaseManager):
if ws_version < 10: if ws_version < 10:
raise VMwareError("Using VMware Workstation requires version 10 or above") raise VMwareError("Using VMware Workstation requires version 10 or above")
elif ws_version == 10: elif ws_version == 10:
yield from self.check_vmrun_version(minimum_required_version="1.13") yield from self.check_vmrun_version(minimum_required_version="1.13.0")
elif ws_version == 11: elif ws_version == 11:
yield from self.check_vmrun_version(minimum_required_version="1.14") yield from self.check_vmrun_version(minimum_required_version="1.14.0")
elif ws_version >= 12: elif ws_version >= 12:
yield from self.check_vmrun_version(minimum_required_version="1.15") yield from self.check_vmrun_version(minimum_required_version="1.15.0")
@asyncio.coroutine @asyncio.coroutine
def check_vmware_version(self): def check_vmware_version(self):
@ -393,7 +393,7 @@ class VMware(BaseManager):
return stdout_data.decode("utf-8", errors="ignore").splitlines() return stdout_data.decode("utf-8", errors="ignore").splitlines()
@asyncio.coroutine @asyncio.coroutine
def check_vmrun_version(self, minimum_required_version="1.13"): def check_vmrun_version(self, minimum_required_version="1.13.0"):
""" """
Checks the vmrun version. Checks the vmrun version.
@ -416,9 +416,8 @@ class VMware(BaseManager):
version = None version = None
if match: if match:
version = match.group(1) version = match.group(1)
log.debug("VMware vmrun version {} detected".format(version)) log.debug("VMware vmrun version {} detected, minimum required: {}".format(version, minimum_required_version))
if parse_version(version) < parse_version(minimum_required_version): if parse_version(version) < parse_version(minimum_required_version):
raise VMwareError("VMware vmrun executable version must be >= version {}".format(minimum_required_version)) raise VMwareError("VMware vmrun executable version must be >= version {}".format(minimum_required_version))
if version is None: if version is None:
log.warning("Could not find VMware vmrun version. Output: {}".format(output)) log.warning("Could not find VMware vmrun version. Output: {}".format(output))