mirror of
https://github.com/ansible-collections/community.general.git
synced 2024-09-14 20:13:21 +02:00
Fix the homebrew module failing because of warnings (#8406)
Instead of checking if there is an error message, which can also be a
warning, we now check the return code.
This commit fixes #8229 #7044
Co-authored-by: Strahinja Kustudic <strahinjak@nordeus.com>
(cherry picked from commit 43cb5a0d54
)
Co-authored-by: Strahinja Kustudic <kustodian@gmail.com>
This commit is contained in:
parent
4c6d439a13
commit
03d944be9a
2 changed files with 4 additions and 2 deletions
2
changelogs/fragments/8406-fix-homebrew-cask-warning.yaml
Normal file
2
changelogs/fragments/8406-fix-homebrew-cask-warning.yaml
Normal file
|
@ -0,0 +1,2 @@
|
||||||
|
bugfixes:
|
||||||
|
- homebrew - do not fail when brew prints warnings (https://github.com/ansible-collections/community.general/pull/8406, https://github.com/ansible-collections/community.general/issues/7044).
|
|
@ -488,9 +488,9 @@ class Homebrew(object):
|
||||||
self.current_package,
|
self.current_package,
|
||||||
]
|
]
|
||||||
rc, out, err = self.module.run_command(cmd)
|
rc, out, err = self.module.run_command(cmd)
|
||||||
if err:
|
if rc != 0:
|
||||||
self.failed = True
|
self.failed = True
|
||||||
self.message = err.strip()
|
self.message = err.strip() or ("Unknown failure with exit code %d" % rc)
|
||||||
raise HomebrewException(self.message)
|
raise HomebrewException(self.message)
|
||||||
data = json.loads(out)
|
data = json.loads(out)
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue