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

Add possibility to configure toolchain location #153

Closed
1 of 2 tasks
boraozgen opened this issue Dec 9, 2020 · 6 comments
Closed
1 of 2 tasks

Add possibility to configure toolchain location #153

boraozgen opened this issue Dec 9, 2020 · 6 comments

Comments

@boraozgen
Copy link

Description

This was possible in CLI 1.

Issue request type

  • Enhancement
  • Bug
@rwalton-arm
Copy link
Contributor

Hi,

Thanks for trying out the tools and for your feedback. This isn't something we're planning on adding to the Python tools as the toolchain location is managed by CMake. If you want to configure the location you may be able to set environment variables as described in this stackoverflow post. A possible alternative would be to edit the toolchain file located in mbed-os/tools/cmake/toolchains/.

@boraozgen
Copy link
Author

Thanks for clarifying this. Maybe this should be added to the docs to avoid confusion.

Still, I noticed at least one line where the toolchain command is hardcoded:

https://github.com/ARMmbed/mbed-os/blob/36df9d10436487625810a43f922b7fb975e5eec7/CMakeLists.txt#L135

Trying to compile with CLion (where the toolchain location is customizable as you mentioned) still fails at this command.

@rwalton-arm
Copy link
Contributor

Thanks for clarifying this. Maybe this should be added to the docs to avoid confusion.

Still, I noticed at least one line where the toolchain command is hardcoded:

https://github.com/ARMmbed/mbed-os/blob/36df9d10436487625810a43f922b7fb975e5eec7/CMakeLists.txt#L135

Trying to compile with CLion (where the toolchain location is customizable as you mentioned) still fails at this command.

That hardcoded toolchain is a bug in mbed-os' CMake. We will look into removing that.

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 15, 2021

I realized we do not have arm-none-eabi-cpp set anywhere, this is a bug and should be fixed. @boraozgen would you be able to send pull request fixing this?

@boraozgen
Copy link
Author

I unfortunately don't have enough knowledge about the tooling to fix this.

@Patater
Copy link
Contributor

Patater commented Feb 11, 2021

Fixed in Mbed OS in ARMmbed/mbed-os#14199

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

4 participants