← Back to team overview

launchpad-reviewers team mailing list archive

[Merge] lp:~abentley/launchpad/new-bzr into lp:launchpad/devel

 

Aaron Bentley has proposed merging lp:~abentley/launchpad/new-bzr into lp:launchpad/devel.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)


= Summary =
Switch to bzr 2.2b4

== Proposed fix ==
None

== Pre-implementation notes ==
Thumper requested this

== Implementation details ==
There were some minor API changes, and the import and loom plugins needed
updates.  Unfortunately, bzr-pqm is not handling merges very well because of
branch format differences, so I had to switch to loom trunk temporarily.  I
will switch to a launchpad-pqm branch before landing.

I had to change the DirectBranchCommit handling because apparently unstacking
the branch has the side-effect of unlocking its repository.  But I think the
revised code is cleaner anyhow.

I disabled documentation of bzrlib because of bad interactions between the
documentation system and bzr's lazy imports.

== Tests ==
bin/test -v

== Demo and Q/A ==
None


= Launchpad lint =

Checking for conflicts and issues in changed files.

Linting changed files:
  lib/lp/code/mail/codehandler.py
  lib/lp/codehosting/vfs/tests/test_transport.py
  versions.cfg
  lib/lp/code/model/diff.py
  lib/lp/codehosting/tests/test_acceptance.py
  lib/lp/codehosting/tests/test_bzrutils.py
  lib/lp/translations/scripts/translations_to_branch.py
  lib/lp/codehosting/codeimport/tests/test_worker.py
  utilities/sourcedeps.conf
  configs/development/apidoc-configure-normal.zcml
  lib/lp/codehosting/puller/worker.py

./lib/lp/code/mail/codehandler.py
      57: E302 expected 2 blank lines, found 1
      60: E302 expected 2 blank lines, found 1
      63: E302 expected 2 blank lines, found 1
     521: Line exceeds 78 characters.
./lib/lp/code/model/diff.py
      13: 'Branch' imported but unused
     144: E301 expected 1 blank line, found 0
./lib/lp/codehosting/tests/test_acceptance.py
     148: E202 whitespace before '}'
./lib/lp/codehosting/tests/test_bzrutils.py
     154: E301 expected 1 blank line, found 0
     166: E301 expected 1 blank line, found 0
     210: E302 expected 2 blank lines, found 3
     263: E301 expected 1 blank line, found 0
./lib/lp/codehosting/codeimport/tests/test_worker.py
      91: E301 expected 1 blank line, found 0
     339: E231 missing whitespace after ','
     348: E231 missing whitespace after ','
     359: E231 missing whitespace after ','
     360: E231 missing whitespace after ','
     373: E231 missing whitespace after ','
     378: E231 missing whitespace after ','
     388: E231 missing whitespace after ','
     395: E231 missing whitespace after ','
     403: E231 missing whitespace after ','
     412: E231 missing whitespace after ','
     417: E231 missing whitespace after ','
     430: E231 missing whitespace after ','
     440: E231 missing whitespace after ','
     445: E302 expected 2 blank lines, found 3
     478: E301 expected 1 blank line, found 0
     618: E301 expected 1 blank line, found 0
     818: E203 whitespace before ':'
    1000: E302 expected 2 blank lines, found 3
    1056: E302 expected 2 blank lines, found 3
./lib/lp/codehosting/puller/worker.py
     444: E231 missing whitespace after ','
     507: E231 missing whitespace after ','
-- 
https://code.launchpad.net/~abentley/launchpad/new-bzr/+merge/31888
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~abentley/launchpad/new-bzr into lp:launchpad/devel.
=== modified file 'configs/development/apidoc-configure-normal.zcml'
--- configs/development/apidoc-configure-normal.zcml	2010-07-24 02:27:19 +0000
+++ configs/development/apidoc-configure-normal.zcml	2010-08-05 20:47:50 +0000
@@ -90,7 +90,6 @@
     <apidoc:rootModule module="martian" />
     <apidoc:rootModule module="manuel" />
     <apidoc:rootModule module="chameleon" />
-    <apidoc:rootModule module="bzrlib" />
     <apidoc:rootModule module="storm" />
 
     <apidoc:bookchapter

