Skip to content

Updated includes of renamed platform header files #3868

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
Mar 3, 2017

Conversation

geky
Copy link
Contributor

@geky geky commented Mar 1, 2017

In hindsight, #3654 should have been the last pr to merge as a part of the most recent release. Later prs have brought in references to the previous header names.

Ran this script again:

sed -i 's/#include "\(.*\)\<\(critical\|sleep\|toolchain\|rtc_time\|semihost_api\|wait_api\|retarget\).h"/#include "\1mbed_\2.h"/' $(find -regex '.*\.\(h\|hpp\|c\|cpp\)')

Also renamed retarget.h -> mbed_retarget.h

cc @bridadan, @adbridge, @0xc0170

@bridadan
Copy link
Contributor

bridadan commented Mar 1, 2017

/morph test

@mbed-bot
Copy link

mbed-bot commented Mar 2, 2017

Result: FAILURE

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 1613

Test failed!

@bridadan
Copy link
Contributor

bridadan commented Mar 2, 2017

I believe for this run one ARCH_PRO and one HEXIWEAR had been knocked offline as the issue persisted in other CI runs. I have reset them so hopefully the same problem won't affect future runs.

/morph test

@mbed-bot
Copy link

mbed-bot commented Mar 2, 2017

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 1618

All builds and test passed!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants