Merge pull request #764 from willthames/fail_copy_on_directory

Revert acceptance of directory as dest for copy
This commit is contained in:
Michael DeHaan 2012-08-02 04:49:17 -07:00
commit 9e0bf168ca

View file

@ -49,7 +49,7 @@ def main():
module.fail_json(msg="Destination %s not readable" % (dest)) module.fail_json(msg="Destination %s not readable" % (dest))
# Allow dest to be directory without compromising md5 check # Allow dest to be directory without compromising md5 check
if (os.path.isdir(dest)): if (os.path.isdir(dest)):
dest = os.join(dest, os.path.basename(src)) module.fail_json(msg="Destination %s cannot be a directory" % (dest))
md5sum_dest = module.md5(dest) md5sum_dest = module.md5(dest)
else: else:
if not os.access(os.path.dirname(dest), os.W_OK): if not os.access(os.path.dirname(dest), os.W_OK):