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.

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

Revert "GH-116380: Speed up glob.[i]glob() by making fewer system calls. (#116392)" #130743

Merged
merged 1 commit into from
Mar 1, 2025

Conversation

barneygale
Copy link
Contributor

@barneygale barneygale commented Mar 1, 2025

This broke tests on the 'aarch64 Fedora Stable Clang Installed 3.x' and 'AMD64 Fedora Stable Clang Installed 3.x' build bots.

This reverts commit da4899b.


📚 Documentation preview 📚: https://cpython-previews--130743.org.readthedocs.build/

…stem calls. (python#116392)"

This broke tests on the 'aarch64 Fedora Stable Clang Installed 3.x' and
'AMD64 Fedora Stable Clang Installed 3.x' build bots.

This reverts commit da4899b.
@barneygale barneygale requested a review from picnixz March 1, 2025 19:36
@barneygale barneygale changed the title Revert "GH-116380: Speed up glob.[i]glob() by making fewer system calls. (#116392) Revert "GH-116380: Speed up glob.[i]glob() by making fewer system calls. (#116392)" Mar 1, 2025
Copy link
Member

@picnixz picnixz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If this a simple application of git revert the I think I can approve it without looking at it too long right?

@barneygale
Copy link
Contributor Author

Indeed, it's a git revert with no embellishments :)

@barneygale barneygale enabled auto-merge (squash) March 1, 2025 19:43
@picnixz
Copy link
Member

picnixz commented Mar 1, 2025

In this case (and in the future), I think you don't need my approval then :) (it would be too hard to actually check that the revert is correct using the web UI)

@barneygale barneygale merged commit 5326c27 into python:main Mar 1, 2025
48 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants