Skip to content

Updated to download mkspiffs 0.2.2 #1159

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

Merged
merged 2 commits into from
Mar 4, 2018
Merged

Conversation

rfestag
Copy link
Contributor

@rfestag rfestag commented Feb 28, 2018

Updated the package template (used by tools/get.py) to download the new mkspiffs tool. This is in reference to #1022, and is necessary because of a change implemented in Dec 2017. Currently, the mkspiffs that is installed by get.py does not work with arduino-esp32, causing problems for https://github.com/me-no-dev/arduino-esp32fs-plugin.

@copercini
Copy link
Contributor

I think it will be necessary to update the SHA-256 hashes of checksum and filesize too, otherwise, the get.py will detect it as corrupt file

@rfestag
Copy link
Contributor Author

rfestag commented Mar 1, 2018

It looks like the sha256 hash checking is commented out, and seems to work properly right now. However, in case that changes in the future, it certainly makes sense to update. When I get back this evening I'll put those together and push an update.

@rfestag
Copy link
Contributor Author

rfestag commented Mar 1, 2018

I just updated the hashes and file sizes. It also turns out that the windows filename changed. I only tested the linux64 version. I also uncommented the hash checking, and it appears to work correctly. Not sure why it is currently commented out, so I didn't commit that change.

@me-no-dev me-no-dev merged commit 1256f2e into espressif:master Mar 4, 2018
@me-no-dev
Copy link
Member

Thanks!

Curclamas pushed a commit to Curclamas/arduino-esp32 that referenced this pull request Aug 21, 2018
* Updated to download mkspiffs 0.2.2

* Updated to have proper hashes and sizes. Also updated to new windows file name
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.

3 participants