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

Add AVR-GCC 13.2.0 formula #321

Closed
wants to merge 2 commits into from

Conversation

davidstritzl
Copy link
Contributor

The new formula is based on the formula for AVR-GCC 12.2.0. The URL, mirror, checksum and patches by the Darwin GCC maintainer were updated to GCC 13. The compile test is kept the same as the produced hex file seems to be unchanged. Testing was done on macOS 14.1 (AArch64).

Furthermore, a new entry for AVR-GCC 13.2.0 was added to the README.

@davidstritzl
Copy link
Contributor Author

Signed commits and rebased onto the latest main.

It should also be noted that I have been succesfully using this formula to target multiple different ATmega microcontrollers for the past few months.

@ladislas
Copy link
Member

ladislas commented May 8, 2024

thanks for the PR! can you check the errors and fix the style? :)

@davidstritzl
Copy link
Contributor Author

The style violations should be fixed now.

I also noticed I wasn't building against the latest Xcode/CommandLineTools. With Xcode 15.3, I got some compilation errors for libc++ (see this GCC issue). I included the patches from the Homebrew Core GCC 13 formula to fix the compilation.

@ladislas
Copy link
Member

ladislas commented May 9, 2024

@davidstritzl there are still some issues

@davidstritzl
Copy link
Contributor Author

That's too bad. I think I have another machine capable of macOS 12 around. I'll have to get it up to speed, though, so it will have to wait until the weekend.

Signed-off-by: David Stritzl <[email protected]>
Signed-off-by: David Stritzl <[email protected]>
@davidstritzl
Copy link
Contributor Author

I tested the formula on both, AArch64 and x86_64, variants of macOS 12. It seems the patch application only failed on the latter, so I disabled this patch for non-AArch64 systems.

Also, I rebased the formula on the lastest main to fix a merge conflict in the README.

@ladislas
Copy link
Member

Thank you very much!

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.

2 participants