From 35bbe5666781d52ceb4c3190bfb35328b76a9952 Mon Sep 17 00:00:00 2001 From: "Michael W. Hudson" Date: Fri, 22 Feb 2002 13:22:55 +0000 Subject: [PATCH] backport akuchling's checkin of revision 1.12 of file_util.py [Bug #220993; may also fix bug #479469] Fix flakiness when old installations are present, by always unlinking the destination file before copying to it. Without the unlink(), the copied file remains owned by its previous UID, causing the subsequent chmod() to fail. Bugfix candidate, though it may cause changes on platforms where file ownership behaves differently. --- Lib/distutils/file_util.py | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/Lib/distutils/file_util.py b/Lib/distutils/file_util.py index 526e4cf593f5..14772fb74aad 100644 --- a/Lib/distutils/file_util.py +++ b/Lib/distutils/file_util.py @@ -36,6 +36,13 @@ def _copy_file_contents (src, dst, buffer_size=16*1024): raise DistutilsFileError, \ "could not open '%s': %s" % (src, errstr) + if os.path.exists(dst): + try: + os.unlink(dst) + except os.error, (errno, errstr): + raise DistutilsFileError, \ + "could not delete '%s': %s" % (dst, errstr) + try: fdst = open(dst, 'wb') except os.error, (errno, errstr): -- 2.47.3