/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 fetch.py

Share sha map cache connections inside threads.

Show diffs side-by-side

added added

removed removed

Lines of Context:
fetch.py
1
 
# Copyright (C) 2008 Canonical Ltd
 
1
# Copyright (C) 2008 Jelmer Vernooij <jelmer@samba.org>
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
14
14
# along with this program; if not, write to the Free Software
15
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
16
 
17
 
from bzrlib import osutils, ui, urlutils
18
 
from bzrlib.errors import InvalidRevisionId
19
 
from bzrlib.inventory import Inventory
20
 
from bzrlib.repository import InterRepository
21
 
from bzrlib.trace import info
22
 
from bzrlib.tsort import topo_sort
23
 
 
24
 
from bzrlib.plugins.git import git
 
17
from cStringIO import (
 
18
    StringIO,
 
19
    )
 
20
import dulwich as git
 
21
from dulwich.client import (
 
22
    SimpleFetchGraphWalker,
 
23
    )
 
24
from dulwich.objects import (
 
25
    Commit,
 
26
    Tag,
 
27
    )
 
28
from dulwich.object_store import (
 
29
    tree_lookup_path,
 
30
    )
 
31
import stat
 
32
 
 
33
from bzrlib import (
 
34
    debug,
 
35
    osutils,
 
36
    trace,
 
37
    ui,
 
38
    urlutils,
 
39
    )
 
40
from bzrlib.errors import (
 
41
    InvalidRevisionId,
 
42
    NoSuchRevision,
 
43
    )
 
44
from bzrlib.inventory import (
 
45
    Inventory,
 
46
    InventoryDirectory,
 
47
    InventoryFile,
 
48
    InventoryLink,
 
49
    )
 
50
from bzrlib.lru_cache import (
 
51
    LRUCache,
 
52
    )
 
53
from bzrlib.repository import (
 
54
    InterRepository,
 
55
    )
 
56
from bzrlib.revision import (
 
57
    NULL_REVISION,
 
58
    )
 
59
from bzrlib.tsort import (
 
60
    topo_sort,
 
61
    )
 
62
 
 
63
from bzrlib.plugins.git.converter import (
 
64
    BazaarObjectStore,
 
65
    )
 
66
from bzrlib.plugins.git.mapping import (
 
67
    DEFAULT_FILE_MODE,
 
68
    inventory_to_tree_and_blobs,
 
69
    text_to_blob,
 
70
    )
25
71
from bzrlib.plugins.git.repository import (
26
 
        LocalGitRepository, 
27
 
        GitRepository, 
28
 
        GitFormat,
29
 
        )
30
 
from bzrlib.plugins.git.remote import RemoteGitRepository
31
 
 
32
 
from dulwich.objects import Commit
33
 
 
34
 
from cStringIO import StringIO
 
72
    LocalGitRepository, 
 
73
    GitRepository, 
 
74
    GitRepositoryFormat,
 
75
    )
 
76
from bzrlib.plugins.git.remote import (
 
77
    RemoteGitRepository,
 
78
    )
35
79
 
36
80
 
37
81
class BzrFetchGraphWalker(object):
 
82
    """GraphWalker implementation that uses a Bazaar repository."""
38
83
 
39
84
    def __init__(self, repository, mapping):
40
85
        self.repository = repository
43
88
        self.heads = set(repository.all_revision_ids())
44
89
        self.parents = {}
45
90
 
 
91
    def __iter__(self):
 
92
        return iter(self.next, None)
 
93
 
46
94
    def ack(self, sha):
47
95
        revid = self.mapping.revision_id_foreign_to_bzr(sha)
48
96
        self.remove(revid)
49
97
 
50
98
    def remove(self, revid):
51
99
        self.done.add(revid)
52
 
        if ref in self.heads:
 
100
        if revid in self.heads:
53
101
            self.heads.remove(revid)
54
102
        if revid in self.parents:
55
103
            for p in self.parents[revid]:
63
111
            self.heads.update([p for p in ps if not p in self.done])
64
112
            try:
65
113
                self.done.add(ret)
66
 
                return self.mapping.revision_id_bzr_to_foreign(ret)
 
114
                return self.mapping.revision_id_bzr_to_foreign(ret)[0]
67
115
            except InvalidRevisionId:
68
116
                pass
69
117
        return None
70
118
 
71
119
 
72
 
def import_git_blob(repo, mapping, path, blob, inv, parent_invs, executable):
 
120
def import_git_blob(texts, mapping, path, hexsha, base_inv, parent_id, 
 
121
    revision_id, parent_invs, shagitmap, lookup_object, executable, symlink):
73
122
    """Import a git blob object into a bzr repository.
74
123
 
75
 
    :param repo: bzr repository
 
124
    :param texts: VersionedFiles to add to
76
125
    :param path: Path in the tree
77
126
    :param blob: A git blob
 
127
    :return: Inventory delta for this file
78
128
    """
79
129
    file_id = mapping.generate_file_id(path)
80
 
    text_revision = inv.revision_id
81
 
    repo.texts.add_lines((file_id, text_revision),
82
 
        [(file_id, p[file_id].revision) for p in parent_invs if file_id in p],
83
 
        osutils.split_lines(blob.data))
84
 
    ie = inv.add_path(path, "file", file_id)
85
 
    ie.revision = text_revision
86
 
    ie.text_size = len(blob.data)
87
 
    ie.text_sha1 = osutils.sha_string(blob.data)
 
130
    if symlink:
 
131
        cls = InventoryLink
 
132
    else:
 
133
        cls = InventoryFile
 
134
    # We just have to hope this is indeed utf-8:
 
135
    ie = cls(file_id, urlutils.basename(path).decode("utf-8"), parent_id)
88
136
    ie.executable = executable
89
 
 
90
 
 
91
 
def import_git_tree(repo, mapping, path, tree, inv, parent_invs, lookup_object):
 
137
    # See if this has changed at all
 
138
    try:
 
139
        base_sha = shagitmap.lookup_blob(file_id, base_inv.revision_id)
 
140
    except KeyError:
 
141
        base_sha = None
 
142
    else:
 
143
        if (base_sha == hexsha and base_inv[file_id].executable == ie.executable
 
144
            and base_inv[file_id].kind == ie.kind):
 
145
            # If nothing has changed since the base revision, we're done
 
146
            return [], []
 
147
    if base_sha == hexsha:
 
148
        ie.text_size = base_inv[file_id].text_size
 
149
        ie.text_sha1 = base_inv[file_id].text_sha1
 
150
        ie.symlink_target = base_inv[file_id].symlink_target
 
151
        ie.revision = base_inv[file_id].revision
 
152
    else:
 
153
        blob = lookup_object(hexsha)
 
154
        if ie.kind == "symlink":
 
155
            ie.symlink_target = blob.data
 
156
            ie.text_size = None
 
157
            ie.text_sha1 = None
 
158
        else:
 
159
            ie.text_size = len(blob.data)
 
160
            ie.text_sha1 = osutils.sha_string(blob.data)
 
161
    # Check what revision we should store
 
162
    parent_keys = []
 
163
    for pinv in parent_invs:
 
164
        if not file_id in pinv:
 
165
            continue
 
166
        if pinv[file_id].text_sha1 == ie.text_sha1:
 
167
            # found a revision in one of the parents to use
 
168
            ie.revision = pinv[file_id].revision
 
169
            break
 
170
        parent_keys.append((file_id, pinv[file_id].revision))
 
171
    if ie.revision is None:
 
172
        # Need to store a new revision
 
173
        ie.revision = revision_id
 
174
        assert file_id is not None
 
175
        assert ie.revision is not None
 
176
        texts.add_lines((file_id, ie.revision), parent_keys,
 
177
            osutils.split_lines(blob.data))
 
178
    if file_id in base_inv:
 
179
        old_path = base_inv.id2path(file_id)
 
180
    else:
 
181
        old_path = None
 
