A couple of questions about (maybe) missing dependencies
Hello, While analyzing the dependency graph of one of our applications, I found some missing dependencies. I'm not sure these are bugs, but I'd like to ask your opinion about them: - zope.app.publisher does not depend anymore on zope.app.pagetemplate, but it uses it in src/zope/app/publisher/browser/viewmeta.py
zope.app.pagetemplate.simpleviewclass import SimpleViewClass
- shouldn't zope.pagetemplate depend on zope.security [untrustedpython]? it imports it in engine.py:
from zope.security.untrustedpython import rcompile
- I think we should release zope.location (added miss dependency on zope.deferredimport, it is ok in trunk but not yet released); - I think we should release zope.sendmail (not it depends on transaction instead of ZODB3, it is ok in trunk but not yet released). Thanks. -- Fabio Tranchitella http://www.kobold.it Free Software Developer and Consultant http://www.tranchitella.it _____________________________________________________________________ 1024D/7F961564, fpr 5465 6E69 E559 6466 BF3D 9F01 2BF8 EE2B 7F96 1564
Hi there, Fabio Tranchitella wrote:
While analyzing the dependency graph of one of our applications, I found some missing dependencies. I'm not sure these are bugs, but I'd like to ask your opinion about them:
- zope.app.publisher does not depend anymore on zope.app.pagetemplate, but it uses it in src/zope/app/publisher/browser/viewmeta.py
zope.app.pagetemplate.simpleviewclass import SimpleViewClass
Seems like a bug at first glance. Feel free to add the dependency back (though I hope we can look into removing it again).
- shouldn't zope.pagetemplate depend on zope.security [untrustedpython]? it imports it in engine.py:
from zope.security.untrustedpython import rcompile
The same as above, seems like a bug, feel free to add the dependency back, better yet if we can devise a way to really get rid of it.
- I think we should release zope.location (added miss dependency on zope.deferredimport, it is ok in trunk but not yet released);
+1
- I think we should release zope.sendmail (not it depends on transaction instead of ZODB3, it is ok in trunk but not yet released).
+1 Regards, Martijn
Hello, * 2009-06-29 10:25, Martijn Faassen wrote:
Seems like a bug at first glance. Feel free to add the dependency back (though I hope we can look into removing it again).
I would love to, but I don't have (yet) commit rights on svn.zope.org. :) Thanks, Fabio
participants (2)
-
Fabio Tranchitella -
Martijn Faassen