Launchpad itself

Merge ~cjwatson/launchpad:remove-zope.securitypolicy into launchpad:master

  1. Git
  2. lp:~cjwatson/launchpad
  3. remove-zope.securitypolicy
  4. Merge into master
Proposed by Colin Watson
Status: Merged
Approved by: Colin Watson
Approved revision: 102e6be7f6ad613056e4df025d00c866b2d453cd
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: ~cjwatson/launchpad:remove-zope.securitypolicy
Merge into: launchpad:master
Diff against target: 24 lines (+0/-2)
2 files modified
setup.cfg (+0/-1)
zcml/zopeapp.zcml (+0/-1)
Reviewer Review Type Date Requested Status
Jürgen Gmach Approve
Review via email: mp+442957@code.launchpad.net

Commit message

Remove zope.securitypolicy

Description of the change

When looking over our Zope dependencies, I happened to notice that we don't really use `zope.securitypolicy`: its `meta.zcml` registers four ZCML directives, none of which we use, and we don't import anything from it anywhere else. We can just drop it.

To post a comment you must log in.
Revision history for this message
Jürgen Gmach (jugmac00) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/setup.cfg b/setup.cfg
2index be574c6..372b8f8 100644
3--- a/setup.cfg
4+++ b/setup.cfg
5@@ -154,7 +154,6 @@ install_requires =
6 zope.publisher
7 zope.schema
8 zope.security
9- zope.securitypolicy
10 zope.sendmail
11 zope.tal
12 zope.tales
13diff --git a/zcml/zopeapp.zcml b/zcml/zopeapp.zcml
14index 16cc755..212166b 100644
15--- a/zcml/zopeapp.zcml
16+++ b/zcml/zopeapp.zcml
17@@ -14,7 +14,6 @@
18 <include package="zope.publisher" file="meta.zcml" />
19 <include package="zope.browsermenu" file="meta.zcml" />
20 <include package="zope.browserpage" file="meta.zcml" />
21- <include package="zope.securitypolicy" file="meta.zcml" />
22 <include package="zope.sendmail" file="meta.zcml" />
23 <include package="zope.app.publication" file="meta.zcml" />
24 <include package="zope.app.publisher.xmlrpc" file="meta.zcml" />

Subscribers

People subscribed via source and target branches

to status/vote changes: