/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

Move push code to push.py.

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.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
    DEFAULT_TREE_MODE,
 
69
    DEFAULT_SYMLINK_MODE,
 
70
    inventory_to_tree_and_blobs,
 
71
    text_to_blob,
 
72
    )
24
73
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
 
74
    LocalGitRepository, 
 
75
    GitRepository, 
 
76
    GitRepositoryFormat,
 
77
    )
 
78
from bzrlib.plugins.git.remote import (
 
79
    RemoteGitRepository,
 
80
    )
37
81
 
38
82
 
39
83
class BzrFetchGraphWalker(object):
75
119
        return None
76
120
 
77
121
 
78
 
def import_git_blob(repo, mapping, path, blob, inv, parent_invs, gitmap, executable):
 
122
def import_git_blob(texts, mapping, path, hexsha, base_inv, parent_id, 
 
123
    revision_id, parent_invs, shagitmap, lookup_object, executable, symlink):
79
124
    """Import a git blob object into a bzr repository.
80
125
 
81
 
    :param repo: bzr repository
 
126
    :param texts: VersionedFiles to add to
82
127
    :param path: Path in the tree
83
128
    :param blob: A git blob
 
129
    :return: Inventory delta for this file
84
130
    """
85
131
    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)
 
132
    if symlink:
 
133
        cls = InventoryLink
 
134
    else:
 
135
        cls = InventoryFile
 
136
    # We just have to hope this is indeed utf-8:
 
137
    ie = cls(file_id, urlutils.basename(path).decode("utf-8"), parent_id)
94
138
    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):
 
139
    # See if this has changed at all
 
140
    try:
 
141
        base_sha = shagitmap.lookup_blob(file_id, base_inv.revision_id)
 
142
    except KeyError:
 
143
        base_sha = None
 
144
    else:
 
145
        if (base_sha == hexsha and base_inv[file_id].executable == ie.executable
 
146
            and base_inv[file_id].kind == ie.kind):
 
147
            # If nothing has changed since the base revision, we're done
 
148
            return []
 
149
    if base_sha == hexsha:
 
150
        ie.text_size = base_inv[file_id].text_size
 
151
        ie.text_sha1 = base_inv[file_id].text_sha1
 
152
        ie.symlink_target = base_inv[file_id].symlink_target
 
153
        ie.revision = base_inv[file_id].revision
 
154
    else:
 
155
        blob = lookup_object(hexsha)
 
156
        if ie.kind == "symlink":
 
157
            ie.symlink_target = blob.data
 
158
            ie.text_size = None
 
159
            ie.text_sha1 = None
 
160
        else:
 
161
            ie.text_size = len(blob.data)
 
162
            ie.text_sha1 = osutils.sha_string(blob.data)
 
163
    # Check what revision we should store
 
164
    parent_keys = []
 
165
    for pinv in parent_invs:
 
166
        if not file_id in pinv:
 
167
            continue
 
168
        if pinv[file_id].text_sha1 == ie.text_sha1:
 
169
            # found a revision in one of the parents to use
 
170
            ie.revision = pinv[file_id].revision
 
171
            break
 
172
        parent_keys.append((file_id, pinv[file_id].revision))
 
173
    if ie.revision is None:
 
174
        # Need to store a new revision
 
175
        ie.revision = revision_id
 
176
        assert file_id is not None
 
177
        assert ie.revision is not None
 
178
        texts.add_lines((file_id, ie.revision), parent_keys,
 
179
            osutils.split_lines(blob.data))
 
180
        if "verify" in debug.debug_flags:
 
181
            assert text_to_blob(blob.data).id == hexsha
 
182
        shagitmap.add_entry(hexsha, "blob", (ie.file_id, ie.revision))
 
183
    if file_id in base_inv:
 
184
        old_path = base_inv.id2path(file_id)
 
185
    else:
 
186
        old_path = None
 
187
    return [(old_path, path, file_id, ie)]
 
188
 
 
189
 
 
190
def import_git_tree(texts, mapping, path, hexsha, base_inv, parent_id, 
 
191
    revision_id, parent_invs, shagitmap, lookup_object):
100
192
    """Import a git tree object into a bzr repository.
101
193
 
102
 
    :param repo: A Bzr repository object
 
194
    :param texts: VersionedFiles object to add to
103
195
    :param path: Path in the tree
104
196
    :param tree: A git tree object
105
 
    :param inv: Inventory object
 
197
    :param base_inv: Base inventory against which to return inventory delta
 
198
    :return: Inventory delta for this subtree
106
199
    """
 
200
    invdelta = []
107
201
    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))
 
202
    # We just have to hope this is indeed utf-8:
 
203
    ie = InventoryDirectory(file_id, urlutils.basename(path.decode("utf-8")), 
 
204
        parent_id)
 
205
    if not file_id in base_inv:
 
206
        # Newly appeared here
 
207
        ie.revision = revision_id
 
208
        texts.add_lines((file_id, ie.revision), [], [])
 
209
        invdelta.append((None, path, file_id, ie))
 
210
    else:
 
211
        # See if this has changed at all
 
212
        try:
 
213
            base_sha = shagitmap.lookup_tree(file_id, base_inv.revision_id)
 
214
        except KeyError:
 
215
            pass
 
216
        else:
 
217
            if base_sha == hexsha:
 
218
                # If nothing has changed since the base revision, we're done
 
219
                return [], {}
 
220
    # Remember for next time
 
221
    existing_children = set()
 
222
    shagitmap.add_entry(hexsha, "tree", (file_id, revision_id))
 
223
    child_modes = {}
 
224
    tree = lookup_object(hexsha)
115
225
    for mode, name, hexsha in tree.entries():
116
 
        entry_kind = (mode & 0700000) / 0100000
117
226
        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,))
 
227
        existing_children.add(basename)
 
228
        child_path = osutils.pathjoin(path, name)
 
229
        if stat.S_ISDIR(mode):
 
230
            subinvdelta, grandchildmodes = import_git_tree(texts, mapping,
 
231
                child_path, hexsha, base_inv, file_id, revision_id, 
 
232
                parent_invs, shagitmap, lookup_object)
 
233
            invdelta.extend(subinvdelta)
 
234
            child_modes.update(grandchildmodes)
 
235
        else:
 
236
            fs_mode = stat.S_IMODE(mode)
 
237
            symlink = stat.S_ISLNK(mode)
 
238
            subinvdelta = import_git_blob(texts, mapping, child_path, hexsha,
 
239
                    base_inv, file_id, revision_id, parent_invs, shagitmap, 
 
240
                    lookup_object, bool(fs_mode & 0111), symlink)
 
241
            invdelta.extend(subinvdelta)
 
242
        if mode not in (DEFAULT_TREE_MODE, DEFAULT_FILE_MODE,
 
243
                        DEFAULT_SYMLINK_MODE, DEFAULT_FILE_MODE|0111):
 
244
            child_modes[child_path] = mode
 
245
    # Remove any children that have disappeared
 
246
    if file_id in base_inv:
 
247
        deletable = [v for k,v in base_inv[file_id].children.iteritems() if k not in existing_children]
 
248
        while deletable:
 
249
            ie = deletable.pop()
 
250
            invdelta.append((base_inv.id2path(ie.file_id), None, ie.file_id, None))
 
251
            if ie.kind == "directory":
 
252
                deletable.extend(ie.children.values())
 
253
    return invdelta, child_modes
131
254
 
132
255
 
133
256
def import_git_objects(repo, mapping, object_iter, target_git_object_retriever, 
134
 
        pb=None):
 
257
        heads, pb=None):
135
258
    """Import a set of git objects into a bzr repository.
136
259
 
137
260
    :param repo: Bazaar repository
142
265
    graph = []
143
266
    root_trees = {}
144
267
    revisions = {}
 
268
    checked = set()
 
269
    heads = list(heads)
 
270
    parent_invs_cache = LRUCache(50)
145
271
    # Find and convert commit objects
146
 
    for o in object_iter.iterobjects():
 
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
147
281
        if isinstance(o, Commit):
148
282
            rev = mapping.import_commit(o)
149
 
            root_trees[rev.revision_id] = object_iter[o.tree]
 
283
            if repo.has_revision(rev.revision_id):
 
284
                continue
 
285
            root_trees[rev.revision_id] = o.tree
150
286
            revisions[rev.revision_id] = rev
151
287
            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))
 
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)
153
296
    # Order the revisions
154
297
    # Create the inventory objects
155
298
    for i, revid in enumerate(topo_sort(graph)):
156
299
        if pb is not None:
157
300
            pb.update("fetching revisions", i, len(graph))
158
 
        root_tree = root_trees[revid]
159
301
        rev = revisions[revid]
160
302
        # 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 
 
303
        # we need to make sure to import the blobs / trees with the right 
162
304
        # path; this may involve adding them more than once.
163
 
        inv = Inventory()
164
 
        inv.revision_id = rev.revision_id
165
305
        def lookup_object(sha):
166
 
            if sha in object_iter:
 
306
            try:
167
307
                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)
 
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 = import_git_tree(repo.texts, mapping, "", 
 
323
            root_trees[revid], base_inv, None, revid, parent_invs, 
 
324
            target_git_object_retriever._idmap, lookup_object)
 
325
        if unusual_modes != {}:
 
326
            ret = "unusual modes: \n"
 
327
            for item in unusual_modes.iteritems():
 
328
                ret += "\t%s: %o\n" % item
 
329
            raise AssertionError(ret)
 
330
        try:
 
331
            basis_id = rev.parent_ids[0]
 
332
        except IndexError:
 
333
            basis_id = NULL_REVISION
 
334
        rev.inventory_sha1, inv = repo.add_inventory_by_delta(basis_id,
 
335
                  inv_delta, rev.revision_id, rev.parent_ids)
 
336
        parent_invs_cache[rev.revision_id] = inv
 
337
        repo.add_revision(rev.revision_id, rev)
 
338
        if "verify" in debug.debug_flags:
 
339
            objs = inventory_to_tree_and_blobs(inv, repo.texts, mapping)
 
340
            for sha1, newobj, path in objs:
 
341
                assert path is not None
 
342
                oldobj = tree_lookup_path(object_iter, root_trees[revid], path)
 
343
                assert oldobj == newobj, "%r != %r in %s" % (oldobj, newobj, path)
 
344
 
 
345
    target_git_object_retriever._idmap.commit()
173
346
 
174
347
 
175
348
class InterGitNonGitRepository(InterRepository):
 
349
    """Base InterRepository that copies revisions from a Git into a non-Git 
 
350
    repository."""
176
351
 
177
 
    _matching_repo_format = GitFormat()
 
352
    _matching_repo_format = GitRepositoryFormat()
178
353
 
179
354
    @staticmethod
180
355
    def _get_repo_format_to_test():
184
359
        """See InterRepository.copy_content."""
185
360
        self.fetch(revision_id, pb, find_ghosts=False)
186
361
 
 
362
    def fetch(self, revision_id=None, pb=None, find_ghosts=False, mapping=None,
 
363
            fetch_spec=None):
 
364
        self.fetch_refs(revision_id=revision_id, pb=pb, find_ghosts=find_ghosts,
 
365
                mapping=mapping, fetch_spec=fetch_spec)
 
366
 
 
367
    def fetch_refs(self, revision_id=None, pb=None, find_ghosts=False, 
 
368
              mapping=None, fetch_spec=None):
 
369
        if mapping is None:
 
370
            mapping = self.source.get_mapping()
 
371
        if revision_id is not None:
 
372
            interesting_heads = [revision_id]
 
373
        elif fetch_spec is not None:
 
374
            interesting_heads = fetch_spec.heads
 
375
        else:
 
376
            interesting_heads = None
 
377
        self._refs = {}
 
378
        def determine_wants(refs):
 
379
            self._refs = refs
 
380
            if interesting_heads is None:
 
381
                ret = [sha for (ref, sha) in refs.iteritems() if not ref.endswith("^{}")]
 
382
            else:
 
383
                ret = [mapping.revision_id_bzr_to_foreign(revid)[0] for revid in interesting_heads if revid != NULL_REVISION]
 
384
            return [rev for rev in ret if not self.target.has_revision(mapping.revision_id_foreign_to_bzr(rev))]
 
385
        self.fetch_objects(determine_wants, mapping, pb)
 
386
        return self._refs
 
387
 
 
388
 
 
389
 
 
390
class InterRemoteGitNonGitRepository(InterGitNonGitRepository):
 
391
    """InterRepository that copies revisions from a remote Git into a non-Git 
 
