[Zope-dev] Re: [Checkins] SVN: zope.app.securitypolicy/trunk/setup.py This package doesn't just work with any versin of zope.i18nmessageid, it needs
On Friday 09 November 2007, Philipp von Weitershausen wrote: > Modified: zope.app.securitypolicy/trunk/setup.py > === > --- zope.app.securitypolicy/trunk/setup.py 2007-11-09 13:08:59 UTC > (rev 81634) +++ zope.app.securitypolicy/trunk/setup.py 2007-11-09 > 14:52:52 UTC (rev 81635) @@ -58,7 +58,7 @@ > 'zope.configuration', > 'zope.exceptions', > 'zope.i18n', > - 'zope.i18nmessageid', > + 'zope.i18nmessageid >= 3.4.2', > 'zope.interface', > 'zope.location', > 'zope.schema', Please let's stop putting versions into setup.py. This is a really bad idea, since it will make it very hard to work with in different project with different version requirements. Regards, Stephan -- Stephan Richter CBU Physics & Chemistry (B.S.) / Tufts Physics (Ph.D. student) Web2k - Web Software Design, Development and Training ___ Zope-Dev maillist - Zope-Dev@zope.org 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] Re: [Checkins] SVN: zope.app.securitypolicy/trunk/setup.py This package doesn't just work with any versin of zope.i18nmessageid, it needs
Stephan Richter wrote: On Friday 09 November 2007, Philipp von Weitershausen wrote: Modified: zope.app.securitypolicy/trunk/setup.py === --- zope.app.securitypolicy/trunk/setup.py 2007-11-09 13:08:59 UTC (rev 81634) +++ zope.app.securitypolicy/trunk/setup.py 2007-11-09 14:52:52 UTC (rev 81635) @@ -58,7 +58,7 @@ 'zope.configuration', 'zope.exceptions', 'zope.i18n', - 'zope.i18nmessageid', + 'zope.i18nmessageid >= 3.4.2', 'zope.interface', 'zope.location', 'zope.schema', Please let's stop putting versions into setup.py. This is a really bad idea, since it will make it very hard to work with in different project with different version requirements. Only the '==' qualifier in setup.py is a bad idea because it locks you down. Other qualifiers are actually quite necessary sometimes. In this case, for instance, we *need* the >= qualifier because zope.app.securitypolicy won't work with just any version of zope.i18nmessageid. It needs at least version 3.4.2 because that one introduced the ZopeMessageFactory object which zope.app.securitypolicy wants to import. Such considerations must always be made when you add a feature to package A, and then use that new feature in package B. Package B must then require at least that new version of package A. ___ Zope-Dev maillist - Zope-Dev@zope.org 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] Re: [Checkins] SVN: zope.app.securitypolicy/trunk/setup.py This package doesn't just work with any versin of zope.i18nmessageid, it needs
Philipp von Weitershausen wrote: [snip] Only the '==' qualifier in setup.py is a bad idea because it locks you down. Other qualifiers are actually quite necessary sometimes. Agreed. In addition hopefully at some point we'll get 'or' support, so you can use == again: zope.i18nmessageid >= 3.4.2 or zope.i18nmessageid == 3.4.2 A tool like buildout can then choose to pick the more specific version indicators if there is nothing else overriding it. This would solve a ton of problems we're currently trying to use external indexes for (either Grok's list or KGS). Once this is in place, the only use for an external package index would be to make sure no releases are ever deleted, or changed without updating the version number. I know I sound like a broken record on this, but fixing versions in indexes is really the wrong place: the version requirements should be in the releases themselves. Any index approach will need as many indexes as there are frameworks and applications. There just needs to be a way to override them from outer dependencies. Regards, Martijn ___ Zope-Dev maillist - Zope-Dev@zope.org 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 )