Mercurial > bitten > bitten-test
changeset 580:c66190747a9c
0.6dev: In the event that loading Recipe fails, `basedir` would not yet have been defined, and removing it would fail.
| author | osimons |
|---|---|
| date | Sun, 12 Jul 2009 21:51:23 +0000 |
| parents | 4ddf60f40a1f |
| children | 2cc06425117a |
| files | bitten/slave.py |
| diffstat | 1 files changed, 2 insertions(+), 1 deletions(-) [+] |
line wrap: on
line diff
--- a/bitten/slave.py +++ b/bitten/slave.py @@ -253,6 +253,7 @@ def _execute_build(self, build_url, fileobj): build_id = build_url and int(build_url.split('/')[-1]) or 0 xml = xmlio.parse(fileobj) + basedir = '' try: recipe = Recipe(xml, os.path.join(self.work_dir, self.build_dir), self.config) @@ -271,7 +272,7 @@ if self.dry_run: self._cancel_build(build_url) finally: - if not self.keep_files: + if not self.keep_files and os.path.isdir(basedir): log.debug('Removing build directory %s' % basedir) _rmtree(basedir) if self.single_build: