New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
mailbox.__all__ list is incomplete #71294
Comments
That's a child issue of bpo-23883, created to propose a patch fixing mailbox module's __all__ list. |
Rather than adding the new class to the list of tests, we could take this opportunity to convert this test file to the new format, unless there's some reason that doesn't work (I don't expect there to be). |
David, what do you mean by "new format"? Can you point me to some existing code? |
Look at your mimetypes patch, for example. That if __name__='__main__' clause is all that is needed. The test_main stuff can be dropped. The question, I suppose, is the reap_children call, and that may be why it hasn't been converted yet. |
Yes, the reap_children call is the culprit. I think resolving this problem is out of the scope of this simple issue (there is separate issue for this). |
This patch looks good to me. I agree the reap_children() thing is a separate problem. |
New changeset 4705b7597c86 by Martin Panter in branch 'default': |
New changeset d62e57958e7c by Martin Panter in branch 'default': |
FTR the closest thing I could find discussing reap_children() is bpo-16968. |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: