[Zope-dev] SVN: zope.component/branches/tseaver-wo_zope_deferred/

[Tres Seaver]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Dan Korostelev wrote:
> 2009/3/4 Tres Seaver <tseaver at palladion.com>:
>> - - Due to the 'test' extra, buildout pulls in a bunch of extra
>>  dependencies, which I would like to zap (ZODB?  really?  just to
>>  verify that the persistent registry survives 'dumps' and 'loads'?)
>>
>> - - 'setup.py test' needs 'zope.testing', but then doesn't do anything
>>  (missing metadata).  If I added the metadata, the tests would then
>>  pull in those extra packages:  maybe I'll work on trimming them down,
>>  too.
> 
> What's the motivation behind removing test dependenices for
> functionality that actually requires these dependencies, like
> ZODB/hookable/etc.?

I was misled by the fact that ZODB isn't imported, except for testing::

 [/home/tseaver/projects/Zope-CVS/zope.component-trunk]
 $ find src/ -name "*.py" | \
   xargs grep -E "(from ZODB)|(import ZODB)"
 src/zope/component/tests.py:    >>> import ZODB.tests.util
 src/zope/component/tests.py:    >>> import ZODB.tests.util
 src/zope/component/tests.py:    >>> import ZODB.tests.util

The actual import is 'persistent':

 $ find src/ -name "*.py" | \
   xargs grep -E "(from BTrees)|(import BTrees)"
 $ find src/ -name "*.py" | \
   xargs grep -E "(from persistent)|(import persistent)"
 src/zope/component/tests.py:import persistent
 src/zope/component/persistentregistry.py:import persistent.mapping
 src/zope/component/persistentregistry.py:import persistent.list

This seems to me like a good candidate for splitting out the persistent
registry into a separate package.  Right now, the 'extra' means that the
package can be *installed* without the ZODB dependencies, but it can't
be properly *tested* without them.


Tres.
- --
===================================================================
Tres Seaver          +1 540-429-0999          tseaver at palladion.com
Palladion Software   "Excellence by Design"    http://palladion.com
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFJrpSv+gerLs4ltQ4RAhaVAJwNDdHUgb4zxO1hSCmo2NzeZH2rkgCbBSz9
2IJTlcwJxzfmsM3/dC+Of0Y=
=ko+r
-----END PGP SIGNATURE-----