-
-
Notifications
You must be signed in to change notification settings - Fork 32
Expose more directory iteration methods #33
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
base: main
Are you sure you want to change the base?
Expose more directory iteration methods #33
Conversation
|
The release 2.6 of littlefs should soon include the fix for the bug that makes this PR fail littlefs-project/littlefs#814 |
5a900ed to
39475fc
Compare
39475fc to
1447766
Compare
|
Rebased on top of #42 , so the tests pass now |
760bb7f to
6f0a3c9
Compare
eca2592 to
eac3a43
Compare
|
Can you please rebase again? |
eac3a43 to
b8def7e
Compare
|
Thinking about how to make this available from the |
b8def7e to
837c389
Compare
837c389 to
4fb156b
Compare
|
Making this available from As part of the trussed ecosystem this is only needed in the core trussed backend which already depends on the full |
But since trussed-dev/trussed#144 it uses Could we add a new function to |
|
Oh right. Surprisingly cargo-semver-checks does not flag this as a breaking change in I'm investigating further if this is not a false positive |
|
I actually see no way of exposing this as part of This is a breaking change because an existing implementation of |
|
I could see adding a new dyn compatible trait (with better names): pub trait DynFilesystemV2: DynFilesytem {
fn read_dir_and_then_unitv2(…); |
Fix #32
Tests fail because of littlefs-project/littlefs#785