=== modified file 'lib/lp/code/mail/codehandler.py'
--- lib/lp/code/mail/codehandler.py	2010-08-02 02:51:42 +0000
+++ lib/lp/code/mail/codehandler.py	2010-08-05 20:47:50 +0000
@@ -545,8 +545,12 @@
         # access to any needed but not supplied revisions.
         md.target_branch = target_url
         md.install_revisions(bzr_branch.repository)
-        bzr_branch.pull(bzr_branch, stop_revision=md.revision_id,
-                        overwrite=True)
+        bzr_branch.lock_write()
+        try:
+            bzr_branch.pull(bzr_branch, stop_revision=md.revision_id,
+                            overwrite=True)
+        finally:
+            bzr_branch.unlock()
 
     def findMergeDirectiveAndComment(self, message):
         """Extract the comment and Merge Directive from a SignedMessage."""

=== modified file 'lib/lp/code/model/diff.py'
--- lib/lp/code/model/diff.py	2010-08-02 02:13:52 +0000
+++ lib/lp/code/model/diff.py	2010-08-05 20:47:50 +0000
@@ -140,7 +140,7 @@
             source_revision)
         merger = Merge3Merger(
             merge_target, merge_target, merge_base, merge_source,
-            do_merge=False)
+            this_branch=target_branch, do_merge=False)
         def dummy_warning(self, *args, **kwargs):
             pass
         real_warning = trace.warning

=== modified file 'lib/lp/codehosting/codeimport/tests/test_worker.py'
--- lib/lp/codehosting/codeimport/tests/test_worker.py	2010-08-02 23:01:15 +0000
+++ lib/lp/codehosting/codeimport/tests/test_worker.py	2010-08-05 20:47:50 +0000
@@ -952,7 +952,7 @@
         t = get_transport(self.get_url('.'))
         t.mkdir('reference')
         a_bzrdir = BzrDir.create(self.get_url('reference'))
-        BranchReferenceFormat().initialize(a_bzrdir, branch)
+        BranchReferenceFormat().initialize(a_bzrdir, target_branch=branch)
         return a_bzrdir.root_transport.base
 
     def test_reject_branch_reference(self):

=== modified file 'lib/lp/codehosting/puller/worker.py'
--- lib/lp/codehosting/puller/worker.py	2010-04-21 01:56:51 +0000
+++ lib/lp/codehosting/puller/worker.py	2010-08-05 20:47:50 +0000
@@ -503,7 +503,7 @@
     def get_boolean(self, prompt):
         """If we're asked to break a lock like a stale lock of ours, say yes.
         """
-        assert prompt.startswith('Break lock'), (
+        assert prompt.startswith('Break '), (
             "Didn't expect prompt %r" % (prompt,))
         branch_id = self.puller_worker_protocol.branch_id
         if get_lock_id_for_branch_id(branch_id) in prompt:

=== modified file 'lib/lp/codehosting/tests/test_acceptance.py'
--- lib/lp/codehosting/tests/test_acceptance.py	2010-04-19 06:35:23 +0000
+++ lib/lp/codehosting/tests/test_acceptance.py	2010-08-05 20:47:50 +0000
@@ -242,10 +242,7 @@
             creator_id, '/~%s/%s/%s' % (user, product, branch))
         branch_url = 'file://' + os.path.abspath(
             os.path.join(branch_root, branch_id_to_path(branch_id)))
-        self.runInChdir(
-            self.local_branch_path,
-            self.run_bzr, ['push', '--create-prefix', branch_url],
-            retcode=None)
+        self.push(self.local_branch_path, branch_url, ['--create-prefix'])
         return branch_url
 
 

=== modified file 'lib/lp/codehosting/tests/test_bzrutils.py'
--- lib/lp/codehosting/tests/test_bzrutils.py	2010-04-23 01:47:30 +0000
+++ lib/lp/codehosting/tests/test_bzrutils.py	2010-08-05 20:47:50 +0000
@@ -14,7 +14,7 @@
 from bzrlib.branch import Branch, BranchReferenceFormat
 from bzrlib.bzrdir import BzrDir, format_registry
 from bzrlib.remote import RemoteBranch
