The README.txt of zc.copy says that the components, provided by this package is apropriate for inclusion in Zope itself. The package provides a more pluggable mechanism for copying generic persistent objects (not only ILocation's) as well as a way to register post-copy hooks to be executed, which is very useful, for example when dealing with persistent objects that have non-ZODB bits (like filesystem based files related and so on). However, the package is really small and mostly contains modified copies of zope.copypastemove's ObjectCopier and zope.location's locationCopy. So, I propose to merge the zc.copy package's changes with original zope.copypastemove/zope.location and deprecate it. If noone objects, I'll do that myself. -- WBR, Dan Korostelev
After looking at the whole copy thing for some more time, I thought that it even makes sense to extract the object cloning functionality to some "zope.copy" (or even "zope.persistentcopy") package that will contain clone and copy functions as well as ICopyHook mechanism, but won't contain IObjectCopier implementation, because the "clone" and "copy" functions from zc.copy are useful without any container context. Then zope.copypastemove and zope.location could just depend on the zope.copy, so we won't introduce many dependencies for zope.location and make people able to easily copy persistent objects w/o installing on zope.copypastemove or even zope.location. 2009/2/8 Dan Korostelev <nadako@gmail.com>:
The README.txt of zc.copy says that the components, provided by this package is apropriate for inclusion in Zope itself.
The package provides a more pluggable mechanism for copying generic persistent objects (not only ILocation's) as well as a way to register post-copy hooks to be executed, which is very useful, for example when dealing with persistent objects that have non-ZODB bits (like filesystem based files related and so on). However, the package is really small and mostly contains modified copies of zope.copypastemove's ObjectCopier and zope.location's locationCopy.
So, I propose to merge the zc.copy package's changes with original zope.copypastemove/zope.location and deprecate it. If noone objects, I'll do that myself.
-- WBR, Dan Korostelev
-- WBR, Dan Korostelev
Okay. I prepared the "zope.copy" package in the SVN for you to check out what I mean. :) 2009/2/8 Dan Korostelev <nadako@gmail.com>:
After looking at the whole copy thing for some more time, I thought that it even makes sense to extract the object cloning functionality to some "zope.copy" (or even "zope.persistentcopy") package that will contain clone and copy functions as well as ICopyHook mechanism, but won't contain IObjectCopier implementation, because the "clone" and "copy" functions from zc.copy are useful without any container context. Then zope.copypastemove and zope.location could just depend on the zope.copy, so we won't introduce many dependencies for zope.location and make people able to easily copy persistent objects w/o installing on zope.copypastemove or even zope.location.
2009/2/8 Dan Korostelev <nadako@gmail.com>:
The README.txt of zc.copy says that the components, provided by this package is apropriate for inclusion in Zope itself.
The package provides a more pluggable mechanism for copying generic persistent objects (not only ILocation's) as well as a way to register post-copy hooks to be executed, which is very useful, for example when dealing with persistent objects that have non-ZODB bits (like filesystem based files related and so on). However, the package is really small and mostly contains modified copies of zope.copypastemove's ObjectCopier and zope.location's locationCopy.
So, I propose to merge the zc.copy package's changes with original zope.copypastemove/zope.location and deprecate it. If noone objects, I'll do that myself.
-- WBR, Dan Korostelev
-- WBR, Dan Korostelev
-- WBR, Dan Korostelev
As the author, +1 and thank you! Gary On Feb 8, 2009, at 11:18 AM, Dan Korostelev wrote:
Okay. I prepared the "zope.copy" package in the SVN for you to check out what I mean. :)
2009/2/8 Dan Korostelev <nadako@gmail.com>:
After looking at the whole copy thing for some more time, I thought that it even makes sense to extract the object cloning functionality to some "zope.copy" (or even "zope.persistentcopy") package that will contain clone and copy functions as well as ICopyHook mechanism, but won't contain IObjectCopier implementation, because the "clone" and "copy" functions from zc.copy are useful without any container context. Then zope.copypastemove and zope.location could just depend on the zope.copy, so we won't introduce many dependencies for zope.location and make people able to easily copy persistent objects w/o installing on zope.copypastemove or even zope.location.
2009/2/8 Dan Korostelev <nadako@gmail.com>:
The README.txt of zc.copy says that the components, provided by this package is apropriate for inclusion in Zope itself.
The package provides a more pluggable mechanism for copying generic persistent objects (not only ILocation's) as well as a way to register post-copy hooks to be executed, which is very useful, for example when dealing with persistent objects that have non-ZODB bits (like filesystem based files related and so on). However, the package is really small and mostly contains modified copies of zope.copypastemove's ObjectCopier and zope.location's locationCopy.
So, I propose to merge the zc.copy package's changes with original zope.copypastemove/zope.location and deprecate it. If noone objects, I'll do that myself.
-- WBR, Dan Korostelev
-- WBR, Dan Korostelev
-- WBR, Dan Korostelev _______________________________________________ Zope-Dev maillist - Zope-Dev@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 )
2009/2/9 Gary Poster <gary.poster@gmail.com>:
As the author, +1 and thank you!
Glad to hear. I'll release the result of the merge soon. BTW, I'd also like to make a final release of zc.copy, replacing its code with dependencies/imports from newer zope.copy and zope.copypastemove. Can you please add me to package owners on PyPI (my name there is "nadako"). Also, should I use deprecated deferred imports or plain imports for that? -- WBR, Dan Korostelev
On Feb 8, 2009, at 5:18 PM, Dan Korostelev wrote:
2009/2/9 Gary Poster <gary.poster@gmail.com>:
As the author, +1 and thank you!
Glad to hear. I'll release the result of the merge soon.
BTW, I'd also like to make a final release of zc.copy, replacing its code with dependencies/imports from newer zope.copy and zope.copypastemove.
You mean 1.1. OK. If I were doing it, I would make 1.1b -> 1.1, and then make a 1.2 that was as you described. If you don't agree or feel like it that's fine.
Can you please add me to package owners on PyPI (my name there is "nadako").
Done.
Also, should I use deprecated deferred imports or plain imports for that?
I'm generally not a fan of the deprecated imports. I'd say just put it in the docs. Again, if you disagree, I don't feel too strongly about it. Gary
participants (2)
-
Dan Korostelev -
Gary Poster