Skip to content
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

Get all versions of GCC working on Apple ARM systems. #249

Closed
wants to merge 1 commit into from

Conversation

DavidEGrayson
Copy link
Contributor

@DavidEGrayson DavidEGrayson commented Jun 17, 2021

This just takes the changes that we did to avr-gcc@11 for ARM support and applies them to all the other GCC versions.

The patch I used for GCC 11 happens to work for all the other versions, and I tested that it is at least necessary for GCC 9. I assume it is necessary for all the other versions too.

I did not test this pull request out on any hardware, but after installing the formulas successfuly on an M1 Mac Mini, I ran these commands successfully:

brew test ./Formula/avr-gcc@*
brew audit ./Formula/avr-gcc@*
brew style ./Formula/avr-gcc@*

I also used commands like diff -ur avr-gcc@11.rb avr-gcc@9.rb to make sure that I remembered to apply all the changes.

And I added GCC 11 and GCC 5 to the README since it seems like they were missing.

Also add missing versions of GCC to the README.
@ladislas
Copy link
Member

Thanks @DavidEGrayson! It would be better to do separate PRs for each formula, because CI will try to build them all and it will take forever...

Copy link
Member

@ladislas ladislas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

well it worked! Looks great! I'll merge now :)
Thanks again for all of this!

Comment on lines +10 to +18
- GCC 5.5.0 - provided as `avr-gcc@5`
- GCC 8.4.0 - provided as `avr-gcc@8`
- GCC 10.2.0 - provided as `avr-gcc@10`
- GCC 11.1.0 - provided as `avr-gcc@11`
- Binutils 2.35.1 - provided as `avr-binutils`
- AVR Libc 2.0.0 - provided as a ressource for each GCC formula
- AVR Libc 2.0.0 - provided as a resource for each GCC formula
- GDB 10.1 - provided as `avr-gdb`

Support for older GCC versions (4, 5, 6, 7) has been removed. Please, raise an issue if you need one back.
Support for older GCC versions (4, 6, 7) has been removed. Please, raise an issue if you need one back.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

neat! Good catch :)

@DavidEGrayson
Copy link
Contributor Author

Great, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants