Ok,

I think I have a fix for this but we have a little semantics issue. I thought that once the Zope config was set it could not be changed. In the new ZOPE_CONFIG patch the environ variable only gets used as a last resort in Zope.app() if no other configuration has been set. To do this I added a isConfigured() function to App.config to check its _config global. I'm assuming that this is the "official" Zope config, correct?

Now there is a test testZopeRunConfigure in lib/python/Zope/Startup/tests/testStarter.py that runs App.config.getConfiguration() and then makes a test config file and sets it. The result is a test where the config is reset within one run. This test fails with the semantics described above. Which is correct? I'm enclosing the patch for reference.

-EAD


Attachment: ZOPE_CONFIG.patch
Description: Binary data



On May 17, 2004, at 12:20 PM, Erik A.Dahl wrote:

Things changed around a bit since I did this patch. I'm trying to get a new one working but am having some issues with the 2.7 trunk in svn. I think in general the startup sequence could use some clean up. I'm working on mapping how it all works maybe that would help make it less opaque.

-EAD

On May 15, 2004, at 12:53 PM, Chris McDonough wrote:

I think Fred might have some ideas about how to do the ZOPE_CONFIG
feature better if you can coax them out of him. ;-)

On Sat, 2004-05-15 at 03:56, Andreas Jung wrote:
--On Samstag, 15. Mai 2004 1:59 Uhr +0200 "Stefan H. Holek"
<[EMAIL PROTECTED]> wrote:


Zope tests fail for me, unfortunately Andreas cannot reproduce this.


The ZOPE_CONFIG patch was responsible for other failures. I reverted the
change
and check for a better solution by next week.

-aj


_______________________________________________
Zope-Dev maillist  -  [EMAIL PROTECTED]
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 )


_______________________________________________
Zope-Dev maillist  -  [EMAIL PROTECTED]
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 )
_______________________________________________
Zope-Dev maillist  -  [EMAIL PROTECTED]
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 )

Reply via email to