Skip to content
This repository was archived by the owner on Apr 24, 2019. It is now read-only.

Update to new mbed OS baseline and add dependencies. #31

Merged
merged 3 commits into from
Jun 16, 2016
Merged

Update to new mbed OS baseline and add dependencies. #31

merged 3 commits into from
Jun 16, 2016

Conversation

SeppoTakalo
Copy link
Contributor

No description provided.

@SeppoTakalo
Copy link
Contributor Author

TEST THIS PLEASE

@SeppoTakalo
Copy link
Contributor Author

How can I restart the tests?

@mbed-ci
Copy link
Collaborator

mbed-ci commented Jun 16, 2016

Verification build successful.

@yogpan01
Copy link
Contributor

rebuild

@mbed-ci
Copy link
Collaborator

mbed-ci commented Jun 16, 2016

Verification build successful.

@yogpan01
Copy link
Contributor

@SeppoTakalo Did you build this locally because it is still giving error in CI or do we need to update mbed-cli in CI ?
Building project 6a365025 (K64F, GCC_ARM) [ERROR] Attempt to override undefined parameter 'mbed-client.reconnection-count' in 'application[*]' [mbed ERROR] "python" returned error code 1.

@mbed-ci
Copy link
Collaborator

mbed-ci commented Jun 16, 2016

Verification build successful.

@yogpan01
Copy link
Contributor

Testing this PR locally, it builds but example doesnt work anymore. It seems to crash. Not sure if the mbed-os.lib version is good one.

@@ -1 +1 @@
https://github.com/ARMmbed/mbed-os/#f6592621b87a97fdf3d42a15e447ccc58dd0c1e4
https://github.com/ARMmbed/mbed-os/#c0c5a70b1907d701f69f37b8812b5fe2ce048ff0
Copy link

Choose a reason for hiding this comment

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

perhaps this shouldn't be needed anymore, or is it ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

What do you meant needed?

In order to build this application against latest changes, we need the master branch from mbed-os.

There has been breaking changes on the mbed-os master, so we need to go past that commit to get the build working.
Crashing is the another issue we need to start investigatin after the build is working.

Copy link
Contributor

Choose a reason for hiding this comment

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

We cannot push this commit in unless it builds and runs properly. At the moment, this builds but doesnt run so no-go for this git tag !

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Agreed.
We are in kind of dealock situation. To get this building, we need to address the Socket.h issue. I'll talk to guys.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Requires: ARMmbed/mbed-os#356

Updating this once that is merged.

@SeppoTakalo
Copy link
Contributor Author

If you see this error

Building project 6a365025 (K64F, GCC_ARM) [ERROR] Attempt to override undefined parameter 'mbed-client.reconnection-count' in 'application[*]' [mbed ERROR] "python" returned error code 1. 

it sounds like it was not trying to build against latest master.
Feature -mechanism disabled nanostack and client, unless they were enabled on mbed_app.json
But enabling the feature requires latest changes from mbed-os and mbedmicro/mbed.

@SeppoTakalo
Copy link
Contributor Author

Updated now. Includes ARMmbed/mbed-os#356 fixes.

@mbed-ci
Copy link
Collaborator

mbed-ci commented Jun 16, 2016

Verification build successful.

@yogpan01
Copy link
Contributor

@SeppoTakalo Not sure if the CI job for this repo is still broken (it might need some fixes to be pushed to mbed-os repo , if not there already ). Have you tested this with Ethernet and WiFi as well ?

@SeppoTakalo
Copy link
Contributor Author

I can verify only the build with those.

@SeppoTakalo
Copy link
Contributor Author

cfg builds
MESH_LOWPAN_ND ok
MESH_THREAD  ok
ETHERNET  ok
WIFI ok

@SeppoTakalo
Copy link
Contributor Author

All the required fixes are in the mbed-os master now.

@SeppoTakalo
Copy link
Contributor Author

What is happening with the test job? I see no updates, nor links.

@SeppoTakalo
Copy link
Contributor Author

rebuild

@mbed-ci
Copy link
Collaborator

mbed-ci commented Jun 16, 2016

Verification build successful.

@SeppoTakalo
Copy link
Contributor Author

@yogpan01 Can you merge this?

@yogpan01
Copy link
Contributor

@SeppoTakalo Some issues with Jenkins job..seems to be stuck in waiting for status to be reported.
Anyway I will go ahead and merge this now based on your results .Thanks for fixing this.

@yogpan01 yogpan01 merged commit 04f8828 into ARMmbed:master Jun 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants