Merge pull request #733 from willthames/copy_to_directory

Allow copy dest argument to be a directory
2012-07-31 05:03:50 -07:00 · 2012-07-31 05:03:50 -07:00 · 89ef236e9c
commit 89ef236e9c
parent e6d898cd17 3cb51acd78
1 changed files with 3 additions and 0 deletions
--- a/library/copy
+++ b/library/copy
@ -45,6 +45,9 @@ def main():
            module.fail_json(msg="Destination %s not writable" % (dest))
        if not os.access(dest, os.R_OK):
            module.fail_json(msg="Destination %s not readable" % (dest))
+        # Allow dest to be directory without compromising md5 check
+        if (os.path.isdir(dest)):
+            dest = os.join(dest, os.path.basename(src))
        md5sum_dest = module.md5(dest)
    else:
        if not os.access(os.path.dirname(dest), os.W_OK):