-from bzrlib.smart import server
+from bzrlib.tests import test_server
 from bzrlib.tests import (
     multiply_tests, TestCase, TestCaseWithTransport, TestLoader,
     TestNotApplicable)
@@ -191,7 +191,7 @@
         # of the branch, repo and bzrdir, even if the branch is a
         # RemoteBranch.
         vfs_branch = self.make_branch('.')
-        smart_server = server.SmartTCPServer_for_testing()
+        smart_server = test_server.SmartTCPServer_for_testing()
         smart_server.start_server(self.get_vfs_only_server())
         self.addCleanup(smart_server.stop_server)
         remote_branch = Branch.open(smart_server.get_url())

=== modified file 'lib/lp/codehosting/vfs/tests/test_transport.py'
--- lib/lp/codehosting/vfs/tests/test_transport.py	2010-04-19 07:05:57 +0000
+++ lib/lp/codehosting/vfs/tests/test_transport.py	2010-08-05 20:47:50 +0000
@@ -43,6 +43,9 @@
             BlockingProxy(branchfs), LocalTransport(local_path_to_url('.')))
         self._chroot_servers = []
 
+    def get_bogus_url(self):
+        return self._scheme + 'bogus'
+
     def _transportFactory(self, url):
         """See `LaunchpadInternalServer._transportFactory`.
 

=== modified file 'lib/lp/translations/scripts/translations_to_branch.py'
--- lib/lp/translations/scripts/translations_to_branch.py	2010-06-16 07:22:57 +0000
+++ lib/lp/translations/scripts/translations_to_branch.py	2010-08-05 20:47:50 +0000
@@ -103,22 +103,16 @@
         # possible again to commit to these branches at some point.
         # When that happens, remove this workaround and just call
         # _makeDirectBranchCommit directly.
-        committer = self._makeDirectBranchCommit(db_branch)
-        if not db_branch.stacked_on:
-            # The normal case.
-            return committer
-
-        self.logger.info("Unstacking branch to work around bug 375013.")
-        try:
-            committer.bzrbranch.set_stacked_on_url(None)
-        finally:
-            committer.unlock()
-        self.logger.info("Done unstacking branch.")
-
-        # This may have taken a while, so commit for good
-        # manners.
-        if self.txn:
-            self.txn.commit()
+        if db_branch.stacked_on:
+            bzrbranch = db_branch.getBzrBranch()
+            self.logger.info("Unstacking branch to work around bug 375013.")
+            bzrbranch.set_stacked_on_url(None)
+            self.logger.info("Done unstacking branch.")
+
+            # This may have taken a while, so commit for good
+            # manners.
+            if self.txn:
+                self.txn.commit()
 
         return self._makeDirectBranchCommit(db_branch)
 

=== modified file 'utilities/sourcedeps.conf'
--- utilities/sourcedeps.conf	2010-08-03 14:59:22 +0000
+++ utilities/sourcedeps.conf	2010-08-05 20:47:50 +0000
@@ -1,7 +1,7 @@
 bzr-builder lp:~launchpad-pqm/bzr-builder/trunk;revno=65
 bzr-git lp:~launchpad-pqm/bzr-git/devel;revno=257
-bzr-hg lp:~launchpad-pqm/bzr-hg/devel;revno=281
-bzr-loom lp:~launchpad-pqm/bzr-loom/trunk;revno=47
+bzr-hg lp:~launchpad-pqm/bzr-hg/devel;revno=282
+bzr-loom lp:bzr-loom;revno=126
 bzr-svn lp:~launchpad-pqm/bzr-svn/devel;revno=2709
 cscvs lp:~launchpad-pqm/launchpad-cscvs/devel;revno=432
 dulwich lp:~launchpad-pqm/dulwich/devel;revno=423

=== modified file 'versions.cfg'
--- versions.cfg	2010-07-26 16:00:01 +0000
+++ versions.cfg	2010-08-05 20:47:50 +0000
@@ -5,7 +5,7 @@
 # Alphabetical, case-insensitive, please! :-)
 
 ampoule = 0.2.0
-bzr = 2.1.0
+bzr = 2.2b4
 chameleon.core = 1.0b35
 chameleon.zpt = 1.0b17
 ClientForm = 0.2.10