From 3cb51acd788e093fe904973e0af2ec68c807d718 Mon Sep 17 00:00:00 2001 From: u348095 Date: Tue, 31 Jul 2012 14:40:50 +1000 Subject: [PATCH] Allow copy dest argument to be a directory Could have used shutil.copy rather than shutil.copyfile, but this implementation preserves the md5 comparison to avoid unnecessary copies --- library/copy | 3 +++ 1 file changed, 3 insertions(+) diff --git a/library/copy b/library/copy index b6021c7b69..66b5921d71 100755 --- 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):