182
    return ([(old_path, path, file_id, ie)],
 
183
            [(hexsha, "blob", (ie.file_id, ie.revision))])
 
184
 
 
185
 
 
186
def import_git_tree(texts, mapping, path, hexsha, base_inv, parent_id, 
 
187
    revision_id, parent_invs, shagitmap, lookup_object):
92
188
    """Import a git tree object into a bzr repository.
93
189
 
94
 
    :param repo: A Bzr repository object
 
190
    :param texts: VersionedFiles object to add to
95
191
    :param path: Path in the tree
96
192
    :param tree: A git tree object
97
 
    :param inv: Inventory object
 
193
    :param base_inv: Base inventory against which to return inventory delta
 
194
    :return: Inventory delta for this subtree
98
195
    """
 
196
    invdelta = []
99
197
    file_id = mapping.generate_file_id(path)
100
 
    text_revision = inv.revision_id
101
 
    repo.texts.add_lines((file_id, text_revision),
102
 
        [(file_id, p[file_id].revision) for p in parent_invs if file_id in p],
103
 
        [])
104
 
    ie = inv.add_path(path, "directory", file_id)
105
 
    ie.revision = text_revision
106
 
    for mode, name, hexsha in tree.entries():
107
 
        entry_kind = (mode & 0700000) / 0100000
 
198
    # We just have to hope this is indeed utf-8:
 
199
    ie = InventoryDirectory(file_id, urlutils.basename(path.decode("utf-8")), 
 
200
        parent_id)
 
201
    if not file_id in base_inv:
 
202
        # Newly appeared here
 
203
        ie.revision = revision_id
 
204
        texts.add_lines((file_id, ie.revision), [], [])
 
205
        invdelta.append((None, path, file_id, ie))
 
206
    else:
 
207
        # See if this has changed at all
 
208
        try:
 
209
            base_sha = shagitmap.lookup_tree(file_id, base_inv.revision_id)
 
210
        except KeyError:
 
211
            pass
 
212
        else:
 
213
            if base_sha == hexsha:
 
214
                # If nothing has changed since the base revision, we're done
 
215
                return [], {}, []
 
216
    # Remember for next time
 
217
    existing_children = set()
 
218
    child_modes = {}
 
219
    shamap = []
 
220
    tree = lookup_object(hexsha)
 
221
    for mode, name, child_hexsha in tree.entries():
108
222
        basename = name.decode("utf-8")
109
 
        if path == "":
110
 
            child_path = name
111
 
        else:
112
 
            child_path = urlutils.join(path, name)
113
 
        if entry_kind == 0:
114
 
            tree = lookup_object(hexsha)
115
 
            import_git_tree(repo, mapping, child_path, tree, inv, parent_invs, lookup_object)
116
 
        elif entry_kind == 1:
117
 
            blob = lookup_object(hexsha)
118
 
            fs_mode = mode & 0777
119
 
            import_git_blob(repo, mapping, child_path, blob, inv, parent_invs, bool(fs_mode & 0111))
120
 
        else:
121
 
            raise AssertionError("Unknown blob kind, perms=%r." % (mode,))
122
 
 
123
 
 
124
 
def import_git_objects(repo, mapping, object_iter, pb=None):
 
223
        existing_children.add(basename)
 
224
        child_path = osutils.pathjoin(path, name)
 
225
        if stat.S_ISDIR(mode):
 
226
            subinvdelta, grandchildmodes, subshamap = import_git_tree(texts, 
 
227
                    mapping, child_path, child_hexsha, base_inv, file_id, 
 
228
                    revision_id, parent_invs, shagitmap, lookup_object)
 
229
            invdelta.extend(subinvdelta)
 
230
            child_modes.update(grandchildmodes)
 
231
            shamap.extend(subshamap)
 
232
        else:
 
233
            fs_mode = stat.S_IMODE(mode)
 
234
            symlink = stat.S_ISLNK(mode)
 
