Am 18.02.2014 10:07, schrieb Georg Brandl:
> Am 18.02.2014 09:55, schrieb tetburr:
>> I have identified where the exception is coming from, and I think I have a 
>> fix,
>> but I'm not entirely sure whether it is valid.
>> 
>> Location: sphinx.util.inspect.is_builtin_class_method()
>> 
>> Problem: __builtins__ is used directly, however Jython does not define
>> __builtins__, therefore an exception is raised.
>> 
>> Possible Solution: use the __builtin__ module (instead of the __builtins__
>> produced by CPython when __name__ == "__main__")
>> 
>> Implementation:
>> At top of 'inspect.py': add import __builtin__
>> Lines 155 and 157: change __builtins__ to __builtin__
>> 
>> Test: With these changes the example provided in the DocString
>> (is_builtin_class_method(int, '__init__')) does return True, rather than 
>> raising
>> the 'not defined' exception.
>> 
>> How can I go about getting this reviewed, and so if appropriate incorporated 
>> in
>> the next release?
> 
> Hi,
> 
> thanks for the research, I'll fix this for 1.2.2 and 1.3.

And committed now.

Georg

-- 
You received this message because you are subscribed to the Google Groups 
"sphinx-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sphinx-users.
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to