[Checkins] SVN: z3c.ptcompat/trunk/ - Added support for browser:addform, browser:editform, browser:form,

Sidnei da Silva sidnei.da.silva at gmail.com
Tue Apr 7 00:07:36 EDT 2009


Hi Stephan,

On Sun, Mar 15, 2009 at 12:34 AM, Stephan Richter
<srichter at cosmos.phy.tufts.edu> wrote:
> On Tuesday 10 March 2009, Stephan Richter wrote:
>> On Monday 09 March 2009, Sidnei da Silva wrote:
>> > +from zope.app.form.browser import metaconfigure as formmeta
>>
>> Ahh, you just added a dependency on zope.app.form! This should clearly go
>> into another package. A lot of people use z3c.ptcompat that are not using
>> zope.app.form.
>
> Sidnei, you have not reverted your change, could you please do so? This is a
> really awful dependency.

I've done as you asked, and made the import conditional.

However, I can't see how this can be conditional at all (in fact, I
couldn't setup an environment to test it!) due to the fact that
zope.app.form is a dependency for zope.app.security and zope.formlib,
which apparently get pulled by zope.app.publisher, which z3c.ptcompat
depends on. I could be wrong though, so please correct me!

-- 
Sidnei da Silva
Canonical Ltd.
 Landscape · Changing the way you manage your systems
http://landscape.canonical.com


More information about the Checkins mailing list