-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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 arm-gnu-toolchain #17528
base: master
Are you sure you want to change the base?
Add arm-gnu-toolchain #17528
Conversation
Khalil Estell seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit 40e5b66gnu-arm-embedded-toolchain/12.2.0
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good work. We use a similar package at my work. I wasn't sure if it would be worth waiting for a general GCC package built from source, because then we could use that to build such a package. I don't think that recipe is configurable enough for this yet, and who knows how long that might take. Having a package for the Arm GNU toolchain has been really helpful and having it on Conan Center should make things even easier for embedded development.
recipes/gnu-arm-embedded-toolchain/all/test_package/CMakeLists.txt
Outdated
Show resolved
Hide resolved
recipes/gnu-arm-embedded-toolchain/all/test_package/CMakeLists.txt
Outdated
Show resolved
Hide resolved
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit e1bc123arm-gnu-toolchain/12.2.1
|
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit c686510arm-gnu-toolchain/12.2.1
|
The error is
Hmmm not sure why this is happening. @memsharded @jwillikers, may I simply download the license to the recipe folder and copy it from source from the |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit aa783a8arm-gnu-toolchain/11.3.0@#55b37cf2ef2320f2666e0dd4f5d381f3
|
Hmmm, back up against the build issue, where CMake cannot figure out which compiler to use. 🤔 |
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit 5ccd502arm-gnu-toolchain/11.3.0@#e134474fe526c27b6da13dc4f300f6f1
|
For now, I'll table working on this PR until I get some help from the conan team for how to make this toolchain work with CCI. I'm probably doing something incorrectly, but it looks like the "correct" way of doing things does not work, and I'm just throwing stuff at the wall hoping it works. Conan 1.x's CMake invocation seems to miss the user toolchain file and simply selects the wrong compiler. |
One more thing to add. Technically the toolchains shouldn't work for ANY of the CCI's build targets because this is a cross compiler. So the toolchain in the test_package is just there to prove that the compiler is available in the virtual environment. Once Conan has support for bare metal arm cortex targets then we can update the test package to reject builds projects that do not target baremetal arm cortex M processors. |
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit c6bc92carm-gnu-toolchain/11.3.0@#e134474fe526c27b6da13dc4f300f6f1
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Poke, still working on this, but have been busy with life. |
Conan's adds host side flags to the conan_toolchain.cmake file which interferes with arm-none-eabi-g++. To fix this, we set CMAKE_CXX_FLAGS to an empty string, then provide target specific flags to the executable.
Conan v1 pipeline ❌Failure in build 20 (
Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability. Conan v2 pipeline ❌
The v2 pipeline failed. Please, review the errors and note this is required for pull requests to be merged. In case this recipe is still not ported to Conan 2.x, please, ping See details:Failure in build 20 (
Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability. |
Hooks produced the following warnings for commit e7ae67earm-gnu-toolchain/11.3.0@#f1dd6f95190583c1f14d70a2ade8212b
|
I need to relax my checks so more profiles can be accepted. But something weird about the one that did run is that the package isn't set as a tool/build package. It's set as a normal package. This could explain why the arm GCC compiler isn't getting selected in the test package and is, instead, using the default system compiler. This seems to be a bug. |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Not stale. |
Specify library name and version:
New addition to ConanCenter.
The author of the toolchain binaries is ARM inc. Original files can be found here:
https://developer.arm.com/downloads/-/arm-gnu-toolchain-downloads
Makes the bare metal ARM GCC toolchain accessible via "tool_requires" for bare metal projects.