Author r.david.murray
Recipients barry, r.david.murray, zooko
Date 2009-10-22.14:41:42
SpamBayes Score 1.13048e-12
Marked as misclassified No
Message-id <1256222504.33.0.120549681776.issue7183@psf.upfronthosting.co.za>
In-reply-to
Content
I added some tests in test_property in my checkout and the __doc__
property is not, as far as I can tell, set read-only by the issue 5890 fix.

The error message referenced in the report is:

AttributeError: 'Boost.Python.StaticProperty' object attribute '__doc__'
is read-only

If Boost.Python.StaticProperty is a subclass of Property, which seems
likely, then the 5890 fix could well be triggering this error, since if
a property created with a subclass is given a docstring, the fix creates
a __doc__ attribute in the subclass's dict.  If the subclass dict is
read only, the above error would be triggered.

I don't know enough about extension types to know if the property code
should be special casing extension types, but if it should, then IMO
that would qualify 5890 as a regression.  Otherwise I don't think it is,
though we could still consider whether making a change to reduce user
pain is worthwhile.  (To be worthwhile it would have to be likely to
reach the users before the Boost fix does...apparently Boos has a fix in
their SVN for this problem.)

I'm guessing that this is a problem only with extension types, not with
pure Python code.
History
Date User Action Args
2009-10-22 14:41:44r.david.murraysetrecipients: + r.david.murray, barry, zooko
2009-10-22 14:41:44r.david.murraysetmessageid: <1256222504.33.0.120549681776.issue7183@psf.upfronthosting.co.za>
2009-10-22 14:41:43r.david.murraylinkissue7183 messages
2009-10-22 14:41:42r.david.murraycreate