ansible/community.general
1
0
Fork 0
mirror of https://github.com/ansible-collections/community.general.git synced 2024-09-14 20:13:21 +02:00
Code Releases Activity

gem: Fix get_installed_versions: correctly parse "default" version. (#783) (#918)

Browse source 
* Fix get_installed_versions: correctly parse "default" version.

gem query output of

    bundler (default: 2.1.4, 1.17.2)

Gets parsed as:

    ['default:', '1.17.2']

Fix this by skipping "default: " if present in the list of versions - by adding
it as an optional part of the regex, grouped as a non-capturing group to keep
the index of existing group.

This now correctly parses the above input as

    ['2.1.4:', '1.17.2']

Fixes #782

* Fix gem get_installed_versions (cont): add changelog fragment

* Update changelogs/fragments/783-fix-gem-installed-versions.yaml as per suggestion

Co-authored-by: Felix Fontein <felix@fontein.de>

Co-authored-by: Felix Fontein <felix@fontein.de>
(cherry picked from commit 6dc98c08fd)

Co-authored-by: Vlad Mencl <vladimir.mencl@reannz.co.nz>
This commit is contained in:
patchback[bot] 2020-09-17 20:01:58 +02:00 committed by GitHub
parent 11315c8c69
commit d4633cfcd5
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
changelogs/fragments/783-fix-gem-installed-versions.yaml Normal file
View file

@ -0,0 +1,2 @@
bugfixes:
- "gem - fix get_installed_versions: correctly parse ``default`` version (https://github.com/ansible-collections/community.general/pull/783)."

2
plugins/modules/packaging/language/gem.py
View file

@ -157,7 +157,7 @@ def get_installed_versions(module, remote=False):
(rc, out, err) = module.run_command(cmd, environ_update=environ, check_rc=True) (rc, out, err) = module.run_command(cmd, environ_update=environ, check_rc=True)
installed_versions = [] installed_versions = []
for line in out.splitlines(): for line in out.splitlines():
match = re.match(r"\S+\s+\((.+)\)", line) match = re.match(r"\S+\s+\((?:default: )?(.+)\)", line)
if match: if match:
versions = match.group(1) versions = match.group(1)
for version in versions.split(', '): for version in versions.split(', '):