diff options
| author | Dagobert Michelsen <dam@opencsw.org> | 2016-08-09 14:47:00 +0200 |
|---|---|---|
| committer | Brian Coca <bcoca@users.noreply.github.com> | 2016-08-09 08:47:00 -0400 |
| commit | 93472c3c542a68f5be45d4c2381a9d8e82f2f04e (patch) | |
| tree | 76e37ec0a4aa3785abce9924370ad338223bcfdf /packaging/os | |
| parent | e8b29dd75065188b538f03629d8dd85034c3a00b (diff) | |
| download | ansible-modules-extras-93472c3c542a68f5be45d4c2381a9d8e82f2f04e.tar.gz | |
Do not return failure when the package is installed and nothing is done (#1852)
Diffstat (limited to 'packaging/os')
| -rw-r--r-- | packaging/os/svr4pkg.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/packaging/os/svr4pkg.py b/packaging/os/svr4pkg.py index 5d8bac17..807e00f5 100644 --- a/packaging/os/svr4pkg.py +++ b/packaging/os/svr4pkg.py @@ -225,9 +225,10 @@ def main(): else: result['changed'] = False + # rc will be none when the package already was installed and no action took place # Only return failed=False when the returncode is known to be good as there may be more # undocumented failure return codes - if rc not in (0, 2, 10, 20): + if rc not in (None, 0, 2, 10, 20): result['failed'] = True else: result['failed'] = False |