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

Enabled example export building for more examples #3331

Merged
merged 1 commit into from
Dec 9, 2016

Conversation

theotherjimmy
Copy link
Contributor

@theotherjimmy theotherjimmy commented Nov 28, 2016

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

  • CI!
  • Check that CI ran the additional tests that were turned on.

@theotherjimmy
Copy link
Contributor Author

/morph export-build

@c1728p9
Copy link
Contributor

c1728p9 commented Nov 28, 2016

Don't merge this patch

@geky
Copy link
Contributor

geky commented Nov 28, 2016

@c1728p9 Should we merge this patch?

@theotherjimmy
Copy link
Contributor Author

Please Don't. @geky happy to clarify for you.

@mbed-bot
Copy link

Result: FAILURE

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

/morph export-build

Output

mbed Build Number: 26

Exporter Build failed!

@theotherjimmy
Copy link
Contributor Author

All failures are that it tried to export and compile uvisior with not gcc

@theotherjimmy
Copy link
Contributor Author

complete success!

@theotherjimmy
Copy link
Contributor Author

/morph export-build

1 similar comment
@theotherjimmy
Copy link
Contributor Author

/morph export-build

@mbed-bot
Copy link

Result: FAILURE

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

/morph export-build

Output

mbed Build Number: 27

Exporter Build failed!

@mbed-bot
Copy link

Result: FAILURE

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

/morph export-build

Output

mbed Build Number: 28

Exporter Build failed!

@theotherjimmy
Copy link
Contributor Author

/morph export-build

@mbed-bot
Copy link

Result: FAILURE

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

/morph export-build

Output

mbed Build Number: 29

Exporter Build failed!

@theotherjimmy
Copy link
Contributor Author

/morph export-build

@mbed-bot
Copy link

Result: SUCCESS

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

/morph export-build

Output

mbed Build Number: 30

All exports and builds passed!

@theotherjimmy
Copy link
Contributor Author

hahaha!

@theotherjimmy theotherjimmy force-pushed the enable-exporters-examples branch from 55c5075 to 03dc17b Compare November 28, 2016 23:31
@theotherjimmy
Copy link
Contributor Author

/morph export-build

@theotherjimmy
Copy link
Contributor Author

/morph export-build

@mbed-bot
Copy link

Result: FAILURE

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

/morph export-build

Output

mbed Build Number: 31

Exporter Build failed!

@mbed-bot
Copy link

Result: FAILURE

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

/morph export-build

Output

mbed Build Number: 32

Exporter Build failed!

@0xc0170
Copy link
Contributor

0xc0170 commented Nov 29, 2016

@theotherjimmy Is it that unstable? the latest build is quite red.

@0xc0170
Copy link
Contributor

0xc0170 commented Nov 29, 2016

Rerunning as it might be the sam eproblem as other PR (jenkins failure).

/morph export-build

@mbed-bot
Copy link

Result: FAILURE

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

/morph export-build

Output

mbed Build Number: 33

Exporter Build failed!

@mbed-bot
Copy link

mbed-bot commented Dec 1, 2016

Result: FAILURE

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

/morph export-build

Output

mbed Build Number: 41

Exporter Build failed!

@theotherjimmy
Copy link
Contributor Author

/morph export-build

@mbed-bot
Copy link

mbed-bot commented Dec 1, 2016

Result: FAILURE

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

/morph export-build

Output

mbed Build Number: 42

Exporter Build failed!

@theotherjimmy
Copy link
Contributor Author

/morph export-build

@mbed-bot
Copy link

mbed-bot commented Dec 2, 2016

Result: SUCCESS

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

/morph export-build

Output

mbed Build Number: 44

All exports and builds passed!

@theotherjimmy
Copy link
Contributor Author

should resolve #2679

"features" : ["IPV6"],
"targets" : [],
"features" : [],
"targets" : ["DISCO_F469NI", "DISCO_F746NG", "K64F", "K66F",
Copy link
Contributor

Choose a reason for hiding this comment

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

misaligned targets?

Copy link
Contributor Author

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

There you go.

@0xc0170 0xc0170 removed the needs: CI label Dec 5, 2016
@0xc0170
Copy link
Contributor

0xc0170 commented Dec 5, 2016

LGTM (just that one nit)

@theotherjimmy theotherjimmy force-pushed the enable-exporters-examples branch from 095302b to 5698085 Compare December 5, 2016 19:44
@0xc0170
Copy link
Contributor

0xc0170 commented Dec 7, 2016

/morph export-build

@mbed-bot
Copy link

mbed-bot commented Dec 7, 2016

Result: SUCCESS

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

/morph export-build

Output

mbed Build Number: 45

All exports and builds passed!

@bridadan
Copy link
Contributor

bridadan commented Dec 8, 2016

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

@mbed-bot
Copy link

mbed-bot commented Dec 8, 2016

Result: SUCCESS

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

/morph test

Output

mbed Build Number: 1230

All builds and test passed!

@0xc0170 0xc0170 merged commit 9da9166 into ARMmbed:master Dec 9, 2016
@bridadan
Copy link
Contributor

bridadan commented Dec 9, 2016

Way to wrestle with the CI and get this in, should make the release process a lot better 👍

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.

6 participants