Skip to content

Enforcing 'standard' build for version 5 release #2300

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

Merged
merged 1 commit into from
Jul 29, 2016

Conversation

bridadan
Copy link
Contributor

@bridadan bridadan commented Jul 28, 2016

This PR enforces that the default_build in a target's definition in hal/targets.json is set to standard to be included in the version 5 release.

This is being enforced because small builds do not support multi-threading. This makes the capabilities more consistent across all boards in the version 5 release.

NOTE: This removes the MOTE_L152RC and the LPC11U68
from the version 5 release list because the 'default_build' for these
targets is currently set to 'small'.

@bridadan bridadan force-pushed the enforce-standard-boards-v5 branch from 96bd3c3 to 4a7da3f Compare July 28, 2016 19:11
@bridadan
Copy link
Contributor Author

bridadan commented Jul 28, 2016

@pan- FYI I had to remove the NRF51_DK and NRF52_DK from the v5 release list to get this PR in (since at this time they are still small builds). So this may conflict with your PR #2280. Depending on which gets merged first, we'll just have to make sure these targets don't get left out :)

Never mind, these boards have since been switched to a standard build.

@bridadan
Copy link
Contributor Author

/morph test

@mbed-bot
Copy link

Result: SUCCESS

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

/morph test

Output

mbed Build Number: 557

All builds and test passed!

This PR enforces that the default_build in a target's definition in
hal/targets.json is set to standard to be included in the version 5
release.

This is being enforced because small builds do not support
multi-threading. This makes the capabilities more consistent across all
boards in the version 5 release.

NOTE: This removes the MOTE_L152RC and the LPC11U68
from the version 5 release list because the 'default_build' for these
targets is currently set to 'small'.
@bridadan bridadan force-pushed the enforce-standard-boards-v5 branch from 4a7da3f to 7a07c38 Compare July 28, 2016 23:48
@bridadan
Copy link
Contributor Author

Ok, removed the change to the two Nordic boards.

/morph test

@mbed-bot
Copy link

Result: FAILURE

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

/morph test

Output

mbed Build Number: 558

Test failed!

@0xc0170
Copy link
Contributor

0xc0170 commented Jul 29, 2016

as I see it, just two LPC targets were removed. Any objections?

@sg- sg- merged commit ea78ed1 into ARMmbed:master Jul 29, 2016
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.

4 participants