-
-
Notifications
You must be signed in to change notification settings - Fork 345
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
Add IDF caching for libraries #2850
Conversation
Account for change from c:/tools/ to d:/opt/, and that Windows idf path contains version.
PR Summary
|
I'm impressed with that summary from 'what-the-diff' - it's actually useful :-) |
Changing to WIP, library caching not quite working as expected for Ubuntu. |
Yep, turning build debugging on it's using relative paths when restoring the cache. Looks like the cache for IDF was saved in Windows and ended up being restored to a sub-folder somewhere. I'll have a dig about and see if the damn thing can use absolute paths like it's been given. |
Hang on, what's |
42a0876
to
928467a
Compare
Link warnings don't have a source reference, but include them Use Parser class to manage state Use methods for state Speed up using regex
Cache esp-idf and tools for library builds. Be nice if we could use the cache from
SmingHub/Sming
directly but that sort of cross-repo sharing isn't supported.Also discovered that cross-OS caching breaks because the cache action uses paths relative to the build directory, even though it's been given absolute paths. So, just simplify the thing and use a separate, combined cache for each OS/IDF combination. Total's about 5GB.
In all the .yml files we don't need to repeat steps for Ubuntu/MacOS as they're the same, so we can use a combined step for those, ie.
not windows
.Scanlog tool:
Also fix a few more warnings.