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 /morphlib/builder2.py | |
| 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.
Diffstat (limited to 'morphlib/builder2.py')
| -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() |