URL: https://github.com/freeipa/freeipa/pull/254 Author: tiran Title: #254: Replace LooseVersion with pkg_resources.parse_version Action: opened
PR body: """ pylint is having a hard time with distutils.version in tox's virtual envs. virtualenv uses some tricks to provide a virtual distutils package, pylint can't cope with. https://github.com/PyCQA/pylint/issues/73 suggests to use pkg_resources instead. pkg_resources' version parser has some more benefits, e.g. PEP 440 conformity. Signed-off-by: Christian Heimes <chei...@redhat.com> """ To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/254/head:pr254 git checkout pr254
From 657f889a64c637444e67008138bdb22f9b078499 Mon Sep 17 00:00:00 2001 From: Christian Heimes <chei...@redhat.com> Date: Fri, 18 Nov 2016 09:26:18 +0100 Subject: [PATCH] Replace LooseVersion with pkg_resources.parse_version pylint is having a hard time with distutils.version in tox's virtual envs. virtualenv uses some tricks to provide a virtual distutils package, pylint can't cope with. https://github.com/PyCQA/pylint/issues/73 suggests to use pkg_resources instead. pkg_resources' version parser has some more benefits, e.g. PEP 440 conformity. Signed-off-by: Christian Heimes <chei...@redhat.com> --- ipaclient/remote_plugins/compat.py | 10 +++++----- ipalib/capabilities.py | 6 +++--- ipalib/frontend.py | 8 +++----- ipalib/plugable.py | 7 +++---- ipaserver/install/krbinstance.py | 7 ++++--- ipaserver/install/server/replicainstall.py | 4 ++-- 6 files changed, 20 insertions(+), 22 deletions(-) diff --git a/ipaclient/remote_plugins/compat.py b/ipaclient/remote_plugins/compat.py index 984eecd..8635079 100644 --- a/ipaclient/remote_plugins/compat.py +++ b/ipaclient/remote_plugins/compat.py @@ -2,12 +2,12 @@ # Copyright (C) 2016 FreeIPA Contributors see COPYING for license # -from distutils.version import LooseVersion import importlib import os import re import sys +from pkg_resources import parse_version import six from ipaclient.frontend import ClientCommand, ClientMethod @@ -58,7 +58,7 @@ def get_package(server_info, client): server_info['version'] = server_version server_info.update_validity() - server_version = LooseVersion(server_version) + server_version = parse_version(server_version) package_names = {} base_name = __name__.rpartition('.')[0] @@ -70,11 +70,11 @@ def get_package(server_info, client): package_names[package_version] = '{}.{}'.format(base_name, name) package_version = None - for version in sorted(package_names, key=LooseVersion): + for version in sorted(package_names, key=parse_version): if (package_version is None or - LooseVersion(package_version) < LooseVersion(version)): + parse_version(package_version) < parse_version(version)): package_version = version - if LooseVersion(version) >= server_version: + if parse_version(version) >= server_version: break package_name = package_names[package_version] diff --git a/ipalib/capabilities.py b/ipalib/capabilities.py index 7ddaea2..a01faeb 100644 --- a/ipalib/capabilities.py +++ b/ipalib/capabilities.py @@ -25,7 +25,7 @@ versions they were introduced in. """ -from distutils import version +from pkg_resources import parse_version VERSION_WITHOUT_CAPABILITIES = u'2.51' @@ -64,6 +64,6 @@ def client_has_capability(client_version, capability): :param client_version: The API version string reported by the client """ - version_tuple = version.LooseVersion(client_version) + version_tuple = parse_version(client_version) - return version_tuple >= version.LooseVersion(capabilities[capability]) + return version_tuple >= parse_version(capabilities[capability]) diff --git a/ipalib/frontend.py b/ipalib/frontend.py index c94d174..d5890b0 100644 --- a/ipalib/frontend.py +++ b/ipalib/frontend.py @@ -20,9 +20,7 @@ """ Base classes for all front-end plugins. """ - -from distutils import version - +from pkg_resources import parse_version import six from ipapython.version import API_VERSION @@ -770,8 +768,8 @@ def verify_client_version(self, client_version): If the client minor version is less than or equal to the server then let the request proceed. """ - server_ver = version.LooseVersion(API_VERSION) - ver = version.LooseVersion(client_version) + server_ver = parse_version(API_VERSION) + ver = parse_version(client_version) if len(ver.version) < 2: raise VersionError(cver=ver.version, sver=server_ver.version, server= self.env.xmlrpc_uri) client_major = ver.version[0] diff --git a/ipalib/plugable.py b/ipalib/plugable.py index 76fb9fd..5d3ccfe 100644 --- a/ipalib/plugable.py +++ b/ipalib/plugable.py @@ -24,8 +24,6 @@ you are unfamiliar with this Python feature, see http://docs.python.org/ref/sequence-types.html """ - -from distutils.version import LooseVersion import operator import sys import threading @@ -36,6 +34,7 @@ import collections import importlib +from pkg_resources import parse_version import six from ipalib import errors @@ -725,8 +724,8 @@ def finalize(self): except KeyError: pass else: - version = LooseVersion(plugin.version) - default_version = LooseVersion(default_version) + version = parse_version(plugin.version) + default_version = parse_version(default_version) if version < default_version: continue self.__default_map[plugin.name] = plugin.version diff --git a/ipaserver/install/krbinstance.py b/ipaserver/install/krbinstance.py index b5cfd79..4fe4471 100644 --- a/ipaserver/install/krbinstance.py +++ b/ipaserver/install/krbinstance.py @@ -24,6 +24,8 @@ import os import pwd import socket + +from pkg_resources import parse_version import dns.name from ipaserver.install import service @@ -38,7 +40,6 @@ from ipaserver.install import ldapupdate from ipaserver.install import certs -from distutils import version from ipaplatform.constants import constants from ipaplatform.tasks import tasks from ipaplatform.paths import paths @@ -291,8 +292,8 @@ def __configure_instance(self): raiseonerr=False, capture_output=True) if result.returncode == 0: verstr = result.output.split()[-1] - ver = version.LooseVersion(verstr) - min = version.LooseVersion(MIN_KRB5KDC_WITH_WORKERS) + ver = parse_version(verstr) + min = parse_version(MIN_KRB5KDC_WITH_WORKERS) if ver >= min: workers = True # Write down config file diff --git a/ipaserver/install/server/replicainstall.py b/ipaserver/install/server/replicainstall.py index a7b333c..52f236e 100644 --- a/ipaserver/install/server/replicainstall.py +++ b/ipaserver/install/server/replicainstall.py @@ -4,7 +4,6 @@ from __future__ import print_function -from distutils.version import LooseVersion import dns.exception as dnsexception import dns.name as dnsname import dns.resolver as dnsresolver @@ -15,6 +14,7 @@ import tempfile import traceback +from pkg_resources import parse_version import six from ipapython import ipaldap, ipautil, sysrestore @@ -512,7 +512,7 @@ def check_remote_version(api): remote_version = env['version'] version = api.env.version - if LooseVersion(remote_version) > LooseVersion(version): + if parse_version(remote_version) > parse_version(version): raise RuntimeError( "Cannot install replica of a server of higher version ({}) than" "the local version ({})".format(remote_version, version))
-- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code
