/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, 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
    inventory_to_tree_and_blobs,
 
69
    text_to_blob,
 
70
    )
24
71
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
 
72
    LocalGitRepository, 
 
73
    GitRepository, 
 
74
    GitRepositoryFormat,
 
75
    )
 
76
from bzrlib.plugins.git.remote import (
 
77
    RemoteGitRepository,
 
78
    )
37
79
 
38
80
 
39
81
class BzrFetchGraphWalker(object):
75
117
        return None
76
118
 
77
119
 
78
 
def import_git_blob(repo, mapping, path, blob, inv, parent_invs, gitmap, executable):
 
120
def import_git_blob(texts, mapping, path, hexsha, base_inv, parent_id, 
 
121
    revision_id, parent_invs, shagitmap, lookup_object, executable, symlink):
79
122
    """Import a git blob object into a bzr repository.
80
123
 
81
 
    :param repo: bzr repository
 
124
    :param texts: VersionedFiles to add to
82
125
    :param path: Path in the tree
83
126
    :param blob: A git blob
 
127
    :return: Inventory delta for this file
84
128
    """
85
129
    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)
 
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)
94
136
    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):
 
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):
100
188
    """Import a git tree object into a bzr repository.
101
189
 
102
 
    :param repo: A Bzr repository object
 
190
    :param texts: VersionedFiles object to add to
103
191
    :param path: Path in the tree
104
192
    :param tree: A git tree object
105
 
    :param inv: Inventory object
 
193
    :param base_inv: Base inventory against which to return inventory delta
 
194
    :return: Inventory delta for this subtree
106
195
    """
 
196
    invdelta = []
107
197
    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
 
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():
117
222
        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,))
 
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
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, 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()
173
347
 
174
348
 
175
349
class InterGitNonGitRepository(InterRepository):
 
350
    """Base InterRepository that copies revisions from a Git into a non-Git 
 
351
    repository."""
176
352
 
177
 
    _matching_repo_format = GitFormat()
 
353
    _matching_repo_format = GitRepositoryFormat()
178
354
 
179
355
    @staticmethod
180
356
    def _get_repo_format_to_test():
184
360
        """See InterRepository.copy_content."""
185
361
        self.fetch(revision_id, pb, find_ghosts=False)
186
362
 
 
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):
 
370
        if mapping is None:
 
371
            mapping = self.source.get_mapping()
 
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("^{}")]
 
383
            else:
 
384
                ret = [mapping.revision_id_bzr_to_foreign(revid)[0] for revid in interesting_heads if revid != NULL_REVISION]
 
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
 
187
395
    def fetch_objects(self, determine_wants, mapping, pb=None):
188
396
        def progress(text):
189
397
            pb.update("git: %s" % text.rstrip("\r\n"), 0, 0)
191
399
        create_pb = None
192
400
        if pb is None:
193
401
            create_pb = pb = ui.ui_factory.nested_progress_bar()
194
 
        target_git_object_retriever = GitObjectConverter(self.target, mapping)
 
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
195
409
        
196
410
        try:
197
411
            self.target.lock_write()
198
412
            try:
199
413
                self.target.start_write_group()
200
414
                try:
201
 
                    objects_iter = self.source.fetch_objects(determine_wants, 
 
415
                    objects_iter = self.source.fetch_objects(
 
416
                                record_determine_wants, 
202
417
                                graph_walker, 
203
 
                                target_git_object_retriever.__getitem__, 
 
418
                                target_git_object_retriever.get_raw, 
204
419
                                progress)
205
420
                    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 
 
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 
232
470
                target.supports_rich_root() and
233
471
                not isinstance(target, GitRepository))
234
472
 
235
473
 
236
474
class InterGitRepository(InterRepository):
 
475
    """InterRepository that copies between Git repositories."""
237
476
 
238
 
    _matching_repo_format = GitFormat()
 
477
    _matching_repo_format = GitRepositoryFormat()
239
478
 
240
479
    @staticmethod
241
480
    def _get_repo_format_to_test():
246
485
        self.fetch(revision_id, pb, find_ghosts=False)
247
486
 
248
487
    def fetch(self, revision_id=None, pb=None, find_ghosts=False, 
249
 
              mapping=None):
 
488
              mapping=None, fetch_spec=None):
250
489
        if mapping is None:
251
490
            mapping = self.source.get_mapping()
252
491
        def progress(text):
253
 
            info("git: %s", text)
 
492
            trace.info("git: %s", text)
254
493
        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:
 
494
        if revision_id is not None:
258
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:
259
501
            determine_wants = lambda x: [y for y in args if not y in r.object_store]
260
502
 
261
503
        graphwalker = SimpleFetchGraphWalker(r.heads().values(), r.get_parents)
262
 
        f, commit = r.object_store.add_pack()
 
504
        f, commit = r.object_store.add_thin_pack()
263
505
        try:
264
 
            self.source._git.fetch_pack(path, determine_wants, graphwalker, f.write, progress)
265
 
            f.close()
 
506
            self.source.fetch_pack(determine_wants, graphwalker, f.write, progress)
266
507
            commit()
267
508
        except:
268
509
            f.close()