diff options
author | George Kraft <george.kraft@calxeda.com> | 2013-11-12 03:46:21 -0600 |
---|---|---|
committer | George Kraft <george.kraft@calxeda.com> | 2013-11-12 03:46:21 -0600 |
commit | 082f03ee0d47bab30e9185866008da24ca8048b2 (patch) | |
tree | 7ecb43573155d37b93bebddc42db6ebccde7089d | |
parent | b6bbd565b2f4672ed0a71dc9972635535bfb68f5 (diff) | |
download | cxmanage-082f03ee0d47bab30e9185866008da24ca8048b2.tar.gz |
CXMAN-254: Remove pkg_resources.require usage from firmware_package
-rw-r--r-- | cxmanage_api/firmware_package.py | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/cxmanage_api/firmware_package.py b/cxmanage_api/firmware_package.py index 2822fd2..7f8e645 100644 --- a/cxmanage_api/firmware_package.py +++ b/cxmanage_api/firmware_package.py @@ -37,6 +37,7 @@ import tarfile import ConfigParser import pkg_resources +import cxmanage_api from cxmanage_api import temp_dir from cxmanage_api.image import Image @@ -81,15 +82,16 @@ class FirmwarePackage(object): % os.path.basename(filename)) if "package" in config.sections(): - cxmanage_ver = config.get("package", - "required_cxmanage_version") - try: - pkg_resources.require("cxmanage>=%s" % cxmanage_ver) - except pkg_resources.VersionConflict: + required_cxmanage_version = config.get( + "package", "required_cxmanage_version" + ) + if (pkg_resources.parse_version(cxmanage_api.__version__) < + pkg_resources.parse_version(required_cxmanage_version)): # @todo: CxmanageVersionError? raise ValueError( - "%s requires cxmanage version %s or later." - % (filename, cxmanage_ver)) + "%s requires cxmanage version %s or later." + % (filename, required_cxmanage_version) + ) if config.has_option("package", "required_socman_version"): self.required_socman_version = config.get("package", |