/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/upgrade.py

  • Committer: Robert Collins
  • Date: 2009-09-01 23:43:40 UTC
  • mto: (4634.6.14 2.0)
  • mto: This revision was merged to the branch mainline in revision 4666.
  • Revision ID: robertc@robertcollins.net-20090901234340-vc52aosyg23iaotw
revno 4639 in bzr.dev introduced a bug in the conversion logic for 'IDS'.

This bug is present in the InterDifferingSerializer code path, which is
used for local conversions (e.g. bzr upgrade on local disk, or bzr
push/pull between different formats, on local disks).

This revision is present in the 2.0rc1 release.

The bug is simple: A variable used to inform the repository where a
delta should be applied from was aliased in a for loop, leaving its
value indeterminate.

In *some* cases this will cause later consistency checks to error, but
not in all cases, as we don't do a manifest check during conversion.

No commits as yet, but getting this out is a priority.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/upgrade.py
 
1
# Copyright (C) 2005, 2008, 2009 Canonical Ltd
 
2
#
 
3
# This program is free software; you can redistribute it and/or modify
 
4
# it under the terms of the GNU General Public License as published by
 
5
# the Free Software Foundation; either version 2 of the License, or
 
6
# (at your option) any later version.
 
7
#
 
8
# This program is distributed in the hope that it will be useful,
 
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
 
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 
11
# GNU General Public License for more details.
 
12
#
 
13
# You should have received a copy of the GNU General Public License
 
14
# along with this program; if not, write to the Free Software
 
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
16
 
 
17
"""bzr upgrade logic."""
 
18
 
 
19
 
 
20
from bzrlib.bzrdir import BzrDir, BzrDirFormat, format_registry
 
21
import bzrlib.errors as errors
 
22
from bzrlib.remote import RemoteBzrDir
 
23
from bzrlib.transport import get_transport
 
24
import bzrlib.ui as ui
 
25
 
 
26
 
 
27
class Convert(object):
 
28
 
 
29
    def __init__(self, url, format=None):
 
30
        self.format = format
 
31
        self.bzrdir = BzrDir.open_unsupported(url)
 
32
        if isinstance(self.bzrdir, RemoteBzrDir):
 
33
            self.bzrdir._ensure_real()
 
34
            self.bzrdir = self.bzrdir._real_bzrdir
 
35
        if self.bzrdir.root_transport.is_readonly():
 
36
            raise errors.UpgradeReadonly
 
37
        self.transport = self.bzrdir.root_transport
 
38
        self.pb = ui.ui_factory.nested_progress_bar()
 
39
        try:
 
40
            self.convert()
 
41
        finally:
 
42
            self.pb.finished()
 
43
 
 
44
    def convert(self):
 
45
        try:
 
46
            branch = self.bzrdir.open_branch()
 
47
            if branch.bzrdir.root_transport.base != \
 
48
                self.bzrdir.root_transport.base:
 
49
                self.pb.note("This is a checkout. The branch (%s) needs to be "
 
50
                             "upgraded separately.",
 
51
                             branch.bzrdir.root_transport.base)
 
52
            del branch
 
53
        except (errors.NotBranchError, errors.IncompatibleRepositories):
 
54
            # might not be a format we can open without upgrading; see e.g.
 
55
            # https://bugs.launchpad.net/bzr/+bug/253891
 
56
            pass
 
57
        if self.format is None:
 
58
            try:
 
59
                rich_root = self.bzrdir.find_repository()._format.rich_root_data
 
60
            except errors.NoRepositoryPresent:
 
61
                rich_root = False # assume no rich roots
 
62
            if rich_root:
 
63
                format_name = "default-rich-root"
 
64
            else:
 
65
                format_name = "default"
 
66
            format = format_registry.make_bzrdir(format_name)
 
67
        else:
 
68
            format = self.format
 
69
        if not self.bzrdir.needs_format_conversion(format):
 
70
            raise errors.UpToDateFormat(self.bzrdir._format)
 
71
        if not self.bzrdir.can_convert_format():
 
72
            raise errors.BzrError("cannot upgrade from bzrdir format %s" %
 
73
                           self.bzrdir._format)
 
74
        self.bzrdir.check_conversion_target(format)
 
75
        self.pb.note('starting upgrade of %s', self.transport.base)
 
76
        self.bzrdir.backup_bzrdir()
 
77
        while self.bzrdir.needs_format_conversion(format):
 
78
            converter = self.bzrdir._format.get_converter(format)
 
79
            self.bzrdir = converter.convert(self.bzrdir, self.pb)
 
80
        self.pb.note("finished")
 
81
 
 
82
 
 
83
def upgrade(url, format=None):
 
84
    """Upgrade to format, or the default bzrdir format if not supplied."""
 
85
    Convert(url, format)