Skip to content

Navigation Menu

Appearance settings

Explore
By company size
By use case
By industry
View all solutions
Topics
- AI
- DevOps
- Security
- Software Development
- View all
Explore
- GitHub Sponsors
  Fund open source developers
- The ReadME Project
  GitHub community articles
Repositories
- Enterprise platform
  AI-powered developer platform
Available add-ons
Pricing

Search code, repositories, users, issues, pull requests...

Search

Clear

Search syntax tips

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Name

Query

To see all available qualifiers, see our documentation.

Appearance settings

You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session.

Dismiss alert

rust-lang / rust Public

Notifications You must be signed in to change notification settings
Fork 13.4k
Star 104k

Code
Issues 5k+
Pull requests 734
Actions
Projects 9
Security
Insights

Additional navigation options

Code
Issues
Pull requests
Actions
Projects
Security
Insights

Unlimit UNIX remove_dir_all() implementation#93160

Closed

hkratz wants to merge 18 commits intorust-lang:masterrust-lang/rust:masterfrom

rusticstuff:unix_remove_dir_all_optrusticstuff/rust:unix_remove_dir_all_optCopy head branch name to clipboard

Conversation Commits18 (18)Checks Files changed

Commits

Commits on Jan 30, 2022

WIP: looping version of unix remove_dir_all

cuviper
authored and
hkratz
committed
Temporarily remove special-cased Macos x86-64 implementation.
hkratz
committed
Readd Macos x86-64 remove_dir_all() special case using dynamic symbols.
hkratz
committed
Use limited Readdir cache and use openat(dirf, "..", O_NOFOLLOW) to go up.
hkratz
committed
replace RawFd with BorrowedFd
hkratz
committed
Store and compare with stat.st_dev as well when going back up via ..
hkratz
committed
fix for some UNIXes: parent directory needs to be reopened after deletion of child.
hkratz
committed

Commits on Jan 31, 2022

Add ui test for calling remove_dir_all() on deep directory hierarchies.
hkratz
committed
test: safe depths for all platforms.
hkratz
committed

Commits on Feb 2, 2022

WIP: lazy read dir datatype
hkratz
committed
refill complete cache if empty when going up
hkratz
committed
cosmetics
hkratz
committed
CI filesystem is apparently slow, 64K deep hierarchy takes <2sec. locally
hkratz
committed
correct limit for readdir cache
hkratz
committed
remove debug output (oops)
hkratz
committed

Commits on Feb 5, 2022

better inode reuse protection
hkratz
committed
expand test to include an empty dir/file in each directory
hkratz
committed
include parent of deletion root dir in reused inode check
hkratz
committed

Footer

© 2025 GitHub, Inc.

Footer navigation

Terms
Privacy
Security
Status
Docs
Contact

You can’t perform that action at this time.