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

Fix unresolved include of mbed_config.h in Eclipse #4616

Merged
merged 1 commit into from Jun 22, 2017

Conversation

Projects
None yet
3 participants
@amq
Contributor

amq commented Jun 22, 2017

Notes:

  • I have accepted the Contributor Agreement on Tue 25 Oct 2016.

Description

There is a minor issue with the Eclipse exporter which causes a warning of an unresolved include of mbed_config.h. This PR solves it by adding the project directory path to the include. Actually, all includes have the project directory path, just not this one.

Status

READY

Migrations

NO

@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Jun 22, 2017

/morph export-build

@mbed-bot

This comment has been minimized.

mbed-bot commented Jun 22, 2017

Result: SUCCESS

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

/morph export-build

Output

mbed Build Number: 73

All exports and builds passed!

@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Jun 22, 2017

Wooo, That was fast! @amq any reservations about merging this? it looks good to me!

@amq

This comment has been minimized.

Contributor

amq commented Jun 22, 2017

@theotherjimmy no reservations :)

@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Jun 22, 2017

Awesome. Merging!

@theotherjimmy theotherjimmy merged commit 75935b4 into ARMmbed:master Jun 22, 2017

4 checks passed

Cam-CI uvisor Build & Test Success
Details
ci/morph-export-build Job has completed
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Jun 22, 2017

Thanks for the fix @amq !

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