FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Revert "GH-116380: Speed up `glob.[i]glob()` by making fewer system calls. (#116392)" by barneygale · Pull Request #130743 · python/cpython · GitHub

Repository navigation

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

Merged
barneygale merged 1 commit into
python:mainfrom
barneygale:revert-glob-speedup
Mar 1, 2025
Merged

barneygale merged 1 commit into
python:mainfrom
barneygale:revert-glob-speedup

Conversation

barneygale commented Mar 1, 2025 •
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Contributor

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 requested a review from picnixz March 1, 2025 19:36
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

picnixz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

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

Copy link
Copy Markdown
Contributor Author

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

barneygale enabled auto-merge (squash) March 1, 2025 19:43

picnixz commented Mar 1, 2025

Copy link
Copy Markdown
Member

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)

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL