-
Notifications
You must be signed in to change notification settings - Fork 56
Update to new mbed OS baseline and add dependencies. #31
Conversation
TEST THIS PLEASE |
How can I restart the tests? |
Verification build successful. |
rebuild |
Verification build successful. |
@SeppoTakalo Did you build this locally because it is still giving error in CI or do we need to update mbed-cli in CI ? |
Verification build successful. |
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 |
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.
perhaps this shouldn't be needed anymore, or is it ?
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.
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.
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.
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 !
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.
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.
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.
Requires: ARMmbed/mbed-os#356
Updating this once that is merged.
If you see this error
it sounds like it was not trying to build against latest master. |
Updated now. Includes ARMmbed/mbed-os#356 fixes. |
Verification build successful. |
@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 ? |
I can verify only the build with those. |
|
All the required fixes are in the mbed-os master now. |
What is happening with the test job? I see no updates, nor links. |
rebuild |
Verification build successful. |
@yogpan01 Can you merge this? |
@SeppoTakalo Some issues with Jenkins job..seems to be stuck in waiting for status to be reported. |
No description provided.