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

simavr1.6 Makefile bug checking for avr-gcc installation #319

Open
Jim-Carroll opened this issue Nov 11, 2023 · 4 comments
Open

simavr1.6 Makefile bug checking for avr-gcc installation #319

Jim-Carroll opened this issue Nov 11, 2023 · 4 comments

Comments

@Jim-Carroll
Copy link

The current version of the simavr1.6 Makefile.common checks if avr-gcc is installed by looking for a folder called avr-gcc in the homebrew Cellar. This did not work on my system (macOS Ventura 13.5). Instead, I have two folders in the Cellar called avr-gcc@9 and avr-gcc@12.

To workaround this issue, I manually created a folder called avr-gcc in the Cellar. This allowed the installation to proceed and complete successfully.

# workaround
mkdir $(brew --prefix)/Cellar/avr-gcc
# now this works
brew install simavr

Looks like the new version of the Makefile fixes this by using a glob pattern, so I guess this cask is just outdated. However, the recommendation to brew install avr-libc is incorrect. No such package exists. I believe the error message should say brew install avr-gcc instead.

https://github.com/buserror/simavr/blob/master/Makefile.common#L69C4-L69C4

@gatk555
Copy link

gatk555 commented Dec 28, 2023

Should this not be opened at buserror/simavr?

@Jim-Carroll
Copy link
Author

No. It has already been fixed upstream. This repo is just using an older version.

buserror/simavr@9f1014a

@gatk555
Copy link

gatk555 commented Dec 29, 2023

My mistake! But there is nothing on the surface to indicate that a simavr build is here.

Thanks, G.

@Jim-Carroll
Copy link
Author

This formula uses simavr v1.6: https://github.com/osx-cross/homebrew-avr/blob/main/Formula/simavr.rb

That version of simavr has the aforementioned bug.

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

No branches or pull requests

2 participants