Re: [Zope-CMF] Re: Plone needs a release this week
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Sorry for my silence yesterday - there was apparently a cable cut where I live so I've been without phone and internet all day and all night yesterday. Looks like everything is in place now, thanks to Wichert and Yvo. If there are no other urgent issues I'll do GS 1.3.1 and CMF 2.1.0-final tonight. jens -BEGIN PGP SIGNATURE- Version: GnuPG v1.4.7 (Darwin) iD8DBQFGuWfyRAx5nvEhZLIRAluuAJ90JbBvyntLjZ6uP9aJcbsQS1i+ggCePIsL LJOF5+oPWfgMpQ6XQxnOqSQ= =jaYa -END PGP SIGNATURE- ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
Re: [Zope-CMF] Re: Plone needs a release this week
Hi! Previously yuppie wrote: > Wichert Akkerman wrote: > >But we can't know if component or factory is used. What if someone uses > >a factory which returns a component with a __module__ set? > > Don't know what __module__ usually looks like if the component is > created at registration time. I did hope there would be a way to tell > the difference, but I might be wrong. I don't think there is. > >>But if we have to keep track of registrations anyway, it might be better > >>to do this for all kinds of registrations instead of using a hack. > > > >That is my preference. I have added a statement to REAMDE.txt indicating > >that export of placeless utilies is not supported at the moment. > > Fine. > > But you accidentally checked in your debug code as well. And please > don't forget to forward port your changes to the trunk. Damn, again. Fixed that and merged the changes to trunk. Wichert. -- Wichert Akkerman <[EMAIL PROTECTED]>It is simple to make things. http://www.wiggy.net/ It is hard to make things simple. ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
[Zope-CMF] Re: Plone needs a release this week
Hi! Wichert Akkerman wrote: Previously yuppie wrote: Wichert Akkerman wrote: Previously yuppie wrote: The check for aq_base should be fine, but your example shows a second issue: type() is used to get the factory. That only works if the class is the factory. So there seems to be indeed a need to store somewhere the factory name :( I think we need that to get the export of placeless components as well. Without it I can't seem to think of a way to determine how the component should be created: using a factory method (which could be the class type itself) or some other way. If 'factory' is used, I also can't see a way to determine the factory used on import. If 'component' is used, there might be a solution: __module__ should be set correctly. Maybe we can loop through the objects in the module and compare them with our component? But we can't know if component or factory is used. What if someone uses a factory which returns a component with a __module__ set? Don't know what __module__ usually looks like if the component is created at registration time. I did hope there would be a way to tell the difference, but I might be wrong. But if we have to keep track of registrations anyway, it might be better to do this for all kinds of registrations instead of using a hack. That is my preference. I have added a statement to REAMDE.txt indicating that export of placeless utilies is not supported at the moment. Fine. But you accidentally checked in your debug code as well. And please don't forget to forward port your changes to the trunk. Cheers, Yuppie ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
Re: [Zope-CMF] Re: Plone needs a release this week
Previously yuppie wrote: > Wichert Akkerman wrote: > >Previously yuppie wrote: > >>The check for aq_base should be fine, but your example shows a second > >>issue: type() is used to get the factory. That only works if the class > >>is the factory. > >> > >>So there seems to be indeed a need to store somewhere the factory name :( > > > >I think we need that to get the export of placeless components as well. > >Without it I can't seem to think of a way to determine how the component > >should be created: using a factory method (which could be the class > >type itself) or some other way. > > If 'factory' is used, I also can't see a way to determine the factory > used on import. > > If 'component' is used, there might be a solution: __module__ should be > set correctly. Maybe we can loop through the objects in the module and > compare them with our component? But we can't know if component or factory is used. What if someone uses a factory which returns a component with a __module__ set? > But if we have to keep track of registrations anyway, it might be better > to do this for all kinds of registrations instead of using a hack. That is my preference. I have added a statement to REAMDE.txt indicating that export of placeless utilies is not supported at the moment. Wichert. -- Wichert Akkerman <[EMAIL PROTECTED]>It is simple to make things. http://www.wiggy.net/ It is hard to make things simple. ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
[Zope-CMF] Re: Plone needs a release this week
Wichert Akkerman wrote: Previously yuppie wrote: The check for aq_base should be fine, but your example shows a second issue: type() is used to get the factory. That only works if the class is the factory. So there seems to be indeed a need to store somewhere the factory name :( I think we need that to get the export of placeless components as well. Without it I can't seem to think of a way to determine how the component should be created: using a factory method (which could be the class type itself) or some other way. If 'factory' is used, I also can't see a way to determine the factory used on import. If 'component' is used, there might be a solution: __module__ should be set correctly. Maybe we can loop through the objects in the module and compare them with our component? But if we have to keep track of registrations anyway, it might be better to do this for all kinds of registrations instead of using a hack. Cheers, Yuppie ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
Re: [Zope-CMF] Re: Plone needs a release this week
Previously yuppie wrote: > The check for aq_base should be fine, but your example shows a second > issue: type() is used to get the factory. That only works if the class > is the factory. > > So there seems to be indeed a need to store somewhere the factory name :( I think we need that to get the export of placeless components as well. Without it I can't seem to think of a way to determine how the component should be created: using a factory method (which could be the class type itself) or some other way. Wichert. -- Wichert Akkerman <[EMAIL PROTECTED]>It is simple to make things. http://www.wiggy.net/ It is hard to make things simple. ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
[Zope-CMF] Re: Plone needs a release this week
Wichert Akkerman wrote: Previously yuppie wrote: Wichert Akkerman wrote: I get something else: if I import this: the export looks like this: That seems to be a similar but different bug. this is caused by the zope.component.registerUtility not storing the factory method but the return value from the factory, so we no longer have the factory method available when we export. The only way to fix that appears to be for us to keep a a registry of factory methods in the setup tool. That may be too big of a change in a minor release though. Not sure if your issue is a problem in real life. AFAICS this only happens if you use a factory that is meant for ZODB objects. In that case you should register an object. Ah, that makes sense I guess. My test-case was this: from Products.MailHost.MailHost import MailHost def genMail(): return MailHost() and a MailHost is indeed a ZODB object. The check for aq_base should be fine, but your example shows a second issue: type() is used to get the factory. That only works if the class is the factory. So there seems to be indeed a need to store somewhere the factory name :( Cheers, Yuppie ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
Re: [Zope-CMF] Re: Plone needs a release this week
Previously yuppie wrote: > Hi! > > > Wichert Akkerman wrote: > >Previously yuppie wrote: > >>Wichert Akkerman wrote: > >>>Previously yuppie wrote: > - The exports created by the new components handler are still flawed, > ISiteRoot and placeless components are not exported correctly. > >>>I'm quite sure I fixed that: I was able to export the components and > >>>import them again. Has that been broken since? > >>Don't know since when these things are broken, but in the latest code > >>there are these 2 bugs: > >> > >> > >>1.) Exporting the ISiteRoot utility, 'object' should be empty. But I get > >>this instead: > >> > >> >> object="SITE_ID"/> > >> > >> > >>2.) By placeless components I mean something like this: > >> > >> > >> > >>The import works fine, but the export looks like this: > >> > >> > > Attached is a diff for test_components.py. Unfortunately the > AssertionErrors are a bit cryptic, but they show that the factory is > exported instead of the component. I'll take a look at those. > >I get something else: if I import this: > > > > > > > >the export looks like this: > > > > > > That seems to be a similar but different bug. > > >this is caused by the zope.component.registerUtility not storing the > >factory method but the return value from the factory, so we no longer > >have the factory method available when we export. > > > >The only way to fix that appears to be for us to keep a a registry of > >factory methods in the setup tool. That may be too big of a change > >in a minor release though. > > Not sure if your issue is a problem in real life. AFAICS this only > happens if you use a factory that is meant for ZODB objects. In that > case you should register an object. Ah, that makes sense I guess. My test-case was this: from Products.MailHost.MailHost import MailHost def genMail(): return MailHost() and a MailHost is indeed a ZODB object. Wichert. -- Wichert Akkerman <[EMAIL PROTECTED]>It is simple to make things. http://www.wiggy.net/ It is hard to make things simple. ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
[Zope-CMF] Re: Plone needs a release this week
Hi! Wichert Akkerman wrote: Previously yuppie wrote: Wichert Akkerman wrote: Previously yuppie wrote: - The exports created by the new components handler are still flawed, ISiteRoot and placeless components are not exported correctly. I'm quite sure I fixed that: I was able to export the components and import them again. Has that been broken since? Don't know since when these things are broken, but in the latest code there are these 2 bugs: 1.) Exporting the ISiteRoot utility, 'object' should be empty. But I get this instead: 2.) By placeless components I mean something like this: The import works fine, but the export looks like this: Attached is a diff for test_components.py. Unfortunately the AssertionErrors are a bit cryptic, but they show that the factory is exported instead of the component. I get something else: if I import this: the export looks like this: That seems to be a similar but different bug. this is caused by the zope.component.registerUtility not storing the factory method but the return value from the factory, so we no longer have the factory method available when we export. The only way to fix that appears to be for us to keep a a registry of factory methods in the setup tool. That may be too big of a change in a minor release though. Not sure if your issue is a problem in real life. AFAICS this only happens if you use a factory that is meant for ZODB objects. In that case you should register an object. So my vote would be: document that exporting of utility factories is currently not supported and will be fixed in GenericSetup 1.4 and tag the current code as 1.3.1. I'm fine with that. Cheers, Yuppie Index: Products/GenericSetup/tests/test_components.py === --- Products/GenericSetup/tests/test_components.py (revision 78655) +++ Products/GenericSetup/tests/test_components.py (working copy) @@ -69,6 +69,8 @@ def verify(self): return True +dummy_utility = DummyUtility() + class DummyTool(SimpleItem): """A dummy tool.""" @@ -113,6 +115,9 @@ + @@ -135,6 +140,9 @@ tool2 = aq_base(self.app['dummy_tool2']) obj.registerUtility(tool2, IDummyInterface, name=u'dummy tool name2') +obj.registerUtility(dummy_utility, IDummyInterface, +name=u'dummy utility name') + def test_body_get(self): self._populate(self._obj) context = DummyExportContext(self.app) @@ -171,6 +179,10 @@ self.failUnless(IDummyInterface.providedBy(util)) self.failUnless(util.verify()) +util = queryUtility(IDummyInterface, name=u'dummy utility name') +self.failUnless(IDummyInterface.providedBy(util)) +self.failUnless(util.verify()) + util = queryUtility(IDummyInterface) self.failUnless(IDummyInterface.providedBy(util)) self.failUnless(util.verify()) ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
Re: [Zope-CMF] Re: Plone needs a release this week
Previously Jens Vagelpohl wrote: > I'm supposed to do a CMF 2.1.0 release today, but the state of these > issues is unclear. Wichert, did you look at it? There are no checkins > into either CMF or GS as far as I can see. Export now yields this: I took care of the GenericSetup changes as far as I think are possible. Exporting of utility factories is broken but requires more changes than I'm willing to make in a minor release. I've also gone over the docstrings in CMFCore to check the BBB remarks there and cleaned up those that were no longer correct. I don't think there are any showstoppers left. Wichert. -- Wichert Akkerman <[EMAIL PROTECTED]>It is simple to make things. http://www.wiggy.net/ It is hard to make things simple. ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
Re: [Zope-CMF] Re: Plone needs a release this week
Previously yuppie wrote: > - It might be useful to review the docstrings in CMFCore.utils and > interfaces._tools. There are misleading comments like "BBB: for use in > 'getToolByName'; in the future, prefer > 'zapi.getUtility(IActionsTool)'." Seems not all docs reflect the latest > tools-as-utilities changes. I took care of that one. Wichert. -- Wichert Akkerman <[EMAIL PROTECTED]>It is simple to make things. http://www.wiggy.net/ It is hard to make things simple. ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
Re: [Zope-CMF] Re: Plone needs a release this week
Previously yuppie wrote: > Wichert Akkerman wrote: > >Previously yuppie wrote: > >>- The exports created by the new components handler are still flawed, > >>ISiteRoot and placeless components are not exported correctly. > > > >I'm quite sure I fixed that: I was able to export the components and > >import them again. Has that been broken since? > > Don't know since when these things are broken, but in the latest code > there are these 2 bugs: > > > 1.) Exporting the ISiteRoot utility, 'object' should be empty. But I get > this instead: > > object="SITE_ID"/> > > > 2.) By placeless components I mean something like this: > > > > The import works fine, but the export looks like this: > > I get something else: if I import this: the export looks like this: this is caused by the zope.component.registerUtility not storing the factory method but the return value from the factory, so we no longer have the factory method available when we export. The only way to fix that appears to be for us to keep a a registry of factory methods in the setup tool. That may be too big of a change in a minor release though. So my vote would be: document that exporting of utility factories is currently not supported and will be fixed in GenericSetup 1.4 and tag the current code as 1.3.1. Wichert. -- Wichert Akkerman <[EMAIL PROTECTED]>It is simple to make things. http://www.wiggy.net/ It is hard to make things simple. ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
Re: [Zope-CMF] Re: Plone needs a release this week
Previously yuppie wrote: > Wichert Akkerman wrote: > >Previously yuppie wrote: > >>- The exports created by the new components handler are still flawed, > >>ISiteRoot and placeless components are not exported correctly. > > > >I'm quite sure I fixed that: I was able to export the components and > >import them again. Has that been broken since? > > Don't know since when these things are broken, but in the latest code > there are these 2 bugs: > > > 1.) Exporting the ISiteRoot utility, 'object' should be empty. But I get > this instead: > > object="SITE_ID"/> I fixed this one again. > 2.) By placeless components I mean something like this: > > > > The import works fine, but the export looks like this: > > Do you have a failing test or another way to reproduce this? Wichert. -- Wichert Akkerman <[EMAIL PROTECTED]>It is simple to make things. http://www.wiggy.net/ It is hard to make things simple. ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
[Zope-CMF] Re: Plone needs a release this week
Hi! Jens Vagelpohl wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 5 Aug 2007, at 20:15, yuppie wrote: I'm supposed to do a CMF 2.1.0 release today, but the state of these issues is unclear. Wichert, did you look at it? There are no checkins into either CMF or GS as far as I can see. Export now yields this: placeless component: This is not what I would call placeless. My example looks like this: 'foo.bar' is the dotted name of an object in a python module, not in the ZODB. The 'component' feature is not used by the CMF, but GenericSetup claims to support it. If you're speaking about something that's not even used by the CMF then that won't be a shostopper for the CMF. For GS, IMHO, this is a bit of an esoteric feature. I don't know any place that uses it. If there's a decision to be made between "get the goddamn release out" and "wait until someone figures out this piece in GS" I'll choose the first option. At least it should be documented somewhere that 'component' support is broken and not ready for use. the site root: I was able to import a snapshot that contained a componentregistry export of this format, though. I guess this only works with a site that has the same ID and no sub-object with this ID. Not very reliable and confusing. Well, I imported it into a site with a different ID. The "Components" tab you provided shows the "old" ID, unfortunately. However, I don't know if that makes a difference in real life or if it's just a cosmetic issue. It makes a difference: It means ISiteRoot returns the wrong site. This is easy to fix - it was fixed once, but without unit tests that made sure it stayed fixed. Wichert volunteered to have a look at this issue. Can I go ahead and roll a GS 1.3.1 and then a CMF 2.1.0 using GS 1.3.1 or are there any showstoppers left? I made a list of things that should be fixed in my opinion. But I can't decide what is a showstopper and what isn't. If it's not an obvious showstopper I feel it's not worth holding up Wichert. Would you agree? Wichert is the person who volunteered to work on these issues, so nobody besides himself is holding him up. Cheers, Yuppie ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
[Zope-CMF] Re: Plone needs a release this week
yuppie wrote: AFAICS the premature GenericSetup 1.3 release has the biggest issues: - There are also some usability issues (strange ordering; strange interface names including '_tools' and '_content'; Ordering and dotted names are fixed now. This makes it much easier to read the diffs. Cheers, Yuppie ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
Re: [Zope-CMF] Re: Plone needs a release this week
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 5 Aug 2007, at 20:15, yuppie wrote: I'm supposed to do a CMF 2.1.0 release today, but the state of these issues is unclear. Wichert, did you look at it? There are no checkins into either CMF or GS as far as I can see. Export now yields this: placeless component: interface="Products.CMFCore.interfaces._tools.IMetadataTool" object="portal_metadata"/> This is not what I would call placeless. My example looks like this: 'foo.bar' is the dotted name of an object in a python module, not in the ZODB. The 'component' feature is not used by the CMF, but GenericSetup claims to support it. If you're speaking about something that's not even used by the CMF then that won't be a shostopper for the CMF. For GS, IMHO, this is a bit of an esoteric feature. I don't know any place that uses it. If there's a decision to be made between "get the goddamn release out" and "wait until someone figures out this piece in GS" I'll choose the first option. the site root: I was able to import a snapshot that contained a componentregistry export of this format, though. I guess this only works with a site that has the same ID and no sub- object with this ID. Not very reliable and confusing. Well, I imported it into a site with a different ID. The "Components" tab you provided shows the "old" ID, unfortunately. However, I don't know if that makes a difference in real life or if it's just a cosmetic issue. Can I go ahead and roll a GS 1.3.1 and then a CMF 2.1.0 using GS 1.3.1 or are there any showstoppers left? I made a list of things that should be fixed in my opinion. But I can't decide what is a showstopper and what isn't. If it's not an obvious showstopper I feel it's not worth holding up Wichert. Would you agree? jens -BEGIN PGP SIGNATURE- Version: GnuPG v1.4.7 (Darwin) iD8DBQFGtjKcRAx5nvEhZLIRAgZNAKCEta0hWr4cO0CIfLvSIbqI1s0TcACfWyv0 iLkfue7OIC5i1E0LzzLYY5g= =nzNS -END PGP SIGNATURE- ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
[Zope-CMF] Re: Plone needs a release this week
Hi! Jens Vagelpohl wrote: On 2 Aug 2007, at 13:55, Wichert Akkerman wrote: Previously yuppie wrote: 1.) Exporting the ISiteRoot utility, 'object' should be empty. But I get this instead: 2.) By placeless components I mean something like this: The import works fine, but the export looks like this: I'm quite sure I had both of those working last time I touched GS. I'll poke at them later this week. I'm supposed to do a CMF 2.1.0 release today, but the state of these issues is unclear. Wichert, did you look at it? There are no checkins into either CMF or GS as far as I can see. Export now yields this: placeless component: This is not what I would call placeless. My example looks like this: 'foo.bar' is the dotted name of an object in a python module, not in the ZODB. The 'component' feature is not used by the CMF, but GenericSetup claims to support it. the site root: I was able to import a snapshot that contained a componentregistry export of this format, though. I guess this only works with a site that has the same ID and no sub-object with this ID. Not very reliable and confusing. Can I go ahead and roll a GS 1.3.1 and then a CMF 2.1.0 using GS 1.3.1 or are there any showstoppers left? I made a list of things that should be fixed in my opinion. But I can't decide what is a showstopper and what isn't. Cheers, Yuppie ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
Re: [Zope-CMF] Re: Plone needs a release this week
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 2 Aug 2007, at 13:55, Wichert Akkerman wrote: Previously yuppie wrote: 1.) Exporting the ISiteRoot utility, 'object' should be empty. But I get this instead: 2.) By placeless components I mean something like this: The import works fine, but the export looks like this: I'm quite sure I had both of those working last time I touched GS. I'll poke at them later this week. I'm supposed to do a CMF 2.1.0 release today, but the state of these issues is unclear. Wichert, did you look at it? There are no checkins into either CMF or GS as far as I can see. Export now yields this: placeless component: the site root: I was able to import a snapshot that contained a componentregistry export of this format, though. Can I go ahead and roll a GS 1.3.1 and then a CMF 2.1.0 using GS 1.3.1 or are there any showstoppers left? jens -BEGIN PGP SIGNATURE- Version: GnuPG v1.4.7 (Darwin) iD8DBQFGtgqsRAx5nvEhZLIRAnYmAJ0Q1/i/MgW8bLltzHRIIsZz2ybdugCfZ6rE B2ArW5JEDOujKPAHvRdTWzs= =i6Bu -END PGP SIGNATURE- ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
Re: [Zope-CMF] Re: Plone needs a release this week
Previously yuppie wrote: > Wichert Akkerman wrote: > >Previously yuppie wrote: > >>- The exports created by the new components handler are still flawed, > >>ISiteRoot and placeless components are not exported correctly. > > > >I'm quite sure I fixed that: I was able to export the components and > >import them again. Has that been broken since? > > Don't know since when these things are broken, but in the latest code > there are these 2 bugs: > > > 1.) Exporting the ISiteRoot utility, 'object' should be empty. But I get > this instead: > > object="SITE_ID"/> > > > 2.) By placeless components I mean something like this: > > > > The import works fine, but the export looks like this: > > I'm quite sure I had both of those working last time I touched GS. I'll poke at them later this week. Wichert. -- Wichert Akkerman <[EMAIL PROTECTED]>It is simple to make things. http://www.wiggy.net/ It is hard to make things simple. ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
Re: [Zope-CMF] Re: Plone needs a release this week
Am 31.07.2007 um 17:17 schrieb Tres Seaver: 'imperative' indicates that the profile executes arbitrary Python code; 'declarative' indecates that the profile applies external configuration data, read from one or more data files in the profile. Running two imperative profiles, A and B, may create different effects based on the order in which they are run, or whether they are run more than once. With declarative profiles, OTOH, the state is all captured in the data file(s), which means there are no ordering depencencies, and it is always "safe" (idempotent) to reapply the step. Thanks for the explanation, Tres. I still find the term distinction somewhat confusing and would suggest "runtime" or "ad hoc" for the second one but if it is an established term in software development then who am I to rewrite the book! While I have a fairly intense dislike of XML I am fully in favour on configuration being kept external to an application although I don't see how conflicts: two products wishing to configure the same object differently can be resolved this way. However, it seems I am not the only one who been confused by the discussion. Would it make sense to move it to a wiki where we have use cases, problems, etc? What I am personally missing are examples of current "gotchas" and how they can be resolved. Does http://zope.org/Products/CMF/docs/requirements/proposals/ GenericSetup_architectureflect the current status? Charlie -- Charlie Clark Helmholtzstr. 20 Düsseldorf D- 40215 Tel: +49-211-938-5360 GSM: +49-178-782-6226 ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
[Zope-CMF] Re: Plone needs a release this week
Wichert Akkerman wrote: Previously yuppie wrote: - The exports created by the new components handler are still flawed, ISiteRoot and placeless components are not exported correctly. I'm quite sure I fixed that: I was able to export the components and import them again. Has that been broken since? Don't know since when these things are broken, but in the latest code there are these 2 bugs: 1.) Exporting the ISiteRoot utility, 'object' should be empty. But I get this instead: 2.) By placeless components I mean something like this: The import works fine, but the export looks like this: Cheers, Yuppie ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
Re: [Zope-CMF] Re: Plone needs a release this week
Previously yuppie wrote: > Hi! > > > Jens Vagelpohl wrote: > > > >On 20 Jul 2007, at 11:00, Wichert Akkerman wrote: > > > >>Previously Jens Vagelpohl wrote: > >>>Next step: What needs fixing for the final? > >> > >>There has been a surprising lack of response to this question. Since I > >>need a CMF 2.1-final for Plone in a few weeks that could be very > >>positive news for me but I have a suspicion that some of you have some > >>outstanding issues. Or is there really nothing that needs to be done > >>before going final? > > > >If no one objects I will cut a CMF 2.1.0-final the weekend of August 5. > > I finally found some time to have a closer look at the latest code. > > > AFAICS the premature GenericSetup 1.3 release has the biggest issues: > > - The exports created by the new components handler are still flawed, > ISiteRoot and placeless components are not exported correctly. I'm quite sure I fixed that: I was able to export the components and import them again. Has that been broken since? Wichert. -- Wichert Akkerman <[EMAIL PROTECTED]>It is simple to make things. http://www.wiggy.net/ It is hard to make things simple. ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
[Zope-CMF] Re: Plone needs a release this week
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Charlie Clark wrote: > Am 31.07.2007 um 15:51 schrieb yuppie: > >> AFAICS the premature GenericSetup 1.3 release has the biggest issues: > > I'm behind the loop on GenericSetup having only encountered it when > writing a content type for a single site. I would appreciate an > explanation of the difference between declarative and imperative as > linguistically they are almost identical. I have never done imports & > exports so would appreciate knowing where to go to read up on them so > that I can contribute to the debate or at least understand it fully. 'imperative' indicates that the profile executes arbitrary Python code; 'declarative' indecates that the profile applies external configuration data, read from one or more data files in the profile. Running two imperative profiles, A and B, may create different effects based on the order in which they are run, or whether they are run more than once. With declarative profiles, OTOH, the state is all captured in the data file(s), which means there are no ordering depencencies, and it is always "safe" (idempotent) to reapply the step. 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 iD8DBQFGr1J++gerLs4ltQ4RAhlXAJ9JzP8Mlm1aTHuY4P6iKqNbkS77PgCgrwvm qIzDT7DK+IHGFyc2V74fHGw= =DrXR -END PGP SIGNATURE- ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
Re: [Zope-CMF] Re: Plone needs a release this week
Am 31.07.2007 um 15:51 schrieb yuppie: AFAICS the premature GenericSetup 1.3 release has the biggest issues: I'm behind the loop on GenericSetup having only encountered it when writing a content type for a single site. I would appreciate an explanation of the difference between declarative and imperative as linguistically they are almost identical. I have never done imports & exports so would appreciate knowing where to go to read up on them so that I can contribute to the debate or at least understand it fully. ... I still consider the VERSION.txt support a mis-feature. VERSION.txt is really nasty anyway but if it is still around you are right: the profile version is not necessarily the same as the product version and so nothing should assume this. five.localsitemanager still has no UI for inspecting and editing component registrations. I added the 'components_xmlconfig.html' view to GenericSetup as an alternative way to do this, but a normal UI would still be useful. CMF seems to be in good shape, but the migration code and documentation could need some cleanup: - The 2.1 migration should use the new 'upgradeStep' feature, the on-the-fly migration code in PortalObjectBase.getSiteManager should be replaced by an 'upgradeStep' - It might be useful to review the docstrings in CMFCore.utils and interfaces._tools. There are misleading comments like "BBB: for use in 'getToolByName'; in the future, prefer 'zapi.getUtility (IActionsTool)'." Seems not all docs reflect the latest tools-as- utilities changes. Yes, please! I recently really struggled writing a browser view because of the lack of doc strings. I can only assume that BBB has something to do with something "bad". I think the most important issues can be resolved before the weekend *if* some people are willing to help. Having recently given a presentation (in German) on CMF 2.1 I hope at least to be contributing to the documentation. However, as noted above there are significant areas with which I have little or no experience. Charlie -- Charlie Clark Helmholtzstr. 20 Düsseldorf D- 40215 Tel: +49-211-938-5360 GSM: +49-178-782-6226 ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
[Zope-CMF] Re: Plone needs a release this week
yuppie wrote: > Jens Vagelpohl wrote: >> On 20 Jul 2007, at 11:00, Wichert Akkerman wrote: >>> Previously Jens Vagelpohl wrote: Next step: What needs fixing for the final? >>> >>> There has been a surprising lack of response to this question. Since I >>> need a CMF 2.1-final for Plone in a few weeks that could be very >>> positive news for me but I have a suspicion that some of you have some >>> outstanding issues. Or is there really nothing that needs to be done >>> before going final? >> >> If no one objects I will cut a CMF 2.1.0-final the weekend of August 5. > > I finally found some time to have a closer look at the latest code. > > AFAICS the premature GenericSetup 1.3 release has the biggest issues: > > - The exports created by the new components handler are still flawed, > ISiteRoot and placeless components are not exported correctly. > > - There are also some usability issues (strange ordering; strange > interface names including '_tools' and '_content'; no support for > sub-sites; more attributes than necessary are required; no support for > removing items), but they might not be release blockers. > > I sent hannosch more detailed information, but if he can't work on these > issues this week, someone else has to resolve them. I'm on a sprint in France this week, which I need to concentrate on (because it hopefully produces some results for my Google Summer of Code project), so I won't have time to work on these issues in this week and I doubt to have much time next week as well. Sorry, Hanno ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
[Zope-CMF] Re: Plone needs a release this week
Hi! Jens Vagelpohl wrote: On 20 Jul 2007, at 11:00, Wichert Akkerman wrote: Previously Jens Vagelpohl wrote: Next step: What needs fixing for the final? There has been a surprising lack of response to this question. Since I need a CMF 2.1-final for Plone in a few weeks that could be very positive news for me but I have a suspicion that some of you have some outstanding issues. Or is there really nothing that needs to be done before going final? If no one objects I will cut a CMF 2.1.0-final the weekend of August 5. I finally found some time to have a closer look at the latest code. AFAICS the premature GenericSetup 1.3 release has the biggest issues: - The exports created by the new components handler are still flawed, ISiteRoot and placeless components are not exported correctly. - There are also some usability issues (strange ordering; strange interface names including '_tools' and '_content'; no support for sub-sites; more attributes than necessary are required; no support for removing items), but they might not be release blockers. I sent hannosch more detailed information, but if he can't work on these issues this week, someone else has to resolve them. - There is also this discussion that had no consequences: http://mail.zope.org/pipermail/zope-cmf/2007-June/026220.html I still consider the VERSION.txt support a mis-feature. five.localsitemanager still has no UI for inspecting and editing component registrations. I added the 'components_xmlconfig.html' view to GenericSetup as an alternative way to do this, but a normal UI would still be useful. CMF seems to be in good shape, but the migration code and documentation could need some cleanup: - The 2.1 migration should use the new 'upgradeStep' feature, the on-the-fly migration code in PortalObjectBase.getSiteManager should be replaced by an 'upgradeStep' - It might be useful to review the docstrings in CMFCore.utils and interfaces._tools. There are misleading comments like "BBB: for use in 'getToolByName'; in the future, prefer 'zapi.getUtility(IActionsTool)'." Seems not all docs reflect the latest tools-as-utilities changes. I think the most important issues can be resolved before the weekend *if* some people are willing to help. Cheers, Yuppie ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
[Zope-CMF] Re: Plone needs a release this week
Wichert Akkerman wrote: > Previously Jens Vagelpohl wrote: >> Next step: What needs fixing for the final? > > There has been a surprising lack of response to this question. Since I > need a CMF 2.1-final for Plone in a few weeks that could be very > positive news for me but I have a suspicion that some of you have some > outstanding issues. Or is there really nothing that needs to be done > before going final? I just fixed the last long-standing issues with the componentregistry handler, which has been the last item on my list. So we are good to go from my side :) Hanno ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
[Zope-CMF] Re: Plone needs a release this week
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Jens Vagelpohl wrote: > Alright, I'm going to do the following today/tonight: > > - do a GS 1.3 beta and create a GS 1.3 branch, and put a release on > zope.org > > - point the GS external on the CMF 2.1 branch to the GS 1.3 beta tag > > - run all unit tests for the CMF 2.1 branch > > - tag CMF 2.1beta and release it > > If there are any concerns let me know ASAP. Sounds great -- thanks for doing this! Trs. - -- === 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 iD8DBQFGllNP+gerLs4ltQ4RAv8WAJ4gCpIdrGspp0MbcQ1/wxXmK7pJfQCg0CFU LR6THFw/IanbLK1vaYMG0ck= =ikyI -END PGP SIGNATURE- ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
Re: [Zope-CMF] Re: Plone needs a release this week
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Alright, I'm going to do the following today/tonight: - - do a GS 1.3 beta and create a GS 1.3 branch, and put a release on zope.org - - point the GS external on the CMF 2.1 branch to the GS 1.3 beta tag - - run all unit tests for the CMF 2.1 branch - - tag CMF 2.1beta and release it If there are any concerns let me know ASAP. jens -BEGIN PGP SIGNATURE- Version: GnuPG v1.4.7 (Darwin) iD8DBQFGlk+ARAx5nvEhZLIRAgklAKCUY9/tD7WeMrEglp+WOfg+g84iJgCeIUxt xulLARQ4py0B8cIW9e2ScpY= =iahK -END PGP SIGNATURE- ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
Re: [Zope-CMF] Re: Plone needs a release this week
Previously Tres Seaver wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > Wichert Akkerman wrote: > > I really need a CMF release for Plone this week, but things seem to have > > suddenly gotten a bit quiet. If nothing happens I'll probably make > > another snapshot of the CMF 2.1 branch this Friday and include that with > > Plone 3.0rc1. We have several external factors that affect our timing > > and we can not postpone things any later. > > > > As I said earlier I'm happy to help with any changes that need to be > > made still, but without feedback there is nothing I can do. > > Are there any showstoppers left for a 2.1 beta? > > - I was worried about the tests that failed last week, but perhaps that >was a glitch on the testing machine? At any rate, I didn't see >checkins which would've fixed them, and they are all passing this >week. > > - I think the 'five:localsitemanager' fixes have landed. > > - The GenericSetup changes from the BBQ sprint have been landed and >polihsed: GS should be ready for a 1.5 beta. > > - Migration for sitse which have been tracking the 2.1 betas may be >flaky at this point. Yuppie has a hack in place to fix things up >on the fly, but isn't happy about it. > > - I think the "roadmap" issues don't block a 2.1 release. I just added the utility unregistration code to the migrateToCMF21 script. The only remaining item that I can think of is this one from Yvo: There is a hack in PortalObjectBase.getSiteManager that fixes old site manager instances on the fly, but I don't want to have that code in the final CMF 2.1 release That should not be a showstopper for a beta release imho. Wichert. -- Wichert Akkerman <[EMAIL PROTECTED]>It is simple to make things. http://www.wiggy.net/ It is hard to make things simple. ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
[Zope-CMF] Re: Plone needs a release this week
Tres Seaver wrote: Wichert Akkerman wrote: I really need a CMF release for Plone this week, but things seem to have suddenly gotten a bit quiet. If nothing happens I'll probably make another snapshot of the CMF 2.1 branch this Friday and include that with Plone 3.0rc1. We have several external factors that affect our timing and we can not postpone things any later. As I said earlier I'm happy to help with any changes that need to be made still, but without feedback there is nothing I can do. Are there any showstoppers left for a 2.1 beta? AFAICS the last showstoppers were fixed today by Wichert. +1 for a new 2.1 beta Cheers, Yuppie ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
[Zope-CMF] Re: Plone needs a release this week
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Wichert Akkerman wrote: > I really need a CMF release for Plone this week, but things seem to have > suddenly gotten a bit quiet. If nothing happens I'll probably make > another snapshot of the CMF 2.1 branch this Friday and include that with > Plone 3.0rc1. We have several external factors that affect our timing > and we can not postpone things any later. > > As I said earlier I'm happy to help with any changes that need to be > made still, but without feedback there is nothing I can do. Are there any showstoppers left for a 2.1 beta? - I was worried about the tests that failed last week, but perhaps that was a glitch on the testing machine? At any rate, I didn't see checkins which would've fixed them, and they are all passing this week. - I think the 'five:localsitemanager' fixes have landed. - The GenericSetup changes from the BBQ sprint have been landed and polihsed: GS should be ready for a 1.5 beta. - Migration for sitse which have been tracking the 2.1 betas may be flaky at this point. Yuppie has a hack in place to fix things up on the fly, but isn't happy about it. - I think the "roadmap" issues don't block a 2.1 release. Thoughts? 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 iD8DBQFGlkC1+gerLs4ltQ4RAml8AKDETM7tTf/pSLTqHTeQgrP/iJB9jwCaA5T8 I1Fj5+gAlLr+EWEnIfStsyg= =nsLe -END PGP SIGNATURE- ___ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
