shutil.copyfile(src,dst) was clobbering the file when the src and dst were

the same.   Added check to verify the two names are not the same.  Does not
check the actual files to see if there is a symbolic link.

Closes SF bug 490165 and Tzot's patch 604600.

diff --git a/Lib/shutil.py b/Lib/shutil.py
index 3076be92e167cb2edb293bcb904c96421ddc3e20..0e60870957cc6f1bce8c4abaa644cbdae5d6bc99 100644 (file)
--- a/Lib/shutil.py
+++ b/Lib/shutil.py
@@ -24,6 +24,11 @@ def copyfile(src, dst):
     """Copy data from src to dst"""
     fsrc = None
     fdst = None
+    # check for same pathname; all platforms
+    _src = os.path.normcase(os.path.abspath(src))
+    _dst = os.path.normcase(os.path.abspath(dst))
+    if _src == _dst:
+        return
     try:
         fsrc = open(src, 'rb')
         fdst = open(dst, 'wb')