-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Fix the issue #3612 , Updating the SPIFFS to f5e26c4 #3623
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
Conversation
Thanks for the PR! To make it easier to follow SPIFFS updates in the future, would you mind adding SPIFFS commit ID to your commit description (e.g. "SPIFFS: update to f5e26c4")? I will then update this branch by adding new mkspiffs release. |
Done I will Update this PR description too |
@igrr are you postponing this to be in sync with mkspiffs? |
Yes, I still need to update package links for mkspiffs, and revert the SPIFFS config option which caused compatibility issues with 2.3.0. |
about SPIFFS config options .. what are the options that are causing the compatibility issues ?? |
According to @me-no-dev it is https://github.com/igrr/mkspiffs/blob/master/spiffs/spiffs_config.h#L202, but it would be nice to test that (i.e. prepare FS with older version, try to read it with the new one). |
OK I got that .. I will test it now |
I would vote for a separate PR just so it outlines the change, but whatever the boss says :) |
Well, I didn't mean adding these extra changes to this PR. It can go in as is. Just would make sense to merge all changes (roughly) at the same time, to prevent someone from grabbing non-compatible versions of mkspiffs and arduino-esp8266. |
I tested the changes .. it seems OK and the compatibility is maintained across the different versions and I tested with various files some of them well above 25k and it's Perfectly OK .. I will make another pull request in the mkspiffs with the this change .. actually I've been meaning to ask .. Do you have a Guide on how to make a proper PRs and mastering git commands ??? sometimes I have difficulties with git in how to manage the remote fork I am making modifications to .. |
@me-no-dev , actually this change is directly related to the issue mentioned in the PR's title |
the same suggested modification in this pull request igrr/mkspiffs#27
further info in this issue pellepl/spiffs#165 and the commit pellepl/spiffs@f5e26c4
also don't forget to update the mkspiffs release
I guess this can only be done by you 😃