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

Release candidate for mbed-os-5.2.2 #3227

Merged
merged 107 commits into from
Nov 8, 2016
Merged

Release candidate for mbed-os-5.2.2 #3227

merged 107 commits into from
Nov 8, 2016

Conversation

adbridge
Copy link
Contributor

@adbridge adbridge commented Nov 8, 2016

No description provided.

bridadan and others added 30 commits November 8, 2016 10:42
This uses similar code that is used withing the toolchains to parallelize
the linking process of all the tests accross all the available CPUs. It
also respects the `-j` parameter if you wish to limit the number of cores
used.
The 'silent' option has always been present in the toolchains API, however
it did not actually stop anything from being printed. Instead, it just
changed what was added to the build log. This make the 'silent' stop all
prints, but ensures that the output for the toolchain is still preserved
and accessible via the 'get_output' function.
This makes use of the reports generated by the building of tests to
prevent output from interleaving when the build is parallelized. This
required some changes to memap to return a generated string from
the 'generate_output' function. I also had an option to stop the prints
from memap to prevent text from interleaving
That let us to ignore build failures for examples on not supported
platforms.
…t and the call to main. Remove direct call to __libc_init_array and main not needed as _start is beeing called.
… init and the call to main. Remove direct call to __libc_init_array and main not needed as _start is beeing called.
…gcc _start , perform zero initialized. => HAL_Init must be done again also in GCC toolchain
This change prevents the standard library from allocating a large buffer
on the heap. On GCC_ARM, this is a saving of 1K. On ARM, this is a saving
of 64 bytes.
@adbridge
Copy link
Contributor Author

adbridge commented Nov 8, 2016

/morph test

@adbridge
Copy link
Contributor Author

adbridge commented Nov 8, 2016

mbed 2 results are as follows:

All targets building successfully.
Testing wise we have the following number of failures:
LPC1768 13/116
KL25Z 9/81
K22F 1/78
NUCLEO_L053R8 1/72
MBED_37 4/27
MBED_38 4/27

This compares with the 5.2.1 release which had the following failures:
LPC1768 18/116
KL25Z 9/81
NUCLEO_L053R8 9/72
MBED_37 8/30
MBED_38 8/30

Overall the test results look a little better than for the previous release, so LGTM.

@mbed-bot
Copy link

mbed-bot commented Nov 8, 2016

Result: SUCCESS

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

/morph test

Output

mbed Build Number: 1025

All builds and test passed!

@adbridge
Copy link
Contributor Author

adbridge commented Nov 8, 2016

mbed-os 5 has also successfully built and ran.
Note the wifi example was run successfully, independently, locally against this PR as it is still disabled in the CI pending the changes that this release introduces.

LGTM for merging

@0xc0170
Copy link
Contributor

0xc0170 commented Nov 8, 2016

+1

@adbridge
Copy link
Contributor Author

adbridge commented Nov 8, 2016

@sg- , @0xc0170 please confirm you're happy to merge

@0xc0170 0xc0170 merged commit a1c0840 into mbed-os-5.2 Nov 8, 2016
@sg-
Copy link
Contributor

sg- commented Nov 8, 2016

LGTM 👍

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

Successfully merging this pull request may close these issues.