Merge lp:~bac/launchpad/bug-604488 into lp:launchpad

Proposed by Brad Crittenden
Status: Merged
Approved by: Curtis Hovey
Approved revision: no longer in the source branch.
Merged at revision: 11119
Proposed branch: lp:~bac/launchpad/bug-604488
Merge into: lp:launchpad
Diff against target: 40 lines (+4/-3)
3 files modified
lib/canonical/configure.zcml (+2/-1)
lib/lp/services/salesforce/configure.zcml (+1/-1)
lib/lp/services/salesforce/interfaces.py (+1/-1)
To merge this branch: bzr merge lp:~bac/launchpad/bug-604488
Reviewer Review Type Date Requested Status
Curtis Hovey (community) Approve
Review via email: mp+29675@code.launchpad.net

Commit message

Register missing zcml for production.

Description of the change

= Summary =

Refactoring of salesforce proxy code created a new configure.zcml file
that was not registered.

== Proposed fix ==

Add registration information to lib/canonical/configure.zcml

== Pre-implementation notes ==

Talk with Curtis.

== Implementation details ==

As above.

== Tests ==

None, as all testing uses zcml overrides and therefore didn't fail as
they were handled properly.

== Demo and Q/A ==

Visit https://launchpad.net/people/+me/+vouchers.

= Launchpad lint =

Checking for conflicts and issues in changed files.

Linting changed files:
  lib/canonical/configure.zcml
  lib/lp/services/salesforce/configure.zcml
  lib/lp/services/salesforce/interfaces.py

To post a comment you must log in.
Revision history for this message
Curtis Hovey (sinzui) wrote :

Thanks for identifying the problem. The test you demonstrated from make harness convinced me that this is the correct fix.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lib/canonical/configure.zcml'
--- lib/canonical/configure.zcml 2010-03-24 15:22:26 +0000
+++ lib/canonical/configure.zcml 2010-07-12 08:41:09 +0000
@@ -22,6 +22,7 @@
22 <include package="lp.services.memcache" />22 <include package="lp.services.memcache" />
23 <include package="lp.services.scripts" />23 <include package="lp.services.scripts" />
24 <include package="lp.services.worlddata" />24 <include package="lp.services.worlddata" />
25 <include package="lp.services.salesforce" />
25 <include package="lazr.uri" />26 <include package="lazr.uri" />
26 <include package="canonical.librarian" />27 <include package="canonical.librarian" />
2728
@@ -75,7 +76,7 @@
75 virtual host layers. The directives come in pairs.76 virtual host layers. The directives come in pairs.
76 The separate registration for the resources namespace (@@) is needed77 The separate registration for the resources namespace (@@) is needed
77 because otherwise the lookup for /@@/ will fail because the78 because otherwise the lookup for /@@/ will fail because the
78 layer-specific defaultView directive also registers the 79 layer-specific defaultView directive also registers the
79 default view name as an unnamed adapter.80 default view name as an unnamed adapter.
80 -->81 -->
81 -->82 -->
8283
=== modified file 'lib/lp/services/salesforce/configure.zcml'
--- lib/lp/services/salesforce/configure.zcml 2010-07-07 19:41:07 +0000
+++ lib/lp/services/salesforce/configure.zcml 2010-07-12 08:41:09 +0000
@@ -1,4 +1,4 @@
1<!-- Copyright 2009 Canonical Ltd. This software is licensed under the1<!-- Copyright 2010 Canonical Ltd. This software is licensed under the
2 GNU Affero General Public License version 3 (see the file LICENSE).2 GNU Affero General Public License version 3 (see the file LICENSE).
3-->3-->
44
55
=== modified file 'lib/lp/services/salesforce/interfaces.py'
--- lib/lp/services/salesforce/interfaces.py 2010-07-07 19:41:07 +0000
+++ lib/lp/services/salesforce/interfaces.py 2010-07-12 08:41:09 +0000
@@ -1,4 +1,4 @@
1# Copyright 2009 Canonical Ltd. This software is licensed under the1# Copyright 2010 Canonical Ltd. This software is licensed under the
2# GNU Affero General Public License version 3 (see the file LICENSE).2# GNU Affero General Public License version 3 (see the file LICENSE).
33
4# pylint: disable-msg=E0211,E02134# pylint: disable-msg=E0211,E0213