This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author ncoghlan
Recipients ezio.melotti, ncoghlan, python-dev, r.david.murray, serhiy.storchaka, skrah, tim.peters
Date 2013-10-27.05:39:40
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1382852381.02.0.442698385305.issue19412@psf.upfronthosting.co.za>
In-reply-to
Content
On my Fedora system, test_gdb still fails when --without-doc-strings is configured, even after updating the decorator to cover both C and Python docstrings as an interim fix. That may be a real bug, though (see issue 19415 for details)

Regarding splitting the flags, I'm not sure I agree with RDM's reasoning in issue 17041. As soon as *any* docstrings are missing (whether it's C docstrings or Python docstrings), various parts of the introspection machinery are going to start behaving strangely.

By splitting the flags in the test suite, we're increasing the coupling between the tests and the implementation. For example, if something docstring dependent that was previously implemented in Python is moved to a C accelerator module, is it right for the test to have to know about that and switch to moving a different decorator?

In my view, "this is a C docstring" and "this is a Python docstring" is an implementation detail that the tests shouldn't have to care about. With the latest update, "test.supports.requires_docstrings" means "I require trustworthy docstrings, and if there's anything wrong with the docstring machinery, don't even try this test" without getting into the specifics of worrying about *why* the docstrings might be unreliable.
History
Date User Action Args
2013-10-27 05:39:41ncoghlansetrecipients: + ncoghlan, tim.peters, ezio.melotti, r.david.murray, skrah, python-dev, serhiy.storchaka
2013-10-27 05:39:41ncoghlansetmessageid: <1382852381.02.0.442698385305.issue19412@psf.upfronthosting.co.za>
2013-10-27 05:39:40ncoghlanlinkissue19412 messages
2013-10-27 05:39:40ncoghlancreate