/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

Fix regression in git-import.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2008 Jelmer Vernooij <jelmer@samba.org>
 
1
# Copyright (C) 2008-2010 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
17
17
from dulwich.objects import (
18
18
    Commit,
19
19
    Tag,
 
20
    Tree,
20
21
    S_ISGITLINK,
21
22
    )
22
23
from dulwich.object_store import (
23
24
    tree_lookup_path,
24
25
    )
 
26
from itertools import (
 
27
    imap,
 
28
    )
 
29
import posixpath
25
30
import re
26
31
import stat
27
32
 
30
35
    osutils,
31
36
    trace,
32
37
    ui,
33
 
    urlutils,
34
38
    )
35
39
from bzrlib.errors import (
36
40
    BzrError,
49
53
from bzrlib.revision import (
50
54
    NULL_REVISION,
51
55
    )
 
56
from bzrlib.revisiontree import (
 
57
    RevisionTree,
 
58
    )
52
59
from bzrlib.tsort import (
53
60
    topo_sort,
54
61
    )
58
65
 
59
66
from bzrlib.plugins.git.mapping import (
60
67
    DEFAULT_FILE_MODE,
61
 
    inventory_to_tree_and_blobs,
62
68
    mode_is_executable,
63
 
    squash_revision,
 
69
    mode_kind,
64
70
    warn_unusual_mode,
65
71
    )
66
72
from bzrlib.plugins.git.object_store import (
67
73
    BazaarObjectStore,
68
 
    LRUInventoryCache,
 
74
    LRUTreeCache,
 
75
    _tree_to_objects,
69
76
    )
70
77
from bzrlib.plugins.git.remote import (
71
78
    RemoteGitRepository,
77
84
    )
78
85
 
79
86
 
80
 
def import_git_blob(texts, mapping, path, hexsha, base_inv, base_inv_shamap,
81
 
    base_ie, parent_id, revision_id, parent_invs, lookup_object,
82
 
    executable, symlink):
 
87
def import_git_blob(texts, mapping, path, name, (base_hexsha, hexsha), 
 
88
        base_inv, parent_id, revision_id,
 
89
        parent_invs, lookup_object, (base_mode, mode), store_updater,
 
90
        lookup_file_id):
83
91
    """Import a git blob object into a bzr repository.
84
92
 
85
93
    :param texts: VersionedFiles to add to
87
95
    :param blob: A git blob
88
96
    :return: Inventory delta for this file
89
97
    """
90
 
    file_id = mapping.generate_file_id(path)
91
 
    if symlink:
 
98
    if mapping.is_control_file(path):
 
99
        return []
 
100
    if base_hexsha == hexsha and base_mode == mode:
 
101
        # If nothing has changed since the base revision, we're done
 
102
        return []
 
103
    file_id = lookup_file_id(path)
 
104
    if stat.S_ISLNK(mode):
92
105
        cls = InventoryLink
93
106
    else:
94
107
        cls = InventoryFile
95
 
    # We just have to hope this is indeed utf-8:
96
 
    ie = cls(file_id, urlutils.basename(path).decode("utf-8"), parent_id)
97
 
    ie.executable = executable
98
 
    # See if this has changed at all
99
 
    if base_ie is None:
100
 
        base_sha = None
101
 
    else:
102
 
        try:
103
 
            base_sha = base_inv_shamap.lookup_blob(file_id, base_ie.revision)
104
 
        except KeyError:
105
 
            base_sha = None
106
 
        else:
107
 
            if (base_sha == hexsha and base_ie.executable == ie.executable
108
 
                and base_ie.kind == ie.kind):
109
 
                # If nothing has changed since the base revision, we're done
110
 
                return [], []
111
 
    if base_sha == hexsha and base_ie.kind == ie.kind:
 
108
    ie = cls(file_id, name.decode("utf-8"), parent_id)
 
109
    if ie.kind == "file":
 
110
        ie.executable = mode_is_executable(mode)
 
111
    if base_hexsha == hexsha and mode_kind(base_mode) == mode_kind(mode):
 
112
        base_ie = base_inv[base_inv.path2id(path)]
112
113
        ie.text_size = base_ie.text_size
113
114
        ie.text_sha1 = base_ie.text_sha1
114
 
        ie.symlink_target = base_ie.symlink_target
 
115
        if ie.kind == "symlink":
 
116
            ie.symlink_target = base_ie.symlink_target
115
117
        if ie.executable == base_ie.executable:
116
118
            ie.revision = base_ie.revision
117
119
        else:
121
123
        if ie.kind == "symlink":
122
124
            ie.revision = None
123
125
            ie.symlink_target = blob.data
124
 
            ie.text_size = None
125
 
            ie.text_sha1 = None
126
126
        else:
127
 
            ie.text_size = len(blob.data)
128
 
            ie.text_sha1 = osutils.sha_string(blob.data)
 
127
            ie.text_size = sum(imap(len, blob.chunked))
 
128
            ie.text_sha1 = osutils.sha_strings(blob.chunked)
129
129
    # Check what revision we should store
130
130
    parent_keys = []
131
131
    for pinv in parent_invs:
132
 
        if pinv.revision_id == base_inv.revision_id:
133
 
            pie = base_ie
134
 
            if pie is None:
135
 
                continue
136
 
        else:
137
 
            try:
138
 
                pie = pinv[file_id]
139
 
            except NoSuchId:
140
 
                continue
141
 
        if pie.text_sha1 == ie.text_sha1 and pie.executable == ie.executable and pie.symlink_target == ie.symlink_target:
 
132
        try:
 
133
            pie = pinv[file_id]
 
134
        except NoSuchId:
 
135
            continue
 
136
        if (pie.text_sha1 == ie.text_sha1 and
 
137
            pie.executable == ie.executable and
 
138
            pie.symlink_target == ie.symlink_target):
142
139
            # found a revision in one of the parents to use
143
140
            ie.revision = pie.revision
144
141
            break
145
 
        parent_keys.append((file_id, pie.revision))
 
142
        parent_key = (file_id, pie.revision)
 
143
        if not parent_key in parent_keys:
 
144
            parent_keys.append(parent_key)
146
145
    if ie.revision is None:
147
146
        # Need to store a new revision
148
147
        ie.revision = revision_id
149
 
        assert file_id is not None
150
148
        assert ie.revision is not None
151
149
        if ie.kind == 'symlink':
152
150
            chunks = []
153
151
        else: 
154
 
            try:
155
 
                chunks = blob.chunked
156
 
            except AttributeError: # older version of dulwich
157
 
                chunks = [blob.data]
158
 
        texts.insert_record_stream([ChunkedContentFactory((file_id, ie.revision), tuple(parent_keys), ie.text_sha1, chunks)])
159
 
    shamap = { ie.file_id: hexsha }
 
152
            chunks = blob.chunked
 
153
        texts.insert_record_stream([
 
154
            ChunkedContentFactory((file_id, ie.revision),
 
155
                tuple(parent_keys), ie.text_sha1, chunks)])
160
156
    invdelta = []
161
 
    if base_ie is not None:
162
 
        old_path = base_inv.id2path(file_id)
163
 
        if base_ie.kind == "directory":
164
 
            invdelta.extend(remove_disappeared_children(old_path, base_ie.children, []))
 
157
    if base_hexsha is not None:
 
158
        old_path = path.decode("utf-8") # Renames are not supported yet
 
159
        if stat.S_ISDIR(base_mode):
 
160
            invdelta.extend(remove_disappeared_children(base_inv, old_path,
 
161
                lookup_object(base_hexsha), [], lookup_object))
165
162
    else:
166
163
        old_path = None
167
 
    invdelta.append((old_path, path, file_id, ie))
168
 
    return (invdelta, shamap)
 
164
    new_path = path.decode("utf-8")
 
165
    invdelta.append((old_path, new_path, file_id, ie))
 
166
    if base_hexsha != hexsha:
 
167
        store_updater.add_object(blob, ie, path)
 
168
    return invdelta
169
169
 
170
170
 
171
171
class SubmodulesRequireSubtrees(BzrError):
173
173
    internal = False
174
174
 
175
175
 
176
 
def import_git_submodule(texts, mapping, path, hexsha, base_inv, base_ie,
177
 
    parent_id, revision_id, parent_invs, lookup_object):
178
 
    file_id = mapping.generate_file_id(path)
179
 
    ie = TreeReference(file_id, urlutils.basename(path.decode("utf-8")),
180
 
        parent_id)
 
176
def import_git_submodule(texts, mapping, path, name, (base_hexsha, hexsha),
 
177
    base_inv, parent_id, revision_id, parent_invs, lookup_object,
 
178
    (base_mode, mode), store_updater, lookup_file_id):
 
179
    if base_hexsha == hexsha and base_mode == mode:
 
180
        return [], {}
 
181
    file_id = lookup_file_id(path)
 
182
    ie = TreeReference(file_id, name.decode("utf-8"), parent_id)
181
183
    ie.revision = revision_id
182
 
    if base_ie is None:
 
184
    if base_hexsha is None:
183
185
        oldpath = None
184
186
    else:
185
187
        oldpath = path
186
 
        if (base_ie.kind == ie.kind and
187
 
            base_ie.reference_revision == ie.reference_revision):
188
 
            ie.revision = base_ie.revision
189
188
    ie.reference_revision = mapping.revision_id_foreign_to_bzr(hexsha)
190
 
    texts.insert_record_stream([ChunkedContentFactory((file_id, ie.revision), (), None, [])])
 
189
    texts.insert_record_stream([
 
190
        ChunkedContentFactory((file_id, ie.revision), (), None, [])])
191
191
    invdelta = [(oldpath, path, file_id, ie)]
192
 
    return invdelta, {}, {}
193
 
 
194
 
 
195
 
def remove_disappeared_children(path, base_children, existing_children):
 
192
    return invdelta, {}
 
193
 
 
194
 
 
195
def remove_disappeared_children(base_inv, path, base_tree, existing_children,
 
196
        lookup_object):
 
197
    """Generate an inventory delta for removed children.
 
198
 
 
199
    :param base_inv: Base inventory against which to generate the 
 
200
        inventory delta.
 
201
    :param path: Path to process (unicode)
 
202
    :param base_tree: Git Tree base object
 
203
    :param existing_children: Children that still exist
 
204
    :param lookup_object: Lookup a git object by its SHA1
 
205
    :return: Inventory delta, as list
 
206
    """
 
207
    assert type(path) is unicode
196
208
    ret = []
197
 
    deletable = [(osutils.pathjoin(path, k), v) for k,v in base_children.iteritems() if k not in existing_children]
198
 
    while deletable:
199
 
        (path, ie) = deletable.pop()
200
 
        ret.append((path, None, ie.file_id, None))
201
 
        if ie.kind == "directory":
202
 
            for name, child_ie in ie.children.iteritems():
203
 
                deletable.append((osutils.pathjoin(path, name), child_ie))
 
209
    for name, mode, hexsha in base_tree.iteritems():
 
210
        if name in existing_children:
 
211
            continue
 
212
        c_path = posixpath.join(path, name.decode("utf-8"))
 
213
        file_id = base_inv.path2id(c_path)
 
214
        assert file_id is not None
 
215
        ret.append((c_path, None, file_id, None))
 
216
        if stat.S_ISDIR(mode):
 
217
            ret.extend(remove_disappeared_children(
 
218
                base_inv, c_path, lookup_object(hexsha), [], lookup_object))
204
219
    return ret
205
220
 
206
221
 
207
 
def import_git_tree(texts, mapping, path, hexsha, base_inv, base_inv_shamap,
208
 
    base_ie, parent_id, revision_id, parent_invs, lookup_object,
209
 
    allow_submodules=False):
 
222
def import_git_tree(texts, mapping, path, name, (base_hexsha, hexsha),
 
223
        base_inv, parent_id, revision_id, parent_invs,
 
224
        lookup_object, (base_mode, mode), store_updater,
 
225
        lookup_file_id, allow_submodules=False):
210
226
    """Import a git tree object into a bzr repository.
211
227
 
212
228
    :param texts: VersionedFiles object to add to
213
 
    :param path: Path in the tree
 
229
    :param path: Path in the tree (str)
 
230
    :param name: Name of the tree (str)
214
231
    :param tree: A git tree object
215
232
    :param base_inv: Base inventory against which to return inventory delta
216
233
    :return: Inventory delta for this subtree
217
234
    """
 
235
    assert type(path) is str
 
236
    assert type(name) is str
 
237
    if base_hexsha == hexsha and base_mode == mode:
 
238
        # If nothing has changed since the base revision, we're done
 
239
        return [], {}
218
240
    invdelta = []
219
 
    file_id = mapping.generate_file_id(path)
 
241
    file_id = lookup_file_id(path)
220
242
    # We just have to hope this is indeed utf-8:
221
 
    ie = InventoryDirectory(file_id, urlutils.basename(path.decode("utf-8")),
222
 
        parent_id)
223
 
    if base_ie is None:
224
 
        # Newly appeared here
 
243
    ie = InventoryDirectory(file_id, name.decode("utf-8"), parent_id)
 
244
    tree = lookup_object(hexsha)
 
245
    if base_hexsha is None:
 
246
        base_tree = None
 
247
        old_path = None # Newly appeared here
 
248
    else:
 
249
        base_tree = lookup_object(base_hexsha)
 
250
        old_path = path.decode("utf-8") # Renames aren't supported yet
 
251
    new_path = path.decode("utf-8")
 
252
    if base_tree is None or type(base_tree) is not Tree:
225
253
        ie.revision = revision_id
226
 
        texts.insert_record_stream([ChunkedContentFactory((file_id, ie.revision), (), None, [])])
227
 
        invdelta.append((None, path, file_id, ie))
228
 
    else:
229
 
        # See if this has changed at all
230
 
        try:
231
 
            base_sha = base_inv_shamap.lookup_tree(file_id)
232
 
        except (KeyError, NotImplementedError):
233
 
            pass
234
 
        else:
235
 
            if base_sha == hexsha:
236
 
                # If nothing has changed since the base revision, we're done
237
 
                return [], {}, []
238
 
        if base_ie.kind != "directory":
239
 
            ie.revision = revision_id
240
 
            texts.insert_record_stream([ChunkedContentFactory((ie.file_id, ie.revision), (), None, [])])
241
 
            invdelta.append((base_inv.id2path(ie.file_id), path, ie.file_id, ie))
242
 
    if base_ie is not None and base_ie.kind == "directory":
243
 
        base_children = base_ie.children
244
 
    else:
245
 
        base_children = {}
 
254
        invdelta.append((old_path, new_path, ie.file_id, ie))
 
255
        texts.insert_record_stream([
 
256
            ChunkedContentFactory((ie.file_id, ie.revision), (), None, [])])
246
257
    # Remember for next time
247
258
    existing_children = set()
248
259
    child_modes = {}
249
 
    shamap = {}
250
 
    tree = lookup_object(hexsha)
251
 
    for mode, name, child_hexsha in tree.entries():
252
 
        basename = name.decode("utf-8")
253
 
        existing_children.add(basename)
254
 
        child_path = osutils.pathjoin(path, name)
255
 
        if stat.S_ISDIR(mode):
256
 
            subinvdelta, grandchildmodes, subshamap = import_git_tree(
257
 
                    texts, mapping, child_path, child_hexsha, base_inv,
258
 
                    base_inv_shamap, base_children.get(basename), file_id,
259
 
                    revision_id, parent_invs, lookup_object,
260
 
                    allow_submodules=allow_submodules)
261
 
        elif S_ISGITLINK(mode): # submodule
 
260
    for child_mode, name, child_hexsha in tree.entries():
 
261
        existing_children.add(name)
 
262
        child_path = posixpath.join(path, name)
 
263
        if type(base_tree) is Tree:
 
264
            try:
 
265
                child_base_mode, child_base_hexsha = base_tree[name]
 
266
            except KeyError:
 
267
                child_base_hexsha = None
 
268
                child_base_mode = 0
 
269
        else:
 
270
            child_base_hexsha = None
 
271
            child_base_mode = 0
 
272
        if stat.S_ISDIR(child_mode):
 
273
            subinvdelta, grandchildmodes = import_git_tree(texts, mapping,
 
274
                child_path, name, (child_base_hexsha, child_hexsha), base_inv,
 
275
                file_id, revision_id, parent_invs, lookup_object, 
 
276
                (child_base_mode, child_mode), store_updater, lookup_file_id,
 
277
                allow_submodules=allow_submodules)
 
278
        elif S_ISGITLINK(child_mode): # submodule
262
279
            if not allow_submodules:
263
280
                raise SubmodulesRequireSubtrees()
264
 
            subinvdelta, grandchildmodes, subshamap = import_git_submodule(
265
 
                    texts, mapping, child_path, child_hexsha, base_inv, base_children.get(basename),
266
 
                    file_id, revision_id, parent_invs, lookup_object)
 
281
            subinvdelta, grandchildmodes = import_git_submodule(texts, mapping,
 
282
                child_path, name, (child_base_hexsha, child_hexsha), base_inv,
 
283
                file_id, revision_id, parent_invs, lookup_object,
 
284
                (child_base_mode, child_mode), store_updater, lookup_file_id)
267
285
        else:
268
 
            subinvdelta, subshamap = import_git_blob(texts, mapping,
269
 
                    child_path, child_hexsha, base_inv, base_inv_shamap,
270
 
                    base_children.get(basename), file_id,
271
 
                    revision_id, parent_invs, lookup_object,
272
 
                    mode_is_executable(mode), stat.S_ISLNK(mode))
 
286
            subinvdelta = import_git_blob(texts, mapping, child_path, name,
 
287
                (child_base_hexsha, child_hexsha), base_inv, file_id,
 
288
                revision_id, parent_invs, lookup_object,
 
289
                (child_base_mode, child_mode), store_updater, lookup_file_id)
273
290
            grandchildmodes = {}
274
291
        child_modes.update(grandchildmodes)
275
292
        invdelta.extend(subinvdelta)
276
 
        shamap.update(subshamap)
277
 
        if mode not in (stat.S_IFDIR, DEFAULT_FILE_MODE,
 
293
        if child_mode not in (stat.S_IFDIR, DEFAULT_FILE_MODE,
278
294
                        stat.S_IFLNK, DEFAULT_FILE_MODE|0111):
279
 
            child_modes[child_path] = mode
 
295
            child_modes[child_path] = child_mode
280
296
    # Remove any children that have disappeared
281
 
    if base_ie is not None and base_ie.kind == "directory":
282
 
        invdelta.extend(remove_disappeared_children(base_inv.id2path(file_id),
283
 
            base_children, existing_children))
284
 
    shamap[file_id] = hexsha
285
 
    return invdelta, child_modes, shamap
 
297
    if base_tree is not None and type(base_tree) is Tree:
 
298
        invdelta.extend(remove_disappeared_children(base_inv, old_path,
 
299
            base_tree, existing_children, lookup_object))
 
300
    store_updater.add_object(tree, ie, path)
 
301
    return invdelta, child_modes
 
302
 
 
303
 
 
304
def verify_commit_reconstruction(target_git_object_retriever, lookup_object,
 
305
    o, rev, ret_tree, parent_trees, mapping, unusual_modes):
 
306
    new_unusual_modes = mapping.export_unusual_file_modes(rev)
 
307
    if new_unusual_modes != unusual_modes:
 
308
        raise AssertionError("unusual modes don't match: %r != %r" % (
 
309
            unusual_modes, new_unusual_modes))
 
310
    # Verify that we can reconstruct the commit properly
 
311
    rec_o = target_git_object_retriever._reconstruct_commit(rev, o.tree, True)
 
312
    if rec_o != o:
 
313
        raise AssertionError("Reconstructed commit differs: %r != %r" % (
 
314
            rec_o, o))
 
315
    diff = []
 
316
    new_objs = {}
 
317
    for path, obj, ie in _tree_to_objects(ret_tree, parent_trees,
 
318
        target_git_object_retriever._cache.idmap, unusual_modes, mapping.BZR_DUMMY_FILE):
 
319
        old_obj_id = tree_lookup_path(lookup_object, o.tree, path)[1]
 
320
        new_objs[path] = obj
 
321
        if obj.id != old_obj_id:
 
322
            diff.append((path, lookup_object(old_obj_id), obj))
 
323
    for (path, old_obj, new_obj) in diff:
 
324
        while (old_obj.type_name == "tree" and
 
325
               new_obj.type_name == "tree" and
 
326
               sorted(old_obj) == sorted(new_obj)):
 
327
            for name in old_obj:
 
328
                if old_obj[name][0] != new_obj[name][0]:
 
329
                    raise AssertionError("Modes for %s differ: %o != %o" %
 
330
                        (path, old_obj[name][0], new_obj[name][0]))
 
331
                if old_obj[name][1] != new_obj[name][1]:
 
332
                    # Found a differing child, delve deeper
 
333
                    path = posixpath.join(path, name)
 
334
                    old_obj = lookup_object(old_obj[name][1])
 
335
                    new_obj = new_objs[path]
 
336
                    break
 
337
        raise AssertionError("objects differ for %s: %r != %r" % (path,
 
338
            old_obj, new_obj))
286
339
 
287
340
 
288
341
def import_git_commit(repo, mapping, head, lookup_object,
289
 
                      target_git_object_retriever, parent_invs_cache):
 
342
                      target_git_object_retriever, trees_cache):
290
343
    o = lookup_object(head)
291
 
    rev = mapping.import_commit(o)
 
344
    rev = mapping.import_commit(o,
 
345
            lambda x: target_git_object_retriever.lookup_git_sha(x)[1][0])
292
346
    # We have to do this here, since we have to walk the tree and
293
347
    # we need to make sure to import the blobs / trees with the right
294
348
    # path; this may involve adding them more than once.
295
 
    parent_invs = parent_invs_cache.get_inventories(rev.parent_ids)
296
 
    if parent_invs == []:
 
349
    parent_trees = trees_cache.revision_trees(rev.parent_ids)
 
350
    if parent_trees == []:
297
351
        base_inv = Inventory(root_id=None)
298
 
        base_ie = None
299
 
        base_inv_shamap = None # Should never be accessed
 
352
        base_tree = None
 
353
        base_mode = None
300
354
    else:
301
 
        base_inv = parent_invs[0]
302
 
        base_ie = base_inv.root
303
 
        base_inv_shamap = target_git_object_retriever._idmap.get_inventory_sha_map(base_inv.revision_id)
304
 
    inv_delta, unusual_modes, shamap = import_git_tree(repo.texts,
305
 
            mapping, "", o.tree, base_inv, base_inv_shamap, base_ie, None,
306
 
            rev.revision_id, parent_invs, lookup_object,
 
355
        base_inv = parent_trees[0].inventory
 
356
        base_tree = lookup_object(o.parents[0]).tree
 
357
        base_mode = stat.S_IFDIR
 
358
    store_updater = target_git_object_retriever._get_updater(rev)
 
359
    store_updater.add_object(o, None, None)
 
360
    fileid_map = mapping.get_fileid_map(lookup_object, o.tree)
 
361
    inv_delta, unusual_modes = import_git_tree(repo.texts,
 
362
            mapping, "", "", (base_tree, o.tree), base_inv,
 
363
            None, rev.revision_id, [p.inventory for p in parent_trees],
 
364
            lookup_object, (base_mode, stat.S_IFDIR), store_updater,
 
365
            fileid_map.lookup_file_id,
307
366
            allow_submodules=getattr(repo._format, "supports_tree_reference", False))
308
 
    entries = []
309
 
    for (oldpath, newpath, fileid, new_ie) in inv_delta:
310
 
        if newpath is None:
311
 
            entries.append((fileid, None, None, None))
312
 
        else:
313
 
            if new_ie.kind in ("file", "symlink"):
314
 
                entries.append((fileid, "blob", shamap[fileid], new_ie.revision))
315
 
            elif new_ie.kind == "directory":
316
 
                entries.append((fileid, "tree", shamap[fileid], rev.revision_id))
317
 
            else:
318
 
                raise AssertionError
319
 
    target_git_object_retriever._idmap.add_entries(rev.revision_id,
320
 
        rev.parent_ids, head, o.tree, entries)
 
367
    store_updater.finish()
321
368
    if unusual_modes != {}:
322
369
        for path, mode in unusual_modes.iteritems():
323
370
            warn_unusual_mode(rev.foreign_revid, path, mode)
328
375
        basis_id = NULL_REVISION
329
376
        base_inv = None
330
377
    rev.inventory_sha1, inv = repo.add_inventory_by_delta(basis_id,
331
 
              inv_delta, rev.revision_id, rev.parent_ids,
332
 
              base_inv)
333
 
    parent_invs_cache.add(rev.revision_id, inv)
 
378
              inv_delta, rev.revision_id, rev.parent_ids, base_inv)
 
379
    ret_tree = RevisionTree(repo, inv, rev.revision_id)
 
380
    trees_cache.add(ret_tree)
334
381
    repo.add_revision(rev.revision_id, rev)
335
382
    if "verify" in debug.debug_flags:
336
 
        new_unusual_modes = mapping.export_unusual_file_modes(rev)
337
 
        if new_unusual_modes != unusual_modes:
338
 
            raise AssertionError("unusual modes don't match: %r != %r" % (unusual_modes, new_unusual_modes))
339
 
        objs = inventory_to_tree_and_blobs(inv, repo.texts, mapping, unusual_modes)
340
 
        for newsha1, newobj, path in objs:
341
 
            assert path is not None
342
 
            if path == "":
343
 
                oldsha1 = o.tree
344
 
            else:
345
 
                (oldmode, oldsha1) = tree_lookup_path(lookup_object, o.tree, path)
346
 
            if oldsha1 != newsha1:
347
 
                raise AssertionError("%r != %r in %s" % (oldsha1, newsha1, path))
 
383
        verify_commit_reconstruction(target_git_object_retriever, 
 
384
            lookup_object, o, rev, ret_tree, parent_trees, mapping,
 
385
            unusual_modes)
348
386
 
349
387
 
350
388
def import_git_objects(repo, mapping, object_iter,
361
399
            return object_iter[sha]
362
400
        except KeyError:
363
401
            return target_git_object_retriever[sha]
364
 
    # TODO: a more (memory-)efficient implementation of this
365
402
    graph = []
366
403
    checked = set()
367
404
    heads = list(set(heads))
368
 
    parent_invs_cache = LRUInventoryCache(repo)
369
 
    target_git_object_retriever.start_write_group() # FIXME: try/finally
 
405
    trees_cache = LRUTreeCache(repo)
370
406
    # Find and convert commit objects
371
407
    while heads:
372
408
        if pb is not None:
378
414
        except KeyError:
379
415
            continue
380
416
        if isinstance(o, Commit):
381
 
            rev = mapping.import_commit(o)
 
417
            rev = mapping.import_commit(o, lambda x: None)
382
418
            if repo.has_revision(rev.revision_id):
383
419
                continue
384
 
            squash_revision(repo, rev)
385
420
            graph.append((o.id, o.parents))
386
421
            heads.extend([p for p in o.parents if p not in checked])
387
422
        elif isinstance(o, Tag):
393
428
    del checked
394
429
    # Order the revisions
395
430
    # Create the inventory objects
396
 
    batch_size = 100
 
431
    batch_size = 1000
397
432
    revision_ids = topo_sort(graph)
398
433
    pack_hints = []
399
434
    if limit is not None:
400
435
        revision_ids = revision_ids[:limit]
401
436
    last_imported = None
402
437
    for offset in range(0, len(revision_ids), batch_size):
403
 
        repo.start_write_group()
 
438
        target_git_object_retriever.start_write_group() 
404
439
        try:
405
 
            for i, head in enumerate(revision_ids[offset:offset+batch_size]):
406
 
                if pb is not None:
407
 
                    pb.update("fetching revisions", offset+i, len(revision_ids))
408
 
                import_git_commit(repo, mapping, head, lookup_object,
409
 
                                  target_git_object_retriever,
410
 
                                  parent_invs_cache)
411
 
                last_imported = head
 
440
            repo.start_write_group()
 
441
            try:
 
442
                for i, head in enumerate(
 
443
                    revision_ids[offset:offset+batch_size]):
 
444
                    if pb is not None:
 
445
                        pb.update("fetching revisions", offset+i,
 
446
                                  len(revision_ids))
 
447
                    import_git_commit(repo, mapping, head, lookup_object,
 
448
                        target_git_object_retriever, trees_cache)
 
449
                    last_imported = head
 
450
            except:
 
451
                repo.abort_write_group()
 
452
                raise
 
453
            else:
 
454
                hint = repo.commit_write_group()
 
455
                if hint is not None:
 
456
                    pack_hints.extend(hint)
412
457
        except:
413
 
            repo.abort_write_group()
 
458
            target_git_object_retriever.abort_write_group()
414
459
            raise
415
460
        else:
416
 
            hint = repo.commit_write_group()
417
 
            if hint is not None:
418
 
                pack_hints.extend(hint)
419
 
    target_git_object_retriever.commit_write_group()
 
461
            target_git_object_retriever.commit_write_group()
420
462
    return pack_hints, last_imported
421
463
 
422
464
 
432
474
        """See InterRepository.copy_content."""
433
475
        self.fetch(revision_id, pb, find_ghosts=False)
434
476
 
435
 
    def fetch(self, revision_id=None, pb=None, find_ghosts=False,
436
 
        mapping=None, fetch_spec=None):
437
 
        self.fetch_refs(revision_id=revision_id, pb=pb,
438
 
            find_ghosts=find_ghosts, mapping=mapping, fetch_spec=fetch_spec)
439
 
 
440
477
 
441
478
class InterGitNonGitRepository(InterGitRepository):
442
479
    """Base InterRepository that copies revisions from a Git into a non-Git
443
480
    repository."""
444
481
 
445
 
    def fetch_refs(self, revision_id=None, pb=None, find_ghosts=False,
 
482
    def fetch_objects(self, determine_wants, mapping, pb=None, limit=None):
 
483
        """Fetch objects from a remote server.
 
484
 
 
485
        :param determine_wants: determine_wants callback
 
486
        :param mapping: BzrGitMapping to use
 
487
        :param pb: Optional progress bar
 
488
        :param limit: Maximum number of commits to import.
 
489
        :return: Tuple with pack hint, last imported revision id and remote refs
 
490
        """
 
491
        raise NotImplementedError(self.fetch_objects)
 
492
 
 
493
    def fetch(self, revision_id=None, pb=None, find_ghosts=False,
446
494
              mapping=None, fetch_spec=None):
447
495
        if mapping is None:
448
496
            mapping = self.source.get_mapping()
452
500
            interesting_heads = fetch_spec.heads
453
501
        else:
454
502
            interesting_heads = None
455
 
        self._refs = {}
456
503
        def determine_wants(refs):
457
 
            self._refs = refs
458
504
            if interesting_heads is None:
459
505
                ret = [sha for (ref, sha) in refs.iteritems() if not ref.endswith("^{}")]
460
506
            else:
461
 
                ret = [mapping.revision_id_bzr_to_foreign(revid)[0] for revid in interesting_heads if revid not in (None, NULL_REVISION)]
462
 
            return [rev for rev in ret if not self.target.has_revision(mapping.revision_id_foreign_to_bzr(rev))]
463
 
        (pack_hint, _) = self.fetch_objects(determine_wants, mapping, pb)
 
507
                ret = [self.source.lookup_bzr_revision_id(revid)[0] for revid in interesting_heads if revid not in (None, NULL_REVISION)]
 
508
            return [rev for rev in ret if not self.target.has_revision(self.source.lookup_foreign_revision_id(rev))]
 
509
        (pack_hint, _, remote_refs) = self.fetch_objects(determine_wants, mapping, pb)
464
510
        if pack_hint is not None and self.target._format.pack_compresses:
465
511
            self.target.pack(hint=pack_hint)
466
 
        if interesting_heads is not None:
467
 
            present_interesting_heads = self.target.has_revisions(interesting_heads)
468
 
            missing_interesting_heads = set(interesting_heads) - present_interesting_heads
469
 
            if missing_interesting_heads:
470
 
                raise AssertionError("Missing interesting heads: %r" % missing_interesting_heads)
471
 
        return self._refs
 
512
        return remote_refs
472
513
 
473
514
 
474
515
_GIT_PROGRESS_RE = re.compile(r"(.*?): +(\d+)% \((\d+)/(\d+)\)")
482
523
        pb.update(text, 0, 0)
483
524
 
484
525
 
 
526
class DetermineWantsRecorder(object):
 
527
 
 
528
    def __init__(self, actual):
 
529
        self.actual = actual
 
530
        self.wants = []
 
531
        self.remote_refs = {}
 
532
 
 
533
    def __call__(self, refs):
 
534
        self.remote_refs = refs
 
535
        self.wants = self.actual(refs)
 
536
        return self.wants
 
537
 
 
538
 
485
539
class InterRemoteGitNonGitRepository(InterGitNonGitRepository):
486
540
    """InterRepository that copies revisions from a remote Git into a non-Git
487
541
    repository."""
495
549
        return set(all_revs) - all_parents
496
550
 
497
551
    def fetch_objects(self, determine_wants, mapping, pb=None, limit=None):
 
552
        """See `InterGitNonGitRepository`."""
498
553
        def progress(text):
499
554
            report_git_progress(pb, text)
500
555
        store = BazaarObjectStore(self.target, mapping)
503
558
            heads = self.get_target_heads()
504
559
            graph_walker = store.get_graph_walker(
505
560
                    [store._lookup_revision_sha1(head) for head in heads])
506
 
            recorded_wants = []
507
 
 
508
 
            def record_determine_wants(heads):
509
 
                wants = determine_wants(heads)
510
 
                recorded_wants.extend(wants)
511
 
                return wants
 
561
            wants_recorder = DetermineWantsRecorder(determine_wants)
512
562
 
513
563
            create_pb = None
514
564
            if pb is None:
515
565
                create_pb = pb = ui.ui_factory.nested_progress_bar()
516
566
            try:
517
567
                objects_iter = self.source.fetch_objects(
518
 
                            record_determine_wants, graph_walker,
519
 
                            store.get_raw, progress)
520
 
                return import_git_objects(self.target, mapping,
521
 
                    objects_iter, store, recorded_wants, pb, limit)
 
568
                    wants_recorder, graph_walker, store.get_raw,
 
569
                    progress)
 
570
                (pack_hint, last_rev) = import_git_objects(self.target, mapping,
 
571
                    objects_iter, store, wants_recorder.wants, pb, limit)
 
572
                return (pack_hint, last_rev, wants_recorder.remote_refs)
522
573
            finally:
523
574
                if create_pb:
524
575
                    create_pb.finished()
528
579
    @staticmethod
529
580
    def is_compatible(source, target):
530
581
        """Be compatible with GitRepository."""
531
 
        # FIXME: Also check target uses VersionedFile
532
582
        return (isinstance(source, RemoteGitRepository) and
533
583
                target.supports_rich_root() and
534
 
                not isinstance(target, GitRepository))
 
584
                not isinstance(target, GitRepository) and
 
585
                target.texts is not None)
535
586
 
536
587
 
537
588
class InterLocalGitNonGitRepository(InterGitNonGitRepository):
539
590
    repository."""
540
591
 
541
592
    def fetch_objects(self, determine_wants, mapping, pb=None, limit=None):
542
 
        """Fetch objects.
543
 
        """
544
 
        wants = determine_wants(self.source._git.get_refs())
 
593
        """See `InterGitNonGitRepository`."""
 
594
        remote_refs = self.source._git.get_refs()
 
595
        wants = determine_wants(remote_refs)
545
596
        create_pb = None
546
597
        if pb is None:
547
598
            create_pb = pb = ui.ui_factory.nested_progress_bar()
549
600
        try:
550
601
            self.target.lock_write()
551
602
            try:
552
 
                return import_git_objects(self.target, mapping,
 
603
                (pack_hint, last_rev) = import_git_objects(self.target, mapping,
553
604
                    self.source._git.object_store,
554
605
                    target_git_object_retriever, wants, pb, limit)
 
606
                return (pack_hint, last_rev, remote_refs)
555
607
            finally:
556
608
                self.target.unlock()
557
609
        finally:
561
613
    @staticmethod
562
614
    def is_compatible(source, target):
563
615
        """Be compatible with GitRepository."""
564
 
        # FIXME: Also check target uses VersionedFile
565
616
        return (isinstance(source, LocalGitRepository) and
566
617
                target.supports_rich_root() and
567
 
                not isinstance(target, GitRepository))
 
618
                not isinstance(target, GitRepository) and
 
619
                target.texts is not None)
568
620
 
569
621
 
570
622
class InterGitGitRepository(InterGitRepository):
576
628
        graphwalker = self.target._git.get_graph_walker()
577
629
        if (isinstance(self.source, LocalGitRepository) and
578
630
            isinstance(self.target, LocalGitRepository)):
579
 
            return self.source._git.fetch(self.target._git, determine_wants,
 
631
            refs = self.source._git.fetch(self.target._git, determine_wants,
580
632
                progress)
 
633
            return (None, None, refs)
581
634
        elif (isinstance(self.source, LocalGitRepository) and
582
635
              isinstance(self.target, RemoteGitRepository)):
583
636
            raise NotImplementedError
588
641
                refs = self.source._git.fetch_pack(determine_wants,
589
642
                    graphwalker, f.write, progress)
590
643
                commit()
591
 
                return refs
 
644
                return (None, None, refs)
592
645
            except:
593
646
                f.close()
594
647
                raise
595
648
        else:
596
649
            raise AssertionError
597
650
 
598
 
    def fetch_refs(self, revision_id=None, pb=None, find_ghosts=False,
 
651
    def fetch(self, revision_id=None, pb=None, find_ghosts=False,
599
652
              mapping=None, fetch_spec=None, branches=None):
600
653
        if mapping is None:
601
654
            mapping = self.source.get_mapping()
610
663
            determine_wants = r.object_store.determine_wants_all
611
664
        else:
612
665
            determine_wants = lambda x: [y for y in args if not y in r.object_store]
613
 
        return self.fetch_objects(determine_wants, mapping)[0]
614
 
 
 
666
        self.fetch_objects(determine_wants, mapping)
615
667
 
616
668
    @staticmethod
617
669
    def is_compatible(source, target):