Re: [Zope-dev] Dependencies question

2008-12-12 Thread Wichert Akkerman
Previously Dan Korostelev wrote: I was looking at dependencies of various zope.* packages and see that some packages depend on other because of ZCML directives and some are not. For example: The zope.app.catalog package depends on zope.app.component, but doesn't use it anywhere in

Re: [Zope-dev] zope.browser?

2008-12-12 Thread Robert Niederreiter
Hi, Am Freitag, den 12.12.2008, 05:06 +0100 schrieb Roger Ineichen: ... Let's keep this pending and discuss at a later time again. ok. please let me know when there's cleard space for features, regards, robert Regards Roger Ineichen ___

Re: [Zope-dev] zope.browser?

2008-12-12 Thread Christian Zagrodnick
On 2008-12-11 17:13:06 +0100, Roger Ineichen d...@projekt01.ch said: Hi Martijn Betreff: Re: [Zope-dev] zope.browser? Martijn Faassen wrote: Hi there, I saw that Roger Ineichen created and released a package called zope.browser. I assume that this package is intended to reduce

[Zope-dev] C-extension in zope.i18nmessageid

2008-12-12 Thread Malthe Borch
I've branched out this package and removed the C-extension. It's not documented in the package why a C-extension is needed or alternatively, what it benefits. If there are no objections, I will merge this into trunk shortly. \malthe ___ Zope-Dev

Re: [Zope-dev] C-extension in zope.i18nmessageid

2008-12-12 Thread Jacob Holm
Hi Malthe Malthe Borch wrote: I've branched out this package and removed the C-extension. It's not documented in the package why a C-extension is needed or alternatively, what it benefits. If there are no objections, I will merge this into trunk shortly. IIRC, the C extension is

Re: [Zope-dev] C-extension in zope.i18nmessageid

2008-12-12 Thread Marius Gedminas
On Fri, Dec 12, 2008 at 11:28:48AM +, Malthe Borch wrote: I've branched out this package and removed the C-extension. It's not documented in the package why a C-extension is needed or alternatively, what it benefits. C extensions are usually optimizations. If there are no objections, I

Re: [Zope-dev] C-extension in zope.i18nmessageid

2008-12-12 Thread Martijn Pieters
On Fri, Dec 12, 2008 at 11:28, Malthe Borch mbo...@gmail.com wrote: I've branched out this package and removed the C-extension. It's not documented in the package why a C-extension is needed or alternatively, what it benefits. The C extension is required to make messageids immutable. Because

Re: [Zope-dev] C-extension in zope.i18nmessageid

2008-12-12 Thread Malthe Borch
Martijn Pieters wrote: The C extension is required to make messageids immutable. Because they are immutable, the security machinery can treat them as rocks, e.g. safe to pass around. Removing the C-extension undoes this, as you cannot make truely immutable. I believe it is possible to do this

[Zope-dev] Zope Tests: 4 OK, 2 Failed

2008-12-12 Thread Zope Tests Summarizer
Summary of messages to the zope-tests list. Period Thu Dec 11 12:00:00 2008 UTC to Fri Dec 12 12:00:00 2008 UTC. There were 6 messages: 6 from Zope Tests. Test failures - Subject: FAILED (failures=2) : Zope-trunk Python-2.4.5 : Linux From: Zope Tests Date: Thu Dec 11 20:33:39 EST

Re: [Zope-dev] zope.browser?

2008-12-12 Thread Martijn Faassen
Hey, Christian Zagrodnick wrote: [snip] That's good. One thing which is not good is that you deprecated the use of ITerms from zope.app.form. I'd just leave the reference/import there like we did with ISite in zope.app.component. Why is such a deprecation warning bad? Wouldn't this

Re: [Zope-dev] zope.browser?

2008-12-12 Thread Brian Sutherland
On Fri, Dec 12, 2008 at 02:24:09PM +0100, Martijn Faassen wrote: Hey, Christian Zagrodnick wrote: [snip] That's good. One thing which is not good is that you deprecated the use of ITerms from zope.app.form. I'd just leave the reference/import there like we did with ISite in

[Zope-dev] z3c.form bug with radio widget label

2008-12-12 Thread Johannes Raggam
hey, i guess i found a bug in z3c.form.browser.radio. in z3c.form.browser.radio, line 49 (class RadioWidget, def update), the line should read label = term.value instead of label = term.token we found this, while trying to display some unicode characters (umlaute) in radio widget labels.

Re: [Zope-dev] zope.browser?

2008-12-12 Thread Christian Zagrodnick
On 2008-12-12 14:24:09 +0100, Martijn Faassen faas...@startifact.com said: Hey, Christian Zagrodnick wrote: [snip] That's good. One thing which is not good is that you deprecated the use of ITerms from zope.app.form. I'd just leave the reference/import there like we did with ISite in

Re: [Zope-dev] zope.browser?

2008-12-12 Thread Robert Niederreiter
Hi, Am Freitag, den 12.12.2008, 15:51 +0100 schrieb Christian Zagrodnick: On 2008-12-12 14:24:09 +0100, Martijn Faassen faas...@startifact.com said: Hey, Christian Zagrodnick wrote: [snip] That's good. One thing which is not good is that you deprecated the use of ITerms from

Re: [Zope-dev] C-extension in zope.i18nmessageid

2008-12-12 Thread Malthe Borch
Malthe Borch wrote: I believe it is possible to do this in pure Python: The implementation may reviewed in this branch: http://svn.zope.org/zope.i18nmessageid/branches/c-extension-less/ \malthe ___ Zope-Dev maillist - Zope-Dev@zope.org

Re: [Zope-dev] C-extension in zope.i18nmessageid

2008-12-12 Thread Marius Gedminas
On Fri, Dec 12, 2008 at 12:45:27PM +, Malthe Borch wrote: Martijn Pieters wrote: The C extension is required to make messageids immutable. Because they are immutable, the security machinery can treat them as rocks, e.g. safe to pass around. Removing the C-extension undoes this, as you

Re: [Zope-dev] z3c.form bug with radio widget label

2008-12-12 Thread Marius Gedminas
On Fri, Dec 12, 2008 at 02:33:53PM +, Johannes Raggam wrote: hey, i guess i found a bug in z3c.form.browser.radio. in z3c.form.browser.radio, line 49 (class RadioWidget, def update), the line should read label = term.value You can't assume ``value`` will be a string. People

Re: [Zope-dev] C-extension in zope.i18nmessageid

2008-12-12 Thread Malthe Borch
Marius Gedminas wrote: Careful: id(some_object) will likely be reused when the old object is garbage collected. This has been worked around using the weak reference dictionary. \malthe ___ Zope-Dev maillist - Zope-Dev@zope.org

Re: [Zope-dev] C-extension in zope.i18nmessageid

2008-12-12 Thread Jim Fulton
On Dec 12, 2008, at 6:28 AM, Malthe Borch wrote: I've branched out this package and removed the C-extension. It's not documented in the package why a C-extension is needed or alternatively, what it benefits. If there are no objections, I will merge this into trunk shortly. I object.

Re: [Zope-dev] C-extension in zope.i18nmessageid

2008-12-12 Thread Martijn Pieters
On Fri, Dec 12, 2008 at 17:01, Jim Fulton j...@zope.com wrote: On Dec 12, 2008, at 6:28 AM, Malthe Borch wrote: I've branched out this package and removed the C-extension. It's not documented in the package why a C-extension is needed or alternatively, what it benefits. If there are no

Re: [Zope-dev] C-extension in zope.i18nmessageid

2008-12-12 Thread Malthe Borch
Martijn Pieters wrote: I object as well, and have asked for Malthe to provide his reasoning here at the Plone Performance Sprint in Bristol, but so far his only motivation is that he wants to see if he can get this to work without a C-extension. I am sceptical he'll be able to, and am not

Re: [Zope-dev] Dependencies question

2008-12-12 Thread Stephan Richter
On Friday 12 December 2008, Wichert Akkerman wrote: Personally, I think, that extras_require way is a nice compromise for that. Because many of packages can be used greatly without ZCML configuration, but getting ZCML-reqired dependencies should be easy. +1 Those zcml dependencies make

Re: [Zope-dev] C-extension in zope.i18nmessageid

2008-12-12 Thread Martijn Faassen
Malthe Borch wrote: Martijn Pieters wrote: I object as well, and have asked for Malthe to provide his reasoning here at the Plone Performance Sprint in Bristol, but so far his only motivation is that he wants to see if he can get this to work without a C-extension. I am sceptical he'll be

Re: [Zope-dev] C-extension in zope.i18nmessageid

2008-12-12 Thread Martijn Pieters
On Fri, Dec 12, 2008 at 18:51, Martijn Faassen faas...@startifact.com wrote: Unless it can be clearly demonstrated that the new method is equivalent in both performance and security, talk of dropping the C extension seems somewhat premature. A pure Python fallback for this module would however

Re: [Zope-dev] C-extension in zope.i18nmessageid

2008-12-12 Thread Malthe Borch
Martijn Faassen wrote: My suspicion from observing the discussions in this thread so far indicate that a drop in code complexity doesn't seem to be a necessary consequence of rewriting to Python either. The proposed implementation has already been implemented (walk up this thread); it is

Re: [Zope-dev] C-extension in zope.i18nmessageid

2008-12-12 Thread Tres Seaver
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Martijn Faassen wrote: Malthe Borch wrote: Martijn Pieters wrote: I object as well, and have asked for Malthe to provide his reasoning here at the Plone Performance Sprint in Bristol, but so far his only motivation is that he wants to see if he

[Zope-dev] adapting to None

2008-12-12 Thread Chris Withers
Hi All, I have a need to be able to adapting certain objects to None, eg: def some_adapter(obj): if something: return None return somethingelse This is tricky, since returning None from an adapter results in a TypeError. I eventually came up with the idea of having a subclass of

[Zope-dev] Fwd: Help with RelStorage ...

2008-12-12 Thread Gareth Bult
Hi, I'm having a bit of a problem that looks like a 32 bit vs 64 bit issue with RelStorage, can anyone help? If I do a clean install of RelStorage and ZODB on a 32 bit Ubuntu system, I get one error when I run the tests as follows;