mirror of
https://github.com/ansible-collections/community.general.git
synced 2024-09-14 20:13:21 +02:00
Merge pull request #895 from lorin/pip-fail
Return both stdout and stderr on pip failures.
This commit is contained in:
commit
4e69818427
1 changed files with 10 additions and 4 deletions
14
library/pip
14
library/pip
|
@ -154,15 +154,15 @@ def main():
|
||||||
changed = 'Successfully installed' in out_pip
|
changed = 'Successfully installed' in out_pip
|
||||||
|
|
||||||
elif name:
|
elif name:
|
||||||
|
|
||||||
installed = _is_package_installed(name, pip, version)
|
installed = _is_package_installed(name, pip, version)
|
||||||
changed = ((installed and state == 'absent') or
|
changed = ((installed and state == 'absent') or
|
||||||
(not installed and state == 'present'))
|
(not installed and state == 'present'))
|
||||||
|
|
||||||
if changed:
|
if changed:
|
||||||
if state == 'present':
|
if state == 'present':
|
||||||
full_name = _get_full_name(name, version)
|
full_name = _get_full_name(name, version)
|
||||||
else:
|
else:
|
||||||
full_name = name
|
full_name = name
|
||||||
|
|
||||||
cmd = '%s %s %s' % (pip, command_map[state], full_name)
|
cmd = '%s %s %s' % (pip, command_map[state], full_name)
|
||||||
|
@ -178,7 +178,13 @@ def main():
|
||||||
err += err_pip
|
err += err_pip
|
||||||
|
|
||||||
if rc != 0:
|
if rc != 0:
|
||||||
module.fail_json(msg=err, cmd=cmd)
|
if not out:
|
||||||
|
msg = err
|
||||||
|
elif not err:
|
||||||
|
msg = out
|
||||||
|
else:
|
||||||
|
msg = "stdout: %s\n:stderr: %s" % (out, err)
|
||||||
|
module.fail_json(msg=msg, cmd=cmd)
|
||||||
|
|
||||||
module.exit_json(changed=changed, cmd=cmd, name=name, version=version,
|
module.exit_json(changed=changed, cmd=cmd, name=name, version=version,
|
||||||
state=state, requirements=requirements, virtualenv=env)
|
state=state, requirements=requirements, virtualenv=env)
|
||||||
|
|
Loading…
Reference in a new issue