diff options
author | Richard Maw <richard.maw@codethink.co.uk> | 2013-07-03 15:26:30 +0000 |
---|---|---|
committer | Richard Maw <richard.maw@codethink.co.uk> | 2013-07-03 15:26:30 +0000 |
commit | e9ae7ca34ecbbab2e1da45a8b5c8bae215ca5267 (patch) | |
tree | d426393b367c5cd89997f316ab49e49d59049fe2 | |
parent | 0e5cf8fbc1a9a24f12d63d69e6cf095878b7d814 (diff) | |
download | morph-e9ae7ca34ecbbab2e1da45a8b5c8bae215ca5267.tar.gz |
Save more than one failed build
I blithely replaced runcmd(['mv', src_dir, dest_dir]) with
os.rename(src_dir, dest_dir), without realising that mv does
extra work when the target is a directory that already exists.
-rw-r--r-- | morphlib/builder2.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/morphlib/builder2.py b/morphlib/builder2.py index c1f1fe50..82aaab00 100644 --- a/morphlib/builder2.py +++ b/morphlib/builder2.py @@ -317,7 +317,7 @@ class ChunkBuilder(BuilderBase): line.rstrip('\n')) src_dir = self.staging_area.dirname dest_dir = os.path.join(self.app.settings['tempdir'], - 'failed') + 'failed', os.path.basename(src_dir)) os.rename(src_dir, dest_dir) raise self.staging_area.chroot_close() |