[issue11591] "python -S" should be robust against e.g. "from site import addsitedir"

Éric Araujo report at bugs.python.org
Tue Mar 22 00:12:36 CET 2011


Éric Araujo <merwok at netwok.org> added the comment:

Looks great, thank you.  I think I’ll also add a docstring to main before committing, now that the function is publicly documented.

Did you have to manually click “Create Patch” to make roundup generate it?  Did you try first to click on the button of the existing repo before adding a new repo entry?  (Still learning how to use it, thanks for experimenting along :)

Brett: Thanks for the review.  If you don’t comment negatively on the doc change, I will commit this.

(Side concern: the module does not define __all__, even though only 4 functions and 4 constants are officially documented.  I’d like to define __all__, but the recentish huge thread on public/private APIs scared me.)

----------

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue11591>
_______________________________________


More information about the Python-bugs-list mailing list