Rev 5122: (vila) Merge 2.1 into 2.2 for lp:707075, and add NEWS entry. (Max Bowsher) in file:///home/pqm/archives/thelove/bzr/2.2/

Canonical.com Patch Queue Manager pqm at pqm.ubuntu.com
Wed Feb 2 12:10:55 UTC 2011


At file:///home/pqm/archives/thelove/bzr/2.2/

------------------------------------------------------------
revno: 5122 [merge]
revision-id: pqm at pqm.ubuntu.com-20110202121053-3g4epxba6ai3m9ya
parent: pqm at pqm.ubuntu.com-20110121013617-5ewl2eejpl0lnsxz
parent: maxb at f2s.com-20110202104822-6mo5qdq1l55d6bv1
committer: Canonical.com Patch Queue Manager <pqm at pqm.ubuntu.com>
branch nick: 2.2
timestamp: Wed 2011-02-02 12:10:53 +0000
message:
  (vila) Merge 2.1 into 2.2 for lp:707075, and add NEWS entry. (Max Bowsher)
modified:
  NEWS                           NEWS-20050323055033-4e00b5db738777ff
  bzrlib/plugins/launchpad/lp_api.py lp_api.py-20090704082908-79il6zl4gugwl3wz-1
=== modified file 'NEWS'
--- a/NEWS	2011-01-20 21:40:17 +0000
+++ b/NEWS	2011-02-02 10:48:22 +0000
@@ -19,6 +19,12 @@
 Bug Fixes
 *********
 
+* Fix communications with the Launchpad web service when using
+  launchpadlib >= 1.5.5.  This was a latent bug in bzr's communication
+  with Launchpad's production instance, which only became a problem when
+  the default instance was switched from edge to production in bzr 2.2.3.
+  (Max Bowsher, #707075)
+
 Improvements
 ************
 

=== modified file 'bzrlib/plugins/launchpad/lp_api.py'
--- a/bzrlib/plugins/launchpad/lp_api.py	2010-11-26 18:13:30 +0000
+++ b/bzrlib/plugins/launchpad/lp_api.py	2011-02-02 09:43:08 +0000
@@ -73,10 +73,25 @@
             installed_version, installed_version)
 
 
+# The older versions of launchpadlib only provided service root constants for
+# edge and staging, whilst newer versions drop edge. Therefore service root
+# URIs for which we do not always have constants are derived from the staging
+# one, which does always exist.
+#
+# It is necessary to derive, rather than use hardcoded URIs because
+# launchpadlib <= 1.5.4 requires service root URIs that end in a path of
+# /beta/, whilst launchpadlib >= 1.5.5 requires service root URIs with no path
+# info.
+#
+# Once we have a hard dependency on launchpadlib >= 1.5.4 we can replace all of
+# bzr's local knowledge of individual Launchpad instances with use of the
+# launchpadlib.uris module.
 LAUNCHPAD_API_URLS = {
-    'production': 'https://api.launchpad.net/beta/',
+    'production': STAGING_SERVICE_ROOT.replace('api.staging.launchpad.net',
+        'api.launchpad.net'),
     'staging': STAGING_SERVICE_ROOT,
-    'dev': 'https://api.launchpad.dev/beta/',
+    'dev': STAGING_SERVICE_ROOT.replace('api.staging.launchpad.net',
+        'api.launchpad.dev'),
     }
 
 




More information about the bazaar-commits mailing list