Skip to content

tempfile module and shutil.rmtree fix#573

Closed
BrianPugh wants to merge 5 commits intomicropython:masterfrom
BrianPugh:tempfile
Closed

tempfile module and shutil.rmtree fix#573
BrianPugh wants to merge 5 commits intomicropython:masterfrom
BrianPugh:tempfile

Conversation

@BrianPugh
Copy link
Contributor

shutil.rmtree had a reference to os.walk, which isn't in micropython. I updated it to use os.ilistdir and added unittests for it.

Initial implementation of the tempfile module. It currently has the following implemented:

  • mkdtemp
  • TemporaryDirectory

I only ever use TemporaryDirectory, and I assume others may have similar experience.

This is to make it much easier for me to update my pathlib unittests to be micropython-compatible.

@BrianPugh BrianPugh mentioned this pull request Nov 10, 2022
@dpgeorge
Copy link
Member

Thank you, these are good improvements and additions!

Merged in ee286ed through a99b801 with reworded commit messages.

I also added commit 56dc65b which adds manifest.py to `tempfile.

@dpgeorge dpgeorge closed this Nov 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants