Chris McDonough wrote:
On 5/14/09 11:05 PM, Chris McDonough wrote:
- It depends on zope.filerepresentation but depends only on its interfaces IReadDirectory, IWriteDirectory, and IDirectoryFactory. (zope.filerepresentation has 32 transitive dependencies).
I found out that zope.container<->zope.filerepresentation is a direct circular dependency and that zope.filerepresentation is a package containing only interfaces. So breaking this dependency won't get us much for zope.container. OTOH, breaking zope.filerepresentation's dependency on zope.container might be a win (I dont know what else depends on zope.filerepresentation). zope.filerepresentation depends only on zope.container.interfaces.IReadContainer and zope.container.interfaces.IWriteContainer.
I'd favor moving the interfaces from zope.filerepresentation into zope.container itself and abandoning the zope.filerepresentation package. Hanno