Florent Xicluna wrote:
Log message for revision 70047:
  Make deprecated packages zope.app.layers and zope.app.skins optional.
  By the way, a zcml feature 'deprecatedlayers' is added.


Hi Florent,

has there been a proposal for this change? Changes that might affect backward compability should be discussed before implemetented. I for one am questioning...

a) the usefulness of this change (the layer code is going to go away soon anyways)

b) potential problems with backward compatibility (it seems that the deprecatedlayers feature needs to be turned on to get backward compatibility)

I also see you've been committing and reverting this change more than once. Our checkin policy is that all unit and functional tests must pass. For invasive changes such as this one, it is also always a good idea to test Zope startup as well.

Either way, I think we should discuss this change first.

Philipp
_______________________________________________
Zope3-dev mailing list
Zope3-dev@zope.org
Unsub: http://mail.zope.org/mailman/options/zope3-dev/archive%40mail-archive.com

Reply via email to