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

Upgrade our testing systems to use GNU Arm Embedded Toolchain 10.3-2021.07 as the new supported version of gcc #14968

Closed
Tracked by #14967
Patater opened this issue Jul 29, 2021 · 11 comments

Comments

@Patater
Copy link
Contributor

Patater commented Jul 29, 2021

No description provided.

@Patater Patater changed the title Upgrade our testing systems to use GNU Arm Embedded Toolchain 10-2021-q2-update as the new supported version of gcc Upgrade our testing systems to use GNU Arm Embedded Toolchain 10.3-2021.07 as the new supported version of gcc Aug 6, 2021
@Patater
Copy link
Contributor Author

Patater commented Aug 19, 2021

I've raised #15007 to move our GitHub actions CI to the new version of GCC.

@saheerb has been working on updating the Jenkins CI.

@LDong-Arm
Copy link
Contributor

It's slightly annoying that Mbed CLI 1 prints

[Warning] @,: Compiler version mismatch: Have 10.3.1; expected version >= 9.0.0 and < 10.0.0

The version range is hardcoded in tools/toolchains/gcc.py which is frozen. As it's just a warning, leaving it as it is probably fine...

@jeromecoutant
Copy link
Collaborator

As it's just a warning, leaving it as it is probably fine...

I would prefer to remove warnings
Thx

@rwalton-arm
Copy link
Contributor

rwalton-arm commented Sep 6, 2021

It's slightly annoying that Mbed CLI 1 prints

[Warning] @,: Compiler version mismatch: Have 10.3.1; expected version >= 9.0.0 and < 10.0.0

The version range is hardcoded in tools/toolchains/gcc.py which is frozen. As it's just a warning, leaving it as it is probably fine...

I don't think fixing that warning would affect the behaviour of the online build system. We should be OK to fix it without creating a new "tools release" for the online compiler. We could probably ignore the frozen tools check in this instance? @Patater

@LDong-Arm
Copy link
Contributor

LDong-Arm commented Sep 6, 2021

Ultimately we encourage users to move away from Mbed CLI 1. But if someone disregards the deprecation and does what the warning says (i.e. downgrades GCC to 9.x), they may encounter issues such as NU_M2354 unable to run due to known issues of the old GCC with Armv8-m.

@Patater
Copy link
Contributor Author

Patater commented Sep 6, 2021

We should be OK to fix it without creating a new "tools release" for the online compiler.

Agreed. The warning is annoying and not helpful for users, and ignored by the online compiler which doesn't use gcc anyway.

@0xc0170
Copy link
Contributor

0xc0170 commented Sep 6, 2021

I'll send the PR now to remove the warning for ver 10 as only two PRs left to be merged (docs and docker image update).

@0xc0170
Copy link
Contributor

0xc0170 commented Sep 6, 2021

Already removed in the docker image update 44e5795

@Patater
Copy link
Contributor Author

Patater commented Sep 7, 2021

Saheer has updated Jenkins CI.

We've also merged #15007 which adds support in GitHub actions via our Docker images.

@Patater Patater closed this as completed Sep 7, 2021
@jeromecoutant
Copy link
Collaborator

Already removed in the docker image update 44e5795

Patch is strange... only GCC_RANGE has to be updated ?

@LDong-Arm
Copy link
Contributor

LDong-Arm commented Sep 7, 2021

Patch is strange... only GCC_RANGE has to be updated ?

ARMCC_RANGE is 6.10 to 7.0 so we won't need to worry about it in a long while.

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

No branches or pull requests

7 participants