On Wed, Feb 16, 2011 at 11:09 AM, Thomas Lotze <[email protected]> wrote: > Hi, > > the buildout section used to have an option named "extended-by" that has > been deprecated for quite a while. The option is not (no longer?) present > in documentation and tests and the only thing left of it is the > implementation and a check that is supposed to result in a deprecation > warning. However, there's a bug in the implementation that looks as if it > was caused by some refactoring, which would cause a NameError to be raised > instead of the deprecation warning being printed. > > To me, this looks as if it was safe to just remove any traces of the > option, including the buggy implementation. The only downside would be > that configurations which still use the option would fail silently > instead of raising the NameError. I don't consider this to be of much > importance in practice, but before going ahead and removing code, I wanted > to give people a chance to disagree. Thank you.
+1 Perhaps add a check and raise an error if the option is present. Jim -- Jim Fulton http://www.linkedin.com/in/jimfulton _______________________________________________ Distutils-SIG maillist - [email protected] http://mail.python.org/mailman/listinfo/distutils-sig
