summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRichard Purdie <rpurdie@linux.intel.com>2010-08-23 13:01:05 +0100
committerRichard Purdie <rpurdie@linux.intel.com>2010-08-23 13:01:05 +0100
commit71a46a44a2f278db1ac1b309ff52b7836f3286b4 (patch)
tree8aa510d5f43c9af7bfb9a90c06823749e46ad50e
parent86c2b5670553322e83015df18d53bebbf51e1af5 (diff)
downloadopenembedded-core-71a46a44a2f278db1ac1b309ff52b7836f3286b4.tar.gz
openembedded-core-71a46a44a2f278db1ac1b309ff52b7836f3286b4.tar.bz2
openembedded-core-71a46a44a2f278db1ac1b309ff52b7836f3286b4.zip
utils.py: Fix bb.copyfile to change the permissions of the file back correctly
Signed-off-by: Richard Purdie <rpurdie@linux.intel.com>
-rw-r--r--bitbake/lib/bb/utils.py6
1 files changed, 4 insertions, 2 deletions
diff --git a/bitbake/lib/bb/utils.py b/bitbake/lib/bb/utils.py
index c8f139b192..9fc56eaa0f 100644
--- a/bitbake/lib/bb/utils.py
+++ b/bitbake/lib/bb/utils.py
@@ -703,9 +703,11 @@ def copyfile(src, dest, newmtime = None, sstat = None):
os.rename(dest + "#new", dest)
except Exception as e:
print('copyfile: copy', src, '->', dest, 'failed.', e)
- os.chmod(src, stat.S_IMODE(sstat[stat.ST_MODE]))
- os.utime(src, (sstat[stat.ST_ATIME], sstat[stat.ST_MTIME]))
return False
+ finally:
+ os.chmod(src, sstat[stat.ST_MODE])
+ os.utime(src, (sstat[stat.ST_ATIME], sstat[stat.ST_MTIME]))
+
else:
#we don't yet handle special, so we need to fall back to /bin/mv
a = getstatusoutput("/bin/cp -f " + "'" + src + "' '" + dest + "'")