Skip to content

Includre rtos.h in mbed.h if present #2227

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 22, 2016
Merged

Includre rtos.h in mbed.h if present #2227

merged 1 commit into from
Jul 22, 2016

Conversation

bogdanm
Copy link
Contributor

@bogdanm bogdanm commented Jul 22, 2016

So that apps don't need to include "rtos.h" explicitly anymore.

So that apps don't need to include "rtos.h" explicitly anymore.
@bogdanm
Copy link
Contributor Author

bogdanm commented Jul 22, 2016

@mbed-bot: TEST

HOST_OSES=windows,linux
BUILD_TOOLCHAINS=GCC_ARM,ARM,IAR
TARGETS=K64F

@bogdanm
Copy link
Contributor Author

bogdanm commented Jul 22, 2016

@0xc0170 @sg-

@mbed-bot
Copy link

[Build ${MBED_BUILD_ID}]
FAILURE: Something went wrong when building and testing.

@bogdanm
Copy link
Contributor Author

bogdanm commented Jul 22, 2016

@mbed-bot: TEST

HOST_OSES=windows,linux
BUILD_TOOLCHAINS=GCC_ARM,ARM,IAR
TARGETS=K64F

@mbed-bot
Copy link

[Build 674]
FAILURE: Something went wrong when building and testing.

@bogdanm
Copy link
Contributor Author

bogdanm commented Jul 22, 2016

Not sure why mbed-bot is failing, but the failures don't seem to be related to this change.

@0xc0170
Copy link
Contributor

0xc0170 commented Jul 22, 2016

+1

We shall not have problems with rtos/rtos.h , as it never gets flatten (opposite to what happens with mbed headers (api/hal))? Just checking to be sure

@bogdanm
Copy link
Contributor Author

bogdanm commented Jul 22, 2016

I can't predict the future :) but I think we're safe for now.

@sg-
Copy link
Contributor

sg- commented Jul 22, 2016

I'll look at testing

@bridadan
Copy link
Contributor

bridadan commented Jul 22, 2016

@bogdanm Sorry about the failure with the bot (even though tests are green), it was caused due to an issue with mbed-ls. Should be resolved now.

@sg-
Copy link
Contributor

sg- commented Jul 22, 2016

@mbed-bot: TEST

HOST_OSES=windows
BUILD_TOOLCHAINS=GCC_ARM,ARM,IAR
TARGETS=K64F,NUCLEO_F411RE,NRF51_DK

@mbed-bot
Copy link

[Build 676]
SUCCESS: Building succeeded and tests were run! Be sure to check the test results

@sg- sg- merged commit 83f24fb into master Jul 22, 2016
@bogdanm bogdanm deleted the include_rtos_by_default branch October 18, 2016 13:53
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.

5 participants