/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

Add tests for revspec.

Show diffs side-by-side

added added

removed removed

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