-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Tres Seaver wrote: > Tres Seaver wrote: >> yuppie wrote: >>> Hi Tres! > >>> Tres Seaver wrote: >>>> I don't know what the purpose of the following checkin was: >>>> >>>> http://svn.zope.org/Zope/trunk/?rev=85827&view=rev >>> See https://mail.zope.org/pipermail/zope-dev/2008-April/031691.html >>>> but the tests all pass if I rip out the import, and Zope starts. I >>>> don't know what code in Zope2 expects the ZopeVocabularyRegistry to be >>>> registered. Can you remember what needed it? >>> A wrong import in CMFCalendar did mask the problem, but I just fixed >>> that and now you can see some CMF tests failing if that import is >>> removed in Five. >> Having tests fail becuase of a removed import-for-side-effects doesn't >> give me a good feeling about the quality of the tests. > >> At any rate, if there is a real feature needed by Zope2 to function >> correctly, then we should move it from zope.app.schema to zope.schema, >> and make it possible to configure cleanly (instead of via side effects). > >> (Later) > >> OK, I have copied the ZopeVocabularyRegistry from >> zope.app.schema.vocabulary into a new module, Products.Five.schema, >> renaming it to Zope2VocabularyRegistry. I arranged to get it installed >> insside Five's initialize(). > >> I updated the failing CMFDefault and CMFCalendar doctests to import and >> install that registry in a new 'setUp', and clear it in 'tearDown'. > > Ugh, I have the sinking feeling that this fix needs to be backported to > the 2.12 branch before we get to a release. Andreas, it is a pretty > small change, actually, and removes any dependency on zope.app.schema: > can I merge it?
Under the "Silence is assent" principle, done. I also backported the "rip out Twisted integration" change. Tres. - -- =================================================================== Tres Seaver +1 540-429-0999 [email protected] 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 iD8DBQFKuYvG+gerLs4ltQ4RAhLvAJ49JvIP3+nlCdWVlwthLeitZdprPQCfekMK O5v8WxQUdThvkIBVZehm8kE= =7pOJ -----END PGP SIGNATURE----- _______________________________________________ Zope-Dev maillist - [email protected] https://mail.zope.org/mailman/listinfo/zope-dev ** No cross posts or HTML encoding! ** (Related lists - https://mail.zope.org/mailman/listinfo/zope-announce https://mail.zope.org/mailman/listinfo/zope )
