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

Toolchain file update #233

Merged

Conversation

robamu
Copy link
Contributor

@robamu robamu commented Jul 15, 2021

  1. Use STM32_TOOLCHAIN_PATH from environment if specified as environmental variable
  2. Find tools like objcopy and size in default paths as well, which allows
    simply adding the binary folder of the toolchain to the path to find them.
    This also allows more consistent behaviour, because this already works
    for -gcc/g++

closes #216

1. Use STM32_TOOLCHAIN_PATH if specified as environmental variable
2. Find tools like objcopy and size in default paths as well, which allows
   simply adding the binary folder of the toolchain to the path to find them.
   This also allows more consistent behaviour, because this already works
   for <cross-compile>-gcc/g++
@atsju
Copy link
Collaborator

atsju commented Jul 15, 2021

For information you might have a look at #218 + #211

@robamu
Copy link
Contributor Author

robamu commented Jul 15, 2021

Ok. Maybe we can merge it first? It looks finished, but I have not read everything yet

@atsju atsju requested review from atsju and Hish15 July 15, 2021 11:45
@atsju
Copy link
Collaborator

atsju commented Jul 15, 2021

I'm at work now. But looks fine, I will test ASAP and it can be merged regardless of other PR. As I said it was only for information.

Copy link
Collaborator

@Hish15 Hish15 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here setting STM32_TOOLCHAIN_PATH in a env var, will make setting the value directly impossible. I think we want to check if the var is not set (using CLI or in the cmakelist) before using ENV var.

cmake/stm32/common.cmake Outdated Show resolved Hide resolved
@atsju atsju requested review from atsju and Hish15 July 15, 2021 15:19
@Hish15 Hish15 merged commit cebbda6 into ObKo:master Jul 18, 2021
@robamu robamu deleted the mueller/find-tool-programs-in-default-paths branch August 8, 2021 13:49
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.

Seach for optional modules in environment as well
3 participants