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

Make all recipes keg-only. #56

Open
tom-radar opened this issue Aug 5, 2023 · 6 comments
Open

Make all recipes keg-only. #56

tom-radar opened this issue Aug 5, 2023 · 6 comments

Comments

@tom-radar
Copy link

The homebrew-core now includes arm-none-eabi-X recipes and this repositories gcc-arm@gcc10 is not keg-only and now conflicts.

See Homebrew/homebrew-core#138370

What do you think?

@ladislas
Copy link
Member

ladislas commented Aug 7, 2023

That's good news for user, though I'm surprised it's "that" simple to build gcc for arm-none-eabi. My understanding was that ARM process was very complicated.

I'm not totally sold on having all our formulas keg-only but I'll review a PR while I think about it.

@fauxpark @salkinium what do you think?

@fauxpark
Copy link
Contributor

fauxpark commented Aug 7, 2023

That formula seems almost completely useless, it does not include newlib or multilib, or pretty much anything as far as I can tell. I tried installing it and test compiling a QMK firmware, it can't even find stdint.h (although it is present under lib/gcc/arm-none-eabi/13.2.0/include/stdint.h?).

My understanding was that ARM process was very complicated.

The main reason this is so "complicated" to build is because in order to build a useful arm-none-eabi-gcc, you probably want newlib... which needs GCC... so the formula first compiles a rather minimal GCC (much like what was submitted to homebrew-core) to bootstrap everything.

Apart from that, from what I understand, building GCC is about as straightforward as it usually is.

@tom-radar
Copy link
Author

I guess this will require a bit more thinking. I have not tried the new homebrew-core recipes yet.

@fauxpark
Copy link
Contributor

fauxpark commented Aug 7, 2023

It looks like all of the formulae here are already keg-only, with the exception of arm-gcc-bin@10. This is what the arm-gcc-bin alias is pointing to, and presumably why it is not also keg-only, despite being a versioned formula?

In any case it still conflicts with everything else in this tap, regardless of the new formulae in homebrew-core.

@tom-radar
Copy link
Author

Right. arm-gcc-bin@10 is not key-only and conflicts with the new formula in homebrew-core.

In any case it still conflicts with everything else in this tap, regardless of the new formulae in homebrew-core.

Does it? You can run any of the key-only versions by pointing to the right gcc binary in the /opt/homebrew/opt directory structure. The arm-gcc-bin@10 was kept not keg-only intentionally (see #50 (comment)) as it was the desired default version by the maintainer of this repository.

@fauxpark
Copy link
Contributor

fauxpark commented Aug 7, 2023

I mean, that's why they are all keg-only except one.

Perhaps it is time to move these formulae to homebrew-core? They at least have access to M1 runners for building bottles :)

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

3 participants