esp_littlefs icon indicating copy to clipboard operation
esp_littlefs copied to clipboard

Add support for mutliple backends

Open raldone01 opened this issue 3 years ago • 2 comments

I don't expect this pull request to get merged as the code changes are very drastic. The sdcard and ram backends are poorly tested. The sdcard backend should work though.

I really just want to show you what I have done. Your vfs abstraction really sped things up. Thanks for your work.

If you want add support for multiple backends you may be able to use some of my code in the future.

raldone01 avatar Dec 21 '20 10:12 raldone01

this is quite the PR. I've only skimmed it, I'll try and give it a thorough read when i have more time. Thank you for contributing!

BrianPugh avatar Dec 22 '20 19:12 BrianPugh

The first merge should be fine but it was a mistake using the github desktop app for the first time.

I added another commit to the pull request which adds a config option to allow unlink to also delete empty directories which is required if someone wants to use the ghc::filesystem on top of this as it only calls the remove function to delete folders and files. The std::filesystem is not yet functional even when the cpp17 flag is used.

raldone01 avatar Jan 10 '21 15:01 raldone01

I'm going to close this PR; it was a very cool contribution, but also quite large. To me, this was more of a puppy than a cake 😄 .

We now have SD support due to #170, and I think that was the most requested by users.

BrianPugh avatar May 04 '24 04:05 BrianPugh

Understandable. SD support was the main reason for my changes. I actually think it's overcomplicated for what it does.

raldone01 avatar May 04 '24 16:05 raldone01