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

Remove macpath module #79652

Closed
vstinner opened this issue Dec 12, 2018 · 5 comments
Closed

Remove macpath module #79652

vstinner opened this issue Dec 12, 2018 · 5 comments
Labels
3.8 only security fixes OS-mac stdlib Python modules in the Lib dir

Comments

@vstinner
Copy link
Member

BPO 35471
Nosy @ronaldoussoren, @vstinner, @ned-deily
PRs
  • bpo-35471: Remove the macpath module #11129
    		•

    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 = None
closed_at = <Date 2018-12-14.12:39:54.066>
created_at = <Date 2018-12-12.15:22:57.356>
labels = ['OS-mac', '3.8', 'library']
title = 'Remove macpath module'
updated_at = <Date 2018-12-14.12:39:54.065>
user = 'https://github.com/vstinner'

    bugs.python.org fields:

    activity = <Date 2018-12-14.12:39:54.065>
actor = 'vstinner'
assignee = 'none'
closed = True
closed_date = <Date 2018-12-14.12:39:54.066>
closer = 'vstinner'
components = ['Library (Lib)', 'macOS']
creation = <Date 2018-12-12.15:22:57.356>
creator = 'vstinner'
dependencies = []
files = []
hgrepos = []
issue_num = 35471
keywords = ['patch']
message_count = 5.0
messages = ['331699', '331700', '331702', '331706', '331822']
nosy_count = 3.0
nosy_names = ['ronaldoussoren', 'vstinner', 'ned.deily']
pr_nums = ['11129']
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue35471'
versions = ['Python 3.8']
    @vstinner
    Copy link
    Member Author

    The module 'macpath' has been deprecated in Python 3.7 by bpo-9850 and scheduled for removal in Python 3.8. Attached PR removes the module.

    @vstinner vstinner added 3.8 only security fixes stdlib Python modules in the Lib dir labels Dec 12, 2018
    @vstinner
    Copy link
    Member Author

    According to the PEP-11, MacOS 9 support has been dropped in Python 2.4 (released in 2004: 14 years ago!). So I think that it's now ok to remove the macpath module, especially because it's deprecated since Python 3.7.

    @ronaldoussoren
    Copy link
    Contributor

    I'm in favour of removing macpath.

    I was against removing the module in the past because parts of the macOS API still used (on probably still use) classic MacOS style paths. Most if not all APIs are by this time deprecated, which makes it less and less like that anyone will use macpath going forward.

    @vstinner
    Copy link
    Member Author

    I was against removing the module in the past because parts of the macOS API still used (on probably still use) classic MacOS style paths. Most if not all APIs are by this time deprecated, which makes it less and less like that anyone will use macpath going forward.

    bpo-9850 has a scary title: "obsolete macpath module dangerously broken and should be removed". Extract:

    "Even if one did have a need to use the obsolete old-style paths, the macpath module is currently practically unusable for anything other than simple character manipulations of the path. Nearly all of the functions that actually call OS routines are broken in one or more ways."

    @vstinner
    Copy link
    Member Author

    New changeset d7538dd by Victor Stinner in branch 'master':
    bpo-35471: Remove the macpath module (GH-11129)
    d7538dd

    @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
    No one assigned
    Labels
    3.8 only security fixes OS-mac stdlib Python modules in the Lib dir
    Projects
    None yet
    Milestone
    No milestone
    Development

    No branches or pull requests
    3 participants
    @vstinner @ronaldoussoren @serhiy-storchaka