nadako at gmail
Aug 22, 2009, 1:35 PM
Post #4 of 4
(370 views)
Permalink
|
2009/8/23 Fabio Tranchitella <kobold[at]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[at]zope.org
http://mail.zope.org/mailman/listinfo/zope-dev
** No cross posts or HTML encoding! **
(Related lists -
http://mail.zope.org/mailman/listinfo/zope-announce
http://mail.zope.org/mailman/listinfo/zope )
|
