-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Enabled example export building for more examples #3331
Enabled example export building for more examples #3331
Conversation
/morph export-build |
Don't merge this patch |
@c1728p9 Should we merge this patch? |
Please Don't. @geky happy to clarify for you. |
Result: FAILUREYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 26 Exporter Build failed! |
All failures are that it tried to export and compile uvisior with not gcc |
complete success! |
/morph export-build |
1 similar comment
/morph export-build |
Result: FAILUREYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 27 Exporter Build failed! |
Result: FAILUREYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 28 Exporter Build failed! |
/morph export-build |
Result: FAILUREYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 29 Exporter Build failed! |
/morph export-build |
Result: SUCCESSYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 30 All exports and builds passed! |
hahaha! |
55c5075
to
03dc17b
Compare
/morph export-build |
03dc17b
to
89a0544
Compare
/morph export-build |
Result: FAILUREYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 31 Exporter Build failed! |
Result: FAILUREYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 32 Exporter Build failed! |
@theotherjimmy Is it that unstable? the latest build is quite red. |
Rerunning as it might be the sam eproblem as other PR (jenkins failure). /morph export-build |
Result: FAILUREYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 33 Exporter Build failed! |
Result: FAILUREYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 41 Exporter Build failed! |
/morph export-build |
Result: FAILUREYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 42 Exporter Build failed! |
/morph export-build |
Result: SUCCESSYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 44 All exports and builds passed! |
should resolve #2679 |
"features" : ["IPV6"], | ||
"targets" : [], | ||
"features" : [], | ||
"targets" : ["DISCO_F469NI", "DISCO_F746NG", "K64F", "K66F", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
misaligned targets?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My editor seems to think that the entire file is misaligned. I'll fix this instance shortly
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There you go.
LGTM (just that one nit) |
095302b
to
5698085
Compare
/morph export-build |
Result: SUCCESSYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 45 All exports and builds passed! |
Nice work @theotherjimmy. We should run the other mbed os tests on this too since it affects the compiles done through mbed build system as well. /morph test |
Result: SUCCESSYour command has finished executing! Here's what you wrote!
OutputAll builds and test passed! |
Way to wrestle with the CI and get this in, should make the release process a lot better 👍 |
Description
Turns on export building for many examples that we should be building, and filters exporters with the toolchain specified in the example JSON file.
TODO