Update bundled copy of distro from 1.3.0 to 1.4.0 (#55459)

* Update bundled copy of distro from 1.3.0 to 1.4.0. Fixes #55302

* ci_complete
This commit is contained in:
Matt Martz 2019-04-23 11:52:51 -05:00 committed by GitHub
parent e0f30ed0ee
commit 8f2976dcee
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 45 additions and 24 deletions

View file

@ -0,0 +1,2 @@
minor_changes:
- distro - Update bundled copy of distro from 1.3.0 to 1.4.0 (https://github.com/ansible/ansible/issues/55302)

View file

@ -21,12 +21,12 @@ The ``distro`` package (``distro`` stands for Linux Distribution) provides
information about the Linux distribution it runs on, such as a reliable information about the Linux distribution it runs on, such as a reliable
machine-readable distro ID, or version information. machine-readable distro ID, or version information.
It is a renewed alternative implementation for Python's original It is the recommended replacement for Python's original
:py:func:`platform.linux_distribution` function, but it provides much more :py:func:`platform.linux_distribution` function, but it provides much more
functionality. An alternative implementation became necessary because Python functionality. An alternative implementation became necessary because Python
3.5 deprecated this function, and Python 3.7 is expected to remove it 3.5 deprecated this function, and Python 3.8 will remove it altogether.
altogether. Its predecessor function :py:func:`platform.dist` was already Its predecessor function :py:func:`platform.dist` was already
deprecated since Python 2.6 and is also expected to be removed in Python 3.7. deprecated since Python 2.6 and will also be removed in Python 3.8.
Still, there are many cases in which access to OS distribution information Still, there are many cases in which access to OS distribution information
is needed. See `Python issue 1322 <https://bugs.python.org/issue1322>`_ for is needed. See `Python issue 1322 <https://bugs.python.org/issue1322>`_ for
more information. more information.
@ -52,7 +52,9 @@ _OS_RELEASE_BASENAME = 'os-release'
#: with blanks translated to underscores. #: with blanks translated to underscores.
#: #:
#: * Value: Normalized value. #: * Value: Normalized value.
NORMALIZED_OS_ID = {} NORMALIZED_OS_ID = {
'ol': 'oracle', # Oracle Enterprise Linux
}
#: Translation table for normalizing the "Distributor ID" attribute returned by #: Translation table for normalizing the "Distributor ID" attribute returned by
#: the lsb_release command, for use by the :func:`distro.id` method. #: the lsb_release command, for use by the :func:`distro.id` method.
@ -866,8 +868,12 @@ class LinuxDistribution(object):
For details, see :func:`distro.codename`. For details, see :func:`distro.codename`.
""" """
return self.os_release_attr('codename') \ try:
or self.lsb_release_attr('codename') \ # Handle os_release specially since distros might purposefully set
# this to empty string to have no codename
return self._os_release_info['codename']
except KeyError:
return self.lsb_release_attr('codename') \
or self.distro_release_attr('codename') \ or self.distro_release_attr('codename') \
or '' or ''
@ -926,6 +932,7 @@ class LinuxDistribution(object):
For details, see :func:`distro.uname_info`. For details, see :func:`distro.uname_info`.
""" """
return self._uname_info
def os_release_attr(self, attribute): def os_release_attr(self, attribute):
""" """
@ -1017,11 +1024,22 @@ class LinuxDistribution(object):
if isinstance(v, bytes): if isinstance(v, bytes):
v = v.decode('utf-8') v = v.decode('utf-8')
props[k.lower()] = v props[k.lower()] = v
if k == 'VERSION': else:
# this handles cases in which the codename is in # Ignore any tokens that are not variable assignments
# the `(CODENAME)` (rhel, centos, fedora) format pass
# or in the `, CODENAME` format (Ubuntu).
codename = re.search(r'(\(\D+\))|,(\s+)?\D+', v) if 'version_codename' in props:
# os-release added a version_codename field. Use that in
# preference to anything else Note that some distros purposefully
# do not have code names. They should be setting
# version_codename=""
props['codename'] = props['version_codename']
elif 'ubuntu_codename' in props:
# Same as above but a non-standard field name used on older Ubuntus
props['codename'] = props['ubuntu_codename']
elif 'version' in props:
# If there is no version_codename, parse it from the version
codename = re.search(r'(\(\D+\))|,(\s+)?\D+', props['version'])
if codename: if codename:
codename = codename.group() codename = codename.group()
codename = codename.strip('()') codename = codename.strip('()')
@ -1029,11 +1047,7 @@ class LinuxDistribution(object):
codename = codename.strip() codename = codename.strip()
# codename appears within paranthese. # codename appears within paranthese.
props['codename'] = codename props['codename'] = codename
else:
props['codename'] = ''
else:
# Ignore any tokens that are not variable assignments
pass
return props return props
@cached_property @cached_property
@ -1126,7 +1140,10 @@ class LinuxDistribution(object):
# file), because we want to use what was specified as best as # file), because we want to use what was specified as best as
# possible. # possible.
match = _DISTRO_RELEASE_BASENAME_PATTERN.match(basename) match = _DISTRO_RELEASE_BASENAME_PATTERN.match(basename)
if match: if 'name' in distro_info \
and 'cloudlinux' in distro_info['name'].lower():
distro_info['id'] = 'cloudlinux'
elif match:
distro_info['id'] = match.group(1) distro_info['id'] = match.group(1)
return distro_info return distro_info
else: else:
@ -1167,6 +1184,8 @@ class LinuxDistribution(object):
# The name is always present if the pattern matches # The name is always present if the pattern matches
self.distro_release_file = filepath self.distro_release_file = filepath
distro_info['id'] = match.group(1) distro_info['id'] = match.group(1)
if 'cloudlinux' in distro_info['name'].lower():
distro_info['id'] = 'cloudlinux'
return distro_info return distro_info
return {} return {}