235
            subinvdelta, subshamap = import_git_blob(texts, mapping, 
 
236
                    child_path, child_hexsha, base_inv, file_id, revision_id, 
 
237
                    parent_invs, shagitmap, lookup_object, 
 
238
                    bool(fs_mode & 0111), symlink)
 
239
            invdelta.extend(subinvdelta)
 
240
            shamap.extend(subshamap)
 
241
        if mode not in (stat.S_IFDIR, DEFAULT_FILE_MODE,
 
242
                        stat.S_IFLNK, DEFAULT_FILE_MODE|0111):
 
243
            child_modes[child_path] = mode
 
244
    # Remove any children that have disappeared
 
245
    if file_id in base_inv:
 
246
        deletable = [v for k,v in base_inv[file_id].children.iteritems() if k not in existing_children]
 
247
        while deletable:
 
248
            ie = deletable.pop()
 
249
            invdelta.append((base_inv.id2path(ie.file_id), None, ie.file_id, None))
 
250
            if ie.kind == "directory":
 
251
                deletable.extend(ie.children.values())
 
252
    shamap.append((hexsha, "tree", (file_id, revision_id)))
 
253
    return invdelta, child_modes, shamap
 
254
 
 
255
 
 
256
def import_git_objects(repo, mapping, object_iter, target_git_object_retriever, 
 
257
        heads, pb=None):
125
258
    """Import a set of git objects into a bzr repository.
126
259
 
127
260
    :param repo: Bazaar repository
129
262
    :param object_iter: Iterator over Git objects.
130
263
    """
131
264
    # TODO: a more (memory-)efficient implementation of this
132
 
    objects = {}
133
 
    for i, o in enumerate(object_iter):
134
 
        if pb is not None:
135
 
            pb.update("fetching objects", i) 
136
 
        objects[o.id] = o
137
265
    graph = []
138
266
    root_trees = {}
139
267
    revisions = {}
 
268
    checked = set()
 
269
    heads = list(heads)
 
270
    parent_invs_cache = LRUCache(50)
140
271
    # Find and convert commit objects
141
 
    for o in objects.itervalues():
 
272
    while heads:
 
273
        if pb is not None:
 
274
            pb.update("finding revisions to fetch", len(graph), None)
 
275
        head = heads.pop()
 
276
        assert isinstance(head, str)
 
277
        try:
 
278
            o = object_iter[head]
 
279
        except KeyError:
 
280
            continue
142
281
        if isinstance(o, Commit):
143
282
            rev = mapping.import_commit(o)
144
 
            root_trees[rev.revision_id] = objects[o.tree]
 
283
            if repo.has_revision(rev.revision_id):
 
284
                continue
 
285
            root_trees[rev.revision_id] = o.tree
145
286
            revisions[rev.revision_id] = rev
146
287
            graph.append((rev.revision_id, rev.parent_ids))
 
288
            target_git_object_retriever._idmap.add_entry(o.sha().hexdigest(),
 
289
                "commit", (rev.revision_id, o.tree))
 
290
            heads.extend([p for p in o.parents if p not in checked])
 
291
        elif isinstance(o, Tag):
 
292
            heads.append(o.object[1])
 
293
        else:
 
294
            trace.warning("Unable to import head object %r" % o)
 
295
        checked.add(head)
147
296
    # Order the revisions
148
297
    # Create the inventory objects
149
298
    for i, revid in enumerate(topo_sort(graph)):
150
299
        if pb is not None:
151
300
            pb.update("fetching revisions", i, len(graph))
152
 
        root_tree = root_trees[revid]
153
301
        rev = revisions[revid]
154
302
        # We have to do this here, since we have to walk the tree and 
155
 
        # we need to make sure to import the blobs / trees with the riht 
 
303
        # we need to make sure to import the blobs / trees with the right 
156
304
        # path; this may involve adding them more than once.
157
 
        inv = Inventory()
158
 
        inv.revision_id = rev.revision_id
159
305
        def lookup_object(sha):
160
 
            if sha in objects:
161
 
                return objects[sha]
162
 
            return reconstruct_git_object(repo, mapping, sha)
163
 
        parent_invs = [repo.get_inventory(r) for r in rev.parent_ids]
164
 
        import_git_tree(repo, mapping, "", root_tree, inv, parent_invs, lookup_object)
165
 
        repo.add_revision(rev.revision_id, rev, inv)
166
 
 
167
 
 
168
 
def reconstruct_git_commit(repo, rev):
169
 
    raise NotImplementedError(self.reconstruct_git_commit)
170
 
 
171
 
 
172
 
def reconstruct_git_object(repo, mapping, sha):
173
 
    # Commit
174
 
    revid = mapping.revision_id_foreign_to_bzr(sha)
175
 
    try:
176
 
        rev = repo.get_revision(revid)
177
 
    except NoSuchRevision:
178
 
        pass
179
 
    else:
180
 
        return reconstruct_git_commit(rev)
181
 
 
182
 
    # TODO: Tree
183
 
    # TODO: Blob
184
 
    raise KeyError("No such object %s" % sha)
185
 
 
186
 
 
187
 
class InterGitRepository(InterRepository):
188
 
 
189
 
    _matching_repo_format = GitFormat()
 
306
            try:
 
307
                return object_iter[sha]
 
308
            except KeyError:
 
309
                return target_git_object_retriever[sha]
 
310
        parent_invs = []
 
311
        for parent_id in rev.parent_ids:
 
312
            try:
 
313
                parent_invs.append(parent_invs_cache[parent_id])
 
314
            except KeyError:
 
315
                parent_inv = repo.get_inventory(parent_id)
 
316
                parent_invs.append(parent_inv)
 
317
                parent_invs_cache[parent_id] = parent_inv
 
318
        if parent_invs == []:
 
319
            base_inv = Inventory(root_id=None)
 
320
        else:
 
321
            base_inv = parent_invs[0]
 
322
        inv_delta, unusual_modes, shamap = import_git_tree(repo.texts, 
 
323
                mapping, "", root_trees[revid], base_inv, None, revid, 
 
324
                parent_invs, target_git_object_retriever._idmap, lookup_object)
 
325
        target_git_object_retriever._idmap.add_entries(shamap)
 
326
        if unusual_modes != {}:
 
327
            ret = "unusual modes: \n"
 
328
            for item in unusual_modes.iteritems():
 
329
                ret += "\t%s: %o\n" % item
 
330
            raise AssertionError(ret)
 
331
        try:
 
332
            basis_id = rev.parent_ids[0]
 
333
        except IndexError:
 
334
            basis_id = NULL_REVISION
 
335
        rev.inventory_sha1, inv = repo.add_inventory_by_delta(basis_id,
 
336
                  inv_delta, rev.revision_id, rev.parent_ids)
 
337
        parent_invs_cache[rev.revision_id] = inv
 
338
        repo.add_revision(rev.revision_id, rev)
 
339
        if "verify" in debug.debug_flags:
 
340
            objs = inventory_to_tree_and_blobs(inv, repo.texts, mapping)
 
341
            for sha1, newobj, path in objs:
 
342
                assert path is not None
 
343
                oldobj = tree_lookup_path(lookup_object, root_trees[revid], path)
 
344
                assert oldobj == newobj, "%r != %r in %s" % (oldobj, newobj, path)
 
345
 
 
346
    target_git_object_retriever._idmap.commit()
 
347
 
 
348
 
 
349
class InterGitNonGitRepository(InterRepository):
 
350
    """Base InterRepository that copies revisions from a Git into a non-Git 
 
351
    repository."""
 
352
 
 
353
    _matching_repo_format = GitRepositoryFormat()
190
354
 
191
355
    @staticmethod
192
356
    def _get_repo_format_to_test():
196
360
        """See InterRepository.copy_content."""
197
361
        self.fetch(revision_id, pb, find_ghosts=False)
198
362
 
199
 
    def fetch(self, revision_id=None, pb=None, find_ghosts=False, 
200
 
              mapping=None):
 
363
    def fetch(self, revision_id=None, pb=None, find_ghosts=False, mapping=None,
 
364
            fetch_spec=None):
 
365
        self.fetch_refs(revision_id=revision_id, pb=pb, find_ghosts=find_ghosts,
 
366
                mapping=mapping, fetch_spec=fetch_spec)
 
367
 
 
368
    def fetch_refs(self, revision_id=None, pb=None, find_ghosts=False, 
 
369
              mapping=None, fetch_spec=None):
201
370
        if mapping is None:
202
371
            mapping = self.source.get_mapping()
203
 
        def progress(text):
204
 
            pb.note("git: %s", text)
205
 
        def determine_wants(heads):
206
 
            if revision_id is None:
207
 
                ret = heads.values()
 
372
        if revision_id is not None:
 
373
            interesting_heads = [revision_id]
 
374
        elif fetch_spec is not None:
 
375
            interesting_heads = fetch_spec.heads
 
376
        else:
 
377
            interesting_heads = None
 
378
        self._refs = {}
 
379
        def determine_wants(refs):
 
380
            self._refs = refs
 
381
            if interesting_heads is None:
 
382
                ret = [sha for (ref, sha) in refs.iteritems() if not ref.endswith("^{}")]
208
383
            else:
209
 
                ret = [mapping.revision_id_bzr_to_foreign(revision_id)]
 
384
                ret = [mapping.revision_id_bzr_to_foreign(revid)[0] for revid in interesting_heads if revid != NULL_REVISION]
210
385
            return [rev for rev in ret if not self.target.has_revision(mapping.revision_id_foreign_to_bzr(rev))]
 
386
        self.fetch_objects(determine_wants, mapping, pb)
 
387
        return self._refs
 
388
 
 
389
 
 
390
 
 
391
class InterRemoteGitNonGitRepository(InterGitNonGitRepository):
 
392
    """InterRepository that copies revisions from a remote Git into a non-Git 
 
393
    repository."""
 
394
 
 
395
    def fetch_objects(self, determine_wants, mapping, pb=None):
 
396
        def progress(text):
 
397
            pb.update("git: %s" % text.rstrip("\r\n"), 0, 0)
211
398
        graph_walker = BzrFetchGraphWalker(self.target, mapping)
212
399
        create_pb = None
213
400
        if pb is None:
214
401
            create_pb = pb = ui.ui_factory.nested_progress_bar()
215
 
        try:
216
 
            self.target.lock_write()
217
 
            try:
218
 
                self.target.start_write_group()
219
 
                try:
220
 
                    import_git_objects(self.target, mapping,
221
 
                        iter(self.source.fetch_objects(determine_wants, graph_walker, 
222
 
                            progress)), pb)
223
 
                finally:
224
 
                    self.target.commit_write_group()
225
 
            finally:
226
 
                self.target.unlock()
227
 
        finally:
228
 
            if create_pb:
229
 
                create_pb.finished()
230
 
 
231
 
    @staticmethod
232
 
    def is_compatible(source, target):
233
 
        """Be compatible with GitRepository."""
234
 
        # FIXME: Also check target uses VersionedFile
 
402
        target_git_object_retriever = BazaarObjectStore(self.target, mapping)
 
403
        recorded_wants = []
 
404
 
 
405
        def record_determine_wants(heads):
 
406
            wants = determine_wants(heads)
 
407
            recorded_wants.extend(wants)
 
408
            return wants
 
409
        
 
410
        try:
 
411
            self.target.lock_write()
 
412
            try:
 
413
                self.target.start_write_group()
 
414
                try:
 
415
                    objects_iter = self.source.fetch_objects(
 
416
                                record_determine_wants, 
 
417
                                graph_walker, 
 
418
                                target_git_object_retriever.get_raw, 
 
419
                                progress)
 
420
                    import_git_objects(self.target, mapping, objects_iter, 
 
421
                            target_git_object_retriever, recorded_wants, pb)
 
422
                finally:
 
423
                    self.target.commit_write_group()
 
424
            finally:
 
425
                self.target.unlock()
 
426
        finally:
 
427
            if create_pb:
 
428
                create_pb.finished()
 
429
 
 
430
    @staticmethod
 
431
    def is_compatible(source, target):
 
432
        """Be compatible with GitRepository."""
 
433
        # FIXME: Also check target uses VersionedFile
 
434
        return (isinstance(source, RemoteGitRepository) and 
 
435
                target.supports_rich_root() and
 
436
                not isinstance(target, GitRepository))
 
437
 
 
438
 
 
439
class InterLocalGitNonGitRepository(InterGitNonGitRepository):
 
440
    """InterRepository that copies revisions from a remote Git into a non-Git 
 
441
    repository."""
 
442
 
 
443
    def fetch_objects(self, determine_wants, mapping, pb=None):
 
444
        wants = determine_wants(self.source._git.get_refs())
 
445
        create_pb = None
 
446
        if pb is None:
 
447
            create_pb = pb = ui.ui_factory.nested_progress_bar()
 
448
        target_git_object_retriever = BazaarObjectStore(self.target, mapping)
 
449
        try:
 
450
            self.target.lock_write()
 
451
            try:
 
452
                self.target.start_write_group()
 
453
                try:
 
454
                    import_git_objects(self.target, mapping, 
 
455
                            self.source._git.object_store, 
 
456
                            target_git_object_retriever, wants, pb)
 
457
                finally:
 
458
                    self.target.commit_write_group()
 
459
            finally:
 
460
                self.target.unlock()
 
461
        finally:
 
462
            if create_pb:
 
463
                create_pb.finished()
 
464
 
 
465
    @staticmethod
 
466
    def is_compatible(source, target):
 
467
        """Be compatible with GitRepository."""
 
468
        # FIXME: Also check target uses VersionedFile
 
469
        return (isinstance(source, LocalGitRepository) and 
 
470
                target.supports_rich_root() and
 
471
                not isinstance(target, GitRepository))
 
472
 
 
473
 
 
474
class InterGitRepository(InterRepository):
 
475
    """InterRepository that copies between Git repositories."""
 
476
 
 
477
    _matching_repo_format = GitRepositoryFormat()
 
478
 
 
479
    @staticmethod
 
480
    def _get_repo_format_to_test():
 
481
        return None
 
482
 
 
483
    def copy_content(self, revision_id=None, pb=None):
 
484
        """See InterRepository.copy_content."""
 
485
        self.fetch(revision_id, pb, find_ghosts=False)
 
486
 
 
487
    def fetch(self, revision_id=None, pb=None, find_ghosts=False, 
 
488
              mapping=None, fetch_spec=None):
 
489
        if mapping is None:
 
490
            mapping = self.source.get_mapping()
 
491
        def progress(text):
 
492
            trace.info("git: %s", text)
 
493
        r = self.target._git
 
494
        if revision_id is not None:
 
495
            args = [mapping.revision_id_bzr_to_foreign(revision_id)[0]]
 
496
        elif fetch_spec is not None:
 
497
            args = [mapping.revision_id_bzr_to_foreign(revid)[0] for revid in fetch_spec.heads]
 
498
        if fetch_spec is None and revision_id is None:
 
499
            determine_wants = r.object_store.determine_wants_all
 
500
        else:
 
501
            determine_wants = lambda x: [y for y in args if not y in r.object_store]
 
502
 
 
503
        graphwalker = SimpleFetchGraphWalker(r.heads().values(), r.get_parents)
 
504
        f, commit = r.object_store.add_thin_pack()
 
505
        try:
 
506
            self.source.fetch_pack(determine_wants, graphwalker, f.write, progress)
 
507
            commit()
 
508
        except:
 
509
            f.close()
 
510
            raise
 
511
 
 
512
    @staticmethod
 
513
    def is_compatible(source, target):
 
514
        """Be compatible with GitRepository."""
235
515
        return (isinstance(source, GitRepository) and 
236
 
                target.supports_rich_root())
 
516
                isinstance(target, GitRepository))