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

cross-arm-none-eabi : Use exactly the same build recipe as the offici… #5907

Closed
wants to merge 1 commit into from

Conversation

teajay-fr
Copy link
Contributor

…al arm toolchain

With this patch, the void build of the arm embedded processor cross
matches the same version as the currently official gcc version 7.3.1.
The arm provided patches are applied and the build recipes match.

This fixes issue 4931 and 1824

…al arm toolchain

With this patch, the void build of the arm embedded processor cross
matches the same version as the currently official gcc version 7.3.1.
The arm provided patches are applied and the build recipes match.

This fixes issue 4931 and 1824
@jnbr
Copy link
Contributor

jnbr commented Apr 5, 2019

Vendoring a massive patch like this one is not really an option.
Is there a source for the patch?

@teajay-fr
Copy link
Contributor Author

Well I could rework the build to use the tar package provided by arm. That would avoid the patch. The outcome should be the same.
Should I reword this package in this direction ?

@jnbr
Copy link
Contributor

jnbr commented Apr 5, 2019

Yes, this would be better. Is there a reason why this patch isn't upstream?

@teajay-fr
Copy link
Contributor Author

There are some parts that come upstream but I don't know exactly why arm and the gcc team choose to go this way. As far as I understand arm provides a specific distribution of gcc which follows the gcc developments. I'm not into the details so I can't say much more. I'll look into eliminating the required patch and build the toolchain from the arm provided package.

@sirikid
Copy link
Contributor

sirikid commented May 28, 2019

@pullmoll @jnbr I suppose this PR can be closed now, since cross-arm-none-eabi has already been fixed in #11571

@pullmoll pullmoll closed this May 30, 2019
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

Successfully merging this pull request may close these issues.

None yet

4 participants