-
Notifications
You must be signed in to change notification settings - Fork 13.3k
BREAKING - Upgrade to upstream newlib 4.0.0 release #7708
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
Changes from 1 commit
Commits
Show all changes
20 commits
Select commit
Hold shift + click to select a range
1395ef9
Upgrade to upstream newlib 3.3.0 release
earlephilhower 43a8df3
Upgrade to Newlib 4.0.0 - needs toochain rebuild
earlephilhower cb3aa7a
Merge branch 'master' into newlib330
earlephilhower b574826
Add new include files part of newlib update
earlephilhower 418f267
Clean rebuild to newlib 4.0.0
earlephilhower a50843a
Merge branch 'master' of https://github.com/esp8266/Arduino into newl…
earlephilhower 7faf5d8
Remove unneeded include file
earlephilhower e989171
Modified local isXXX functions in newlib
earlephilhower 5ba4d33
Merge branch 'master' into newlib330
earlephilhower 8298952
Adjust macro defined isXXX_l character classifiers
earlephilhower 9540bbe
Dieable CI caching and fix time header include
earlephilhower 38f12f0
The tools/sdk/libc directory isn't used, remove it
earlephilhower 7d36d28
Fix 64-bit time for LittleFS
earlephilhower e9f7823
Update to jjsuwa-sys3175 additions to GCC and newlib
earlephilhower f15f254
Merge branch 'master' of https://github.com/esp8266/Arduino into newl…
earlephilhower 58476d0
Rebuild w/addl GCC patches, new BearSSL flags
earlephilhower f66e46a
Remove one more redundant lib.a file
earlephilhower 25b4bc4
Merge branch 'master' of https://github.com/esp8266/Arduino into newl…
earlephilhower 01e7c85
Rebuild eboot with new G++
earlephilhower da5621d
Reapply GH CI caching to pull-requests
earlephilhower File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
iirc, changing 'key' will force it to use a different cache
it will be expired at some point, but it is not as clear as something like travis (rip)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thx. I think generally we're OK and the un-done patch here was just a stopgap during development.
The original commenting-out was because the releases have a datestamp in them which ends up making all builds in a day conflict. So get.py would find gcc-20121101.tgz(from 9AM) in the cached dir and not try and d/l the newer one gcc-20121101.tgz(from 11AM). The cached version's MD5 would mismatch, so the build failed w/o a toolchain. Easy fix, drop the caching. W/everything coming from GH.com, anyway, the cache probably doesn't make much difference...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
True, speeds on github are really good, so cache most of the time takes longer than just fetching the sources directly...
I think it also could be this 'template', making key depend on the .json so it is always new whenever toolchain is modified
via https://github.com/actions/cache#creating-a-cache-key
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's an awesome idea! Right now the cache will grow w/obsolete versions of the tools, but one based on the JSON would fix that problem.
Did you want the PR for this? OTW I can throw it together quickly.