Re: [Zope3-dev] zope.i18n dependency in zope.configuration

2006-10-22 Thread Fred Drake

On 10/22/06, Baiju M [EMAIL PROTECTED] wrote:

So any module imported should be added as a dependency.
Then, what about modules imported in test files (*.txt or test*.py files) ?


Pretty much; there are cases where a package might import something
conditionally in order to improve integration, but if it isn't
available, the package as a whole can function just fine on its own.
In those cases, no dependency should be recorded.

Ideally, dependencies of the implementation and the tests should be
separated, but we've been treating them as one so far.  Moving to eggs
should allow improvements along these lines.


 -Fred

--
Fred L. Drake, Jr.fdrake at gmail.com
Every sin is the result of a collaboration. --Lucius Annaeus Seneca
___
Zope3-dev mailing list
Zope3-dev@zope.org
Unsub: http://mail.zope.org/mailman/options/zope3-dev/archive%40mail-archive.com



Re: [Zope3-dev] zope.i18n dependency in zope.configuration

2006-10-21 Thread Fred Drake

On 10/21/06, Baiju M [EMAIL PROTECTED] wrote:

   I was looking at zope.configuration source, zope.i18n is given as a
dependency in
DEPENDENCIES.cfg, but it is not imported anywhere in that package. So,
can I remove
'zope.i18n' line from DEPENDENCIES.cfg file?


It looks like zope.i18n can go, but the simpler zope.i18nmessageid
needs to be added (done on the trunk).  Maintenance branches should be
checked for similar updates.

Thanks!


 -Fred

--
Fred L. Drake, Jr.fdrake at gmail.com
Every sin is the result of a collaboration. --Lucius Annaeus Seneca
___
Zope3-dev mailing list
Zope3-dev@zope.org
Unsub: http://mail.zope.org/mailman/options/zope3-dev/archive%40mail-archive.com



Re: [Zope3-dev] zope.i18n dependency in zope.configuration

2006-10-21 Thread Baiju M

On 10/22/06, Fred Drake [EMAIL PROTECTED] wrote:

On 10/21/06, Baiju M [EMAIL PROTECTED] wrote:
I was looking at zope.configuration source, zope.i18n is given as a
 dependency in
 DEPENDENCIES.cfg, but it is not imported anywhere in that package. So,
 can I remove
 'zope.i18n' line from DEPENDENCIES.cfg file?

It looks like zope.i18n can go, but the simpler zope.i18nmessageid
needs to be added (done on the trunk).  Maintenance branches should be
checked for similar updates.


So any module imported should be added as a dependency.
Then, what about modules imported in test files (*.txt or test*.py files) ?

Regards,
Baiju M
___
Zope3-dev mailing list
Zope3-dev@zope.org
Unsub: http://mail.zope.org/mailman/options/zope3-dev/archive%40mail-archive.com