Skip to content
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.

Sign up for GitHub

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

Jump to bottom

os.__all__ is incomplete #62754

Closed
ronaldoussoren opened this issue Jul 25, 2013 · 6 comments
Closed

os.__all__ is incomplete #62754

ronaldoussoren opened this issue Jul 25, 2013 · 6 comments
Assignees
@ronaldoussoren
Labels
stdlib Python modules in the Lib dir type-bug An unexpected behavior, bug, or error

Comments

@ronaldoussoren
Copy link
Contributor

BPO 18554
Nosy @ronaldoussoren, @vadmium, @serhiy-storchaka, @1st1
Files
  • incomplete-os-all.txt
    		•

    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:

    assignee = 'https://github.com/ronaldoussoren'
closed_at = <Date 2014-09-26.16:34:24.315>
created_at = <Date 2013-07-25.13:44:58.643>
labels = ['type-bug', 'library']
title = 'os.__all__ is incomplete'
updated_at = <Date 2014-09-26.16:34:24.314>
user = 'https://github.com/ronaldoussoren'

    bugs.python.org fields:

    activity = <Date 2014-09-26.16:34:24.314>
actor = 'yselivanov'
assignee = 'ronaldoussoren'
closed = True
closed_date = <Date 2014-09-26.16:34:24.315>
closer = 'yselivanov'
components = ['Library (Lib)']
creation = <Date 2013-07-25.13:44:58.643>
creator = 'ronaldoussoren'
dependencies = []
files = ['31033']
hgrepos = []
issue_num = 18554
keywords = ['patch', 'needs review']
message_count = 6.0
messages = ['193696', '209846', '225512', '225751', '227631', '227632']
nosy_count = 5.0
nosy_names = ['ronaldoussoren', 'python-dev', 'martin.panter', 'serhiy.storchaka', 'yselivanov']
pr_nums = []
priority = 'normal'
resolution = 'fixed'
stage = 'commit review'
status = 'closed'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue18554'
versions = ['Python 2.7', 'Python 3.4', 'Python 3.5']
    @ronaldoussoren
    Copy link
    Contributor Author

    On Unix the __all__ of the os module doesn't list the names imported from the posix module.

    The attached patch ensures that exports from the posix module are added to __all__, just like they are for NT and for Python 2.7.

    @ronaldoussoren ronaldoussoren added stdlib Python modules in the Lib dir type-bug An unexpected behavior, bug, or error labels Jul 25, 2013
    @1st1
    Copy link
    Member

    1st1 commented Jan 31, 2014

    bump?

    @vadmium
    Copy link
    Member

    vadmium commented Aug 19, 2014

    I ran into this today, trying to do “help(os)”. The workaround was to do “import posix; help(posix)”.

    @serhiy-storchaka
    Copy link
    Member

    LGTM. Please commit.

    @python-dev
    Copy link
    Mannequin

    python-dev mannequin commented Sep 26, 2014

    New changeset 7230978647a8 by Yury Selivanov in branch 'default':
    os: Include posix functions in os.__all__. Closes issue bpo-18554.
    https://hg.python.org/cpython/rev/7230978647a8

    @1st1
    Copy link
    Member

    1st1 commented Sep 26, 2014

    Thanks for the patch.

    I've committed this to 3.5 only, as there is a slight chance that it breaks backwards compatibility for some scripts.

    @1st1 1st1 closed this as completed Sep 26, 2014
    @ezio-melotti ezio-melotti transferred this issue from another repository Apr 10, 2022
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Assignees

    @ronaldoussoren ronaldoussoren

    Labels
    stdlib Python modules in the Lib dir type-bug An unexpected behavior, bug, or error
    Projects
    None yet
    Milestone
    No milestone
    Development

    No branches or pull requests
    4 participants
    @1st1 @ronaldoussoren @vadmium @serhiy-storchaka