2009/8/23 Fabio Tranchitella <kob...@kobold.it>:

> * 2009-08-21 21:15, Dan Korostelev wrote:
>> > What about removing the use of zope.deferredimport from zope.location and
>> > make a new major release of zope.location?
>> +1, just change that to plain imports with # BBB comments.
> Would it also be possible to remove the dependency of zope.location on
> zope.copy? It is only needed to register an adapter for ICopyHook, which
> can be made optional in the ZCML.
> To me it looks a wise choice, because zope.location is well usable without
> zope.copy, and if somebody needs the hook it is because he will be using
> zope.copy, so he must depend on it.
> I prepared these changes in a branch, can somebody review them?

It seems somewhat strange to have a module that imports from a package
that is not in dependencies. Also, zope.copy is a very tiny module
with no dependencies beyond zope.interface and no C code, so I don't
see any problem with having it in dependencies. However, any option is
fine with me, so +0 :-)

BTW probably it's better to isolate the copy hook adapter in a
submodule with more sane name, like "copyhook" and deprecate the
"pickling" module, so people could understand those things easier. :-)

WBR, Dan Korostelev
Zope-Dev maillist  -  Zope-Dev@zope.org
**  No cross posts or HTML encoding!  **
(Related lists - 
 http://mail.zope.org/mailman/listinfo/zope )

Reply via email to