Indent big section of code before actual changes.

src/zc/buildout/buildout.py

@@ -652,159 +652,161 @@ class Buildout(DictMixin):
         installed_parts = installed_part_options['buildout']['parts']
         installed_parts = installed_parts and installed_parts.split() or []
 
-        if install_args:
-            install_parts = install_args
-            uninstall_missing = False
-        else:
-            install_parts = conf_parts
-            uninstall_missing = True
-
-        # load and initialize recipes
-        [self[part]['recipe'] for part in install_parts]
-        if not install_args:
-            install_parts = self._parts
-
-        if self._log_level < logging.DEBUG:
-            sections = list(self)
-            sections.sort()
-            print_()
-            print_('Configuration data:')
-            for section in sorted(self._data):
-                _save_options(section, self[section], sys.stdout)
-            print_()
-
-
-        # compute new part recipe signatures
-        self._compute_part_signatures(install_parts)
-
-        # uninstall parts that are no-longer used or who's configs
-        # have changed
-        for part in reversed(installed_parts):
-            if part in install_parts:
-                old_options = installed_part_options[part].copy()
-                installed_files = old_options.pop('__buildout_installed__')
-                new_options = self.get(part)
-                if old_options == new_options:
-                    # The options are the same, but are all of the
-                    # installed files still there?  If not, we should
-                    # reinstall.
-                    if not installed_files:
-                        continue
-                    for f in installed_files.split('\n'):
-                        if not os.path.exists(self._buildout_path(f)):
-                            break
-                    else:
-                        continue
-
-                # output debugging info
-                if self._logger.getEffectiveLevel() < logging.DEBUG:
-                    for k in old_options:
-                        if k not in new_options:
-                            self._logger.debug("Part %s, dropped option %s.",
-                                               part, k)
-                        elif old_options[k] != new_options[k]:
-                            self._logger.debug(
-                                "Part %s, option %s changed:\n%r != %r",
-                                part, k, new_options[k], old_options[k],
-                                )
-                    for k in new_options:
-                        if k not in old_options:
-                            self._logger.debug("Part %s, new option %s.",
-                                               part, k)
-
-            elif not uninstall_missing:
-                continue
+        try:
+            if install_args:
+                install_parts = install_args
+                uninstall_missing = False
+            else:
+                install_parts = conf_parts
+                uninstall_missing = True
+
+            # load and initialize recipes
+            [self[part]['recipe'] for part in install_parts]
+            if not install_args:
+                install_parts = self._parts
+
+            if self._log_level < logging.DEBUG:
+                sections = list(self)
+                sections.sort()
+                print_()
+                print_('Configuration data:')
+                for section in sorted(self._data):
+                    _save_options(section, self[section], sys.stdout)
+                print_()
+
+
+            # compute new part recipe signatures
+            self._compute_part_signatures(install_parts)
+
+            # uninstall parts that are no-longer used or who's configs
+            # have changed
+            for part in reversed(installed_parts):
+                if part in install_parts:
+                    old_options = installed_part_options[part].copy()
+                    installed_files = old_options.pop('__buildout_installed__')
+                    new_options = self.get(part)
+                    if old_options == new_options:
+                        # The options are the same, but are all of the
+                        # installed files still there?  If not, we should
+                        # reinstall.
+                        if not installed_files:
+                            continue
+                        for f in installed_files.split('\n'):
+                            if not os.path.exists(self._buildout_path(f)):
+                                break
+                        else:
+                            continue
+
+                    # output debugging info
+                    if self._logger.getEffectiveLevel() < logging.DEBUG:
+                        for k in old_options:
+                            if k not in new_options:
+                                self._logger.debug(
+                                    "Part %s, dropped option %s.", part, k)
+                            elif old_options[k] != new_options[k]:
+                                self._logger.debug(
+                                    "Part %s, option %s changed:\n%r != %r",
+                                    part, k, new_options[k], old_options[k])
+                        for k in new_options:
+                            if k not in old_options:
+                                self._logger.debug(
+                                    "Part %s, new option %s.", part, k)
+
+                elif not uninstall_missing:
+                    continue
 
-            self._uninstall_part(part, installed_part_options)
-            installed_parts = [p for p in installed_parts if p != part]
+                self._uninstall_part(part, installed_part_options)
+                installed_parts = [p for p in installed_parts if p != part]
 
-            if installed_exists:
-                self._update_installed(parts=' '.join(installed_parts))
+                if installed_exists:
+                    self._update_installed(parts=' '.join(installed_parts))
 
-        # Check for unused buildout options:
-        _check_for_unused_options_in_section(self, 'buildout')
+            # Check for unused buildout options:
+            _check_for_unused_options_in_section(self, 'buildout')
 
-        # install new parts
-        for part in install_parts:
-            signature = self[part].pop('__buildout_signature__')
-            saved_options = self[part].copy()
-            recipe = self[part].recipe
-            if part in installed_parts: # update
-                need_to_save_installed = False
-                __doing__ = 'Updating %s.', part
-                self._logger.info(*__doing__)
-                old_options = installed_part_options[part]
-                old_installed_files = old_options['__buildout_installed__']
+            # install new parts
+            for part in install_parts:
+                signature = self[part].pop('__buildout_signature__')
+                saved_options = self[part].copy()
+                recipe = self[part].recipe
+                if part in installed_parts: # update
+                    need_to_save_installed = False
+                    __doing__ = 'Updating %s.', part
+                    self._logger.info(*__doing__)
+                    old_options = installed_part_options[part]
+                    old_installed_files = old_options['__buildout_installed__']
 
-                try:
-                    update = recipe.update
-                except AttributeError:
-                    update = recipe.install
-                    self._logger.warning(
-                        "The recipe for %s doesn't define an update "
-                        "method. Using its install method.",
-                        part)
+                    try:
+                        update = recipe.update
+                    except AttributeError:
+                        update = recipe.install
+                        self._logger.warning(
+                            "The recipe for %s doesn't define an update "
+                            "method. Using its install method.",
+                            part)
 
-                try:
-                    installed_files = self[part]._call(update)
-                except:
-                    installed_parts.remove(part)
-                    self._uninstall(old_installed_files)
-                    if installed_exists:
-                        self._update_installed(
-                            parts=' '.join(installed_parts))
-                    raise
-
-                old_installed_files = old_installed_files.split('\n')
-                if installed_files is None:
-                    installed_files = old_installed_files
-                else:
-                    if isinstance(installed_files, str):
+                    try:
+                        installed_files = self[part]._call(update)
+                    except:
+                        installed_parts.remove(part)
+                        self._uninstall(old_installed_files)
+                        if installed_exists:
+                            self._update_installed(
+                                parts=' '.join(installed_parts))
+                        raise
+
+                    old_installed_files = old_installed_files.split('\n')
+                    if installed_files is None:
+                        installed_files = old_installed_files
+                    else:
+                        if isinstance(installed_files, str):
+                            installed_files = [installed_files]
+                        else:
+                            installed_files = list(installed_files)
+
+                        need_to_save_installed = [
+                            p for p in installed_files
+                            if p not in old_installed_files]
+
+                        if need_to_save_installed:
+                            installed_files = (old_installed_files
+                                               + need_to_save_installed)
+
+                else: # install
+                    need_to_save_installed = True
+                    __doing__ = 'Installing %s.', part
+                    self._logger.info(*__doing__)
+                    installed_files = self[part]._call(recipe.install)
+                    if installed_files is None:
+                        self._logger.warning(
+                            "The %s install returned None.  A path or "
+                            "iterable os paths should be returned.",
+                            part)
+                        installed_files = ()
+                    elif isinstance(installed_files, str):
                         installed_files = [installed_files]
                     else:
                         installed_files = list(installed_files)
 
-                    need_to_save_installed = [
-                        p for p in installed_files
-                        if p not in old_installed_files]
-
-                    if need_to_save_installed:
-                        installed_files = (old_installed_files
-                                           + need_to_save_installed)
-
-            else: # install
-                need_to_save_installed = True
-                __doing__ = 'Installing %s.', part
-                self._logger.info(*__doing__)
-                installed_files = self[part]._call(recipe.install)
-                if installed_files is None:
-                    self._logger.warning(
-                        "The %s install returned None.  A path or "
-                        "iterable os paths should be returned.",
-                        part)
-                    installed_files = ()
-                elif isinstance(installed_files, str):
-                    installed_files = [installed_files]
-                else:
-                    installed_files = list(installed_files)
-
-            installed_part_options[part] = saved_options
-            saved_options['__buildout_installed__'
-                          ] = '\n'.join(installed_files)
-            saved_options['__buildout_signature__'] = signature
+                installed_part_options[part] = saved_options
+                saved_options['__buildout_installed__'
+                              ] = '\n'.join(installed_files)
+                saved_options['__buildout_signature__'] = signature
 
-            installed_parts = [p for p in installed_parts if p != part]
-            installed_parts.append(part)
-            _check_for_unused_options_in_section(self, part)
+                installed_parts = [p for p in installed_parts if p != part]
+                installed_parts.append(part)
+                _check_for_unused_options_in_section(self, part)
 
-            if need_to_save_installed:
-                installed_part_options['buildout']['parts'] = (
-                    ' '.join(installed_parts))
-                self._save_installed_options(installed_part_options)
-                installed_exists = True
-            else:
-                assert installed_exists
-                self._update_installed(parts=' '.join(installed_parts))
+                if need_to_save_installed:
+                    installed_part_options['buildout']['parts'] = (
+                        ' '.join(installed_parts))
+                    self._save_installed_options(installed_part_options)
+                    installed_exists = True
+                else:
+                    assert installed_exists
+                    self._update_installed(parts=' '.join(installed_parts))
+        finally:
+            pass
 
         if installed_develop_eggs:
             if not installed_exists: