-
Notifications
You must be signed in to change notification settings - Fork 81
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
Change: avr-gcc@14 formula to use avr-libc 2.2.1 #344
Conversation
If this is all good, I'll open PRs for the other version-specific |
Thanks @retsplines! I've started the CI |
@retsplines ci failed. If you look at the logs it should be some easy fixes. You can run the commands locally to check for errors |
Fixed the style/lint error. There was one other which relates to a missing dependency on |
I'm going to deprecate gcc@5 and gcc@6 as homebrew did Homebrew/homebrew-core#106902 I'll let you know when I'm done so you can rebase on main |
@retsplines done, I've updated it myself |
7ff82a2
to
d041ec1
Compare
@fauxpark on macos-15 it fails because
would you know why it says so? I have the same error if I run the process on my machine |
ah I think it's because of binutils, i'll bump the formula |
Yeah, dependencies always have to be bottled first. Was gonna get that PRed in a bit. |
I hope #347 does the trick I'll be out for the rest of the day, if you have time feel free to edit my PR to get it working if needed :) |
d041ec1
to
2539532
Compare
@retsplines @fauxpark I've done the bottle upload manually as CI was failing. |
The pr-pull failed because the release already exists - this PR needed to also add |
avr-libc
2.2.1 adds support for many new Microchip devices.The changes to io.h give an idea of the number of new chips supported.
Another user opened a PR for this which ended up becoming stale due to some outstanding style changes and the desire to not change multiple formulae per PR, so this is re-raised as a separate PR per GCC version.