/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 workingtree.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:
workingtree.py
1
 
# Copyright (C) 2008 Jelmer Vernooij <jelmer@samba.org>
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., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
 
 
17
 
"""An adapter between a Git index and a Bazaar Working Tree"""
18
 
 
19
 
import os
20
 
 
21
 
from bzrlib import (
22
 
    inventory,
23
 
    lockable_files,
24
 
    lockdir,
25
 
    transport,
26
 
    urlutils,
27
 
    workingtree,
28
 
    )
29
 
 
30
 
from dulwich.index import Index
31
 
 
32
 
class GitWorkingTree(workingtree.WorkingTree):
33
 
    """A Git working tree."""
34
 
 
35
 
    def __init__(self, bzrdir, repo, branch):
36
 
        self.basedir = bzrdir.transport.base
37
 
        self.bzrdir = bzrdir
38
 
        self.repository = repo
39
 
        self._branch = branch
40
 
        self._transport = bzrdir.transport
41
 
 
42
 
        self.controldir = urlutils.join(self.repository._git.path, 'bzr')
43
 
 
44
 
        try:
45
 
            os.makedirs(self.controldir)
46
 
            os.makedirs(os.path.join(self.controldir, 'lock'))
47
 
        except OSError:
48
 
            pass
49
 
 
50
 
        self._control_files = lockable_files.LockableFiles(
51
 
            transport.get_transport(self.controldir), 'lock', lockdir.LockDir)
52
 
 
53
 
        self._format = GitWorkingTreeFormat()
54
 
 
55
 
        self.index = Index(os.path.join(self.repository._git.controldir(), 
56
 
            "index"))
57
 
 
58
 
    def unlock(self):
59
 
        # non-implementation specific cleanup
60
 
        self._cleanup()
61
 
 
62
 
        # reverse order of locking.
63
 
        try:
64
 
            return self._control_files.unlock()
65
 
        finally:
66
 
            self.branch.unlock()
67
 
 
68
 
    def is_control_filename(self, path):
69
 
        return os.path.basename(path) == ".git"
70
 
 
71
 
    def _get_inventory(self):
72
 
        return inventory.Inventory()
73
 
 
74
 
    inventory = property(_get_inventory,
75
 
                         doc="Inventory of this Tree")
76
 
 
77
 
 
78
 
class GitWorkingTreeFormat(workingtree.WorkingTreeFormat):
79
 
 
80
 
    def get_format_description(self):
81
 
        return "Git Working Tree"