From e39e8ba308364f16e3b74db96b15415ab97b5f52 Mon Sep 17 00:00:00 2001 From: James Cammarata Date: Wed, 30 Dec 2015 13:49:39 -0500 Subject: [PATCH] Fix logic mistake in unarchive action plugin --- lib/ansible/plugins/action/unarchive.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/lib/ansible/plugins/action/unarchive.py b/lib/ansible/plugins/action/unarchive.py index cd89b936fe..b6c43a3c59 100644 --- a/lib/ansible/plugins/action/unarchive.py +++ b/lib/ansible/plugins/action/unarchive.py @@ -69,14 +69,14 @@ class ActionModule(ActionBase): source = self._loader.path_dwim_relative(self._loader.get_basedir(), 'files', source) remote_checksum = self._remote_checksum(dest, all_vars=task_vars) - if remote_checksum != '3': - result['failed'] = True - result['msg'] = "dest '%s' must be an existing dir" % dest - return result - elif remote_checksum == '4': + if remote_checksum == '4': result['failed'] = True result['msg'] = "python isn't present on the system. Unable to compute checksum" return result + elif remote_checksum != '3': + result['failed'] = True + result['msg'] = "dest '%s' must be an existing dir" % dest + return result if copy: # transfer the file to a remote tmp location