392
    repository."""
 
393
 
187
394
    def fetch_objects(self, determine_wants, mapping, pb=None):
188
395
        def progress(text):
189
396
            pb.update("git: %s" % text.rstrip("\r\n"), 0, 0)
191
398
        create_pb = None
192
399
        if pb is None:
193
400
            create_pb = pb = ui.ui_factory.nested_progress_bar()
194
 
        target_git_object_retriever = GitObjectConverter(self.target, mapping)
 
401
        target_git_object_retriever = BazaarObjectStore(self.target, mapping)
 
402
        recorded_wants = []
 
403
 
 
404
        def record_determine_wants(heads):
 
405
            wants = determine_wants(heads)
 
406
            recorded_wants.extend(wants)
 
407
            return wants
195
408
        
196
409
        try:
197
410
            self.target.lock_write()
198
411
            try:
199
412
                self.target.start_write_group()
200
413
                try:
201
 
                    objects_iter = self.source.fetch_objects(determine_wants, 
 
414
                    objects_iter = self.source.fetch_objects(
 
415
                                record_determine_wants, 
202
416
                                graph_walker, 
203
 
                                target_git_object_retriever.__getitem__, 
 
417
                                target_git_object_retriever.get_raw, 
204
418
                                progress)
205
419
                    import_git_objects(self.target, mapping, objects_iter, 
206
 
                            target_git_object_retriever, pb)
207
 
                finally:
208
 
                    self.target.commit_write_group()
209
 
            finally:
210
 
                self.target.unlock()
211
 
        finally:
212
 
            if create_pb:
213
 
                create_pb.finished()
214
 
 
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
 
    @staticmethod
228
 
    def is_compatible(source, target):
229
 
        """Be compatible with GitRepository."""
230
 
        # FIXME: Also check target uses VersionedFile
231
 
        return (isinstance(source, GitRepository) and 
 
420
                            target_git_object_retriever, recorded_wants, pb)
 
421
                finally:
 
422
                    self.target.commit_write_group()
 
423
            finally:
 
424
                self.target.unlock()
 
425
        finally:
 
426
            if create_pb:
 
427
                create_pb.finished()
 
428
 
 
429
    @staticmethod
 
430
    def is_compatible(source, target):
 
431
        """Be compatible with GitRepository."""
 
432
        # FIXME: Also check target uses VersionedFile
 
433
        return (isinstance(source, RemoteGitRepository) and 
 
434
                target.supports_rich_root() and
 
435
                not isinstance(target, GitRepository))
 
436
 
 
437
 
 
438
class InterLocalGitNonGitRepository(InterGitNonGitRepository):
 
439
    """InterRepository that copies revisions from a remote Git into a non-Git 
 
440
    repository."""
 
441
 
 
442
    def fetch_objects(self, determine_wants, mapping, pb=None):
 
443
        wants = determine_wants(self.source._git.get_refs())
 
444
        create_pb = None
 
445
        if pb is None:
 
446
            create_pb = pb = ui.ui_factory.nested_progress_bar()
 
447
        target_git_object_retriever = BazaarObjectStore(self.target, mapping)
 
448
        try:
 
449
            self.target.lock_write()
 
450
            try:
 
451
                self.target.start_write_group()
 
452
                try:
 
453
                    import_git_objects(self.target, mapping, 
 
454
                            self.source._git.object_store, 
 
455
                            target_git_object_retriever, wants, pb)
 
456
                finally:
 
457
                    self.target.commit_write_group()
 
458
            finally:
 
459
                self.target.unlock()
 
460
        finally:
 
461
            if create_pb:
 
462
                create_pb.finished()
 
463
 
 
464
    @staticmethod
 
465
    def is_compatible(source, target):
 
466
        """Be compatible with GitRepository."""
 
467
        # FIXME: Also check target uses VersionedFile
 
468
        return (isinstance(source, LocalGitRepository) and 
232
469
                target.supports_rich_root() and
233
470
                not isinstance(target, GitRepository))
234
471
 
235
472
 
236
473
class InterGitRepository(InterRepository):
 
474
    """InterRepository that copies between Git repositories."""
237
475
 
238
 
    _matching_repo_format = GitFormat()
 
476
    _matching_repo_format = GitRepositoryFormat()
239
477
 
240
478
    @staticmethod
241
479
    def _get_repo_format_to_test():
246
484
        self.fetch(revision_id, pb, find_ghosts=False)
247
485
 
248
486
    def fetch(self, revision_id=None, pb=None, find_ghosts=False, 
249
 
              mapping=None):
 
487
              mapping=None, fetch_spec=None):
250
488
        if mapping is None:
251
489
            mapping = self.source.get_mapping()
252
490
        def progress(text):
253
 
            info("git: %s", text)
 
491
            trace.info("git: %s", text)
254
492
        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:
 
493
        if revision_id is not None:
258
494
            args = [mapping.revision_id_bzr_to_foreign(revision_id)[0]]
 
495
        elif fetch_spec is not None:
 
496
            args = [mapping.revision_id_bzr_to_foreign(revid)[0] for revid in fetch_spec.heads]
 
497
        if fetch_spec is None and revision_id is None:
 
498
            determine_wants = r.object_store.determine_wants_all
 
499
        else:
259
500
            determine_wants = lambda x: [y for y in args if not y in r.object_store]
260
501
 
261
502
        graphwalker = SimpleFetchGraphWalker(r.heads().values(), r.get_parents)
262
 
        f, commit = r.object_store.add_pack()
 
503
        f, commit = r.object_store.add_thin_pack()
263
504
        try:
264
 
            self.source._git.fetch_pack(path, determine_wants, graphwalker, f.write, progress)
265
 
            f.close()
 
505
            self.source.fetch_pack(determine_wants, graphwalker, f.write, progress)
266
506
            commit()
267
507
        except:
268
508
            f.close()