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

[Exporters] Append targets to scan list for exporting tests #3188

Merged
merged 1 commit into from
Nov 10, 2016

Conversation

theotherjimmy
Copy link
Contributor

@theotherjimmy theotherjimmy commented Nov 2, 2016

Description

The tests were assuming that the hal directory would contain both target code and common code. After the restructure, the target code lives in targets, so we should scan that too.

Status

READY

Reviews

Steps to test or reproduce

#3117

@sg-
Copy link
Contributor

sg- commented Nov 3, 2016

/morph export-build

@mbed-bot
Copy link

mbed-bot commented Nov 4, 2016

Result: SUCCESS

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

/morph export-build

Copy link

@kgoveas kgoveas left a comment

Choose a reason for hiding this comment

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

Works for gcc_arm. Reports that eclipse_gcc_arm is under development.

@theotherjimmy
Copy link
Contributor Author

thanks for testing @kgoveas! The eclipse_gcc_arm 'under development' issue was a problem with master at the moment I created this patch, I fixed it and it recently got merged to master. More recently than I created this PR.

@sg-
Copy link
Contributor

sg- commented Nov 7, 2016

@marhil01 Can you restart CI
@mazimkhan Comment about stalled CI status?

@mazimkhan
Copy link

"Cam-CI uvisor Build & Test" job has passed. Job failed to update status due to GitHub rate limit issue:

Setting pull request status Cam-CI uvisor Build & Test to SUCCESS with message: Success
[Pipeline] echo
Failed to update commit status. err: java.io.IOException: API rate limit reached
[Pipeline] echo
Retrying in 3 sec.
[Pipeline] sleep
[Pipeline] setGitHubPullRequestStatus
Setting pull request status Cam-CI uvisor Build & Test to SUCCESS with message: Success
[Pipeline] echo
Failed to update commit status. err: java.io.IOException: API rate limit reached
[Pipeline] echo
Retrying in 3 sec.
[Pipeline] sleep
[Pipeline] setGitHubPullRequestStatus
Setting pull request status Cam-CI uvisor Build & Test to SUCCESS with message: Success
[Pipeline] echo
Failed to update commit status. err: java.io.IOException: API rate limit reached
[Pipeline] echo
Retrying in 3 sec.
[Pipeline] sleep
[Pipeline] setGitHubPullRequestStatus
Setting pull request status Cam-CI uvisor Build & Test to SUCCESS with message: Success
[Pipeline] echo
Failed to update commit status. err: java.io.IOException: API rate limit reached
[Pipeline] echo
Retrying in 3 sec.
[Pipeline] sleep
[Pipeline] setGitHubPullRequestStatus
Setting pull request status Cam-CI uvisor Build & Test to SUCCESS with message: Success
[Pipeline] echo
Failed to update commit status. err: java.io.IOException: API rate limit reached

@bridadan
Copy link
Contributor

bridadan commented Nov 8, 2016

@sg- I was able to restart the Oulu CI, looks like it passed.

@adbridge adbridge merged commit e0c7d3b into ARMmbed:master Nov 10, 2016
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.

8 participants