/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

Cope with map for Tree objects becoming invalid.

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