Bazaar

Merge lp:~lifeless/bzr/bug-422849 into lp:bzr/2.0

  1. bug-422849
  2. Merge into 2.0
Proposed by Robert Collins
Status: Merged
Merged at revision: not available
Proposed branch: lp:~lifeless/bzr/bug-422849
Merge into: lp:bzr/2.0
Diff against target: 38 lines
To merge this branch: bzr merge lp:~lifeless/bzr/bug-422849
Reviewer Review Type Date Requested Status
John A Meinel Approve
Review via email: mp+11023@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Robert Collins (lifeless) wrote :

Bug fix for 422849, no tests as yet. Should pass PQM as the error is solely an aliasing bug.

Revision history for this message
Martin Pool (mbp) wrote :

> Bug fix for 422849, no tests as yet. Should pass PQM as the error is solely an
> aliasing bug.

In interactive testing this seems to be fixing the conversion that previously failed for me, but it hasn't finished yet.

Revision history for this message
John A Meinel (jameinel) wrote :

Of course, it would be nice to have a test here, but getting the fix in prevents people from borking their codebase.

review: Approve
Revision history for this message
Martin Pool (mbp) wrote :

> In interactive testing this seems to be fixing the conversion that previously
> failed for me, but it hasn't finished yet.

It did complete and I'm now checking it.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'NEWS'
--- NEWS 2009-08-31 00:25:33 +0000
+++ NEWS 2009-09-02 00:35:10 +0000
@@ -24,6 +24,10 @@
24 that has a ghost in the mainline ancestry.24 that has a ghost in the mainline ancestry.
25 (John Arbash Meinel, #419241)25 (John Arbash Meinel, #419241)
2626
27* Local data conversion will generate correct deltas. This is a critical
28 bugfix vs 2.0rc1, and all 2.0rc1 users should upgrade to 2.0rc2 before
29 converting repositories. (Robert Collins, #422849)
30
27Documentation31Documentation
28*************32*************
2933
3034
=== modified file 'bzrlib/repository.py'
--- bzrlib/repository.py 2009-08-30 22:02:45 +0000
+++ bzrlib/repository.py 2009-09-02 00:35:10 +0000
@@ -3844,6 +3844,9 @@
3844 possible_trees.append((basis_id, cache[basis_id]))3844 possible_trees.append((basis_id, cache[basis_id]))
3845 basis_id, delta = self._get_delta_for_revision(tree, parent_ids,3845 basis_id, delta = self._get_delta_for_revision(tree, parent_ids,
3846 possible_trees)3846 possible_trees)
3847 revision = self.source.get_revision(current_revision_id)
3848 pending_deltas.append((basis_id, delta,
3849 current_revision_id, revision.parent_ids))
3847 if self._converting_to_rich_root:3850 if self._converting_to_rich_root:
3848 self._revision_id_to_root_id[current_revision_id] = \3851 self._revision_id_to_root_id[current_revision_id] = \
3849 tree.get_root_id()3852 tree.get_root_id()
@@ -3878,9 +3881,6 @@
3878 if entry.revision == file_revision:3881 if entry.revision == file_revision:
3879 texts_possibly_new_in_tree.remove(file_key)3882 texts_possibly_new_in_tree.remove(file_key)
3880 text_keys.update(texts_possibly_new_in_tree)3883 text_keys.update(texts_possibly_new_in_tree)
3881 revision = self.source.get_revision(current_revision_id)
3882 pending_deltas.append((basis_id, delta,
3883 current_revision_id, revision.parent_ids))
3884 pending_revisions.append(revision)3884 pending_revisions.append(revision)
3885 cache[current_revision_id] = tree3885 cache[current_revision_id] = tree
3886 basis_id = current_revision_id3886 basis_id = current_revision_id

Subscribers

People subscribed via source and target branches