Skip to content

[tools-build.py]Enforce supported toolchains #2172

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

Merged
merged 1 commit into from
Jul 15, 2016

Conversation

theotherjimmy
Copy link
Contributor

by skipping unsupported combinations

before:

$ python tools/build.py -t IAR -m NRF51_DK --source . --build build
[SNIP]
Compile: us_ticker.c
  #error Compiler not supported.
   ^
"Z:\home\jimbri01\src\mbedmicro\mbed\hal\targets\hal\TARGET_NORDIC\TARGET_MCU_NRF51822\us_ticker.c",382  Fatal error[Pe035]: #error directive: Compiler not supported.
Fatal error detected, aborting.

  #error Compiler not supported.
   ^
"Z:\home\jimbri01\src\mbedmicro\mbed\hal\targets\hal\TARGET_NORDIC\TARGET_MCU_NRF51822\us_ticker.c",382  Fatal error[Pe035]: #error directive: Compiler not supported.
Fatal error detected, aborting.


Completed in: (8.96)s

Build failures:
  * IAR::NRF51_DK

after:

$ python tools/build.py -t IAR -m NRF51_DK --source . --build build
IAR::NRF51_DK skipped: toolchain not supported

Completed in: (0.00)s

Build skipped:
  * IAR::NRF51_DK

by skipping unsupported combinations
@theotherjimmy
Copy link
Contributor Author

cc @bridadan @screamerbg

@bridadan
Copy link
Contributor

@theotherjimmy Should we merge this guy first so we can test with it before merging this PR?

@theotherjimmy
Copy link
Contributor Author

Yes, that is a good idea.

@screamerbg
Copy link
Contributor

LGTM

@0xc0170 0xc0170 merged commit 3d20b10 into ARMmbed:master Jul 15, 2016
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.

4 participants