-
Notifications
You must be signed in to change notification settings - Fork 11
CMake: Fix linking of greentea, unity and utest #119
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
Conversation
From Travis:
But mbed-os/requirements.txt does contain mbed-greentea. |
Travis is failing because mbed-os-tools use a newer prettytable with Python 3 than icetea supports. PR ARMmbed/mbed-os#14880 will fix. |
Manually started a nightly run that points to this PR. |
ARMmbed/mbed-os#14880 has been merged |
The test application uses the greentea client for synchronization with the host and unity for assertions, and unity in Mbed OS contains references to utest. We have recently split them from one mbed-greentea library into three, so the test application needs to link all of them.
Forced pushed to rerun Travis |
Travis still appears to be caching Python dependencies.
|
I'll clear the cache on Travis |
Cleared cache for this PR. Rerunning Travis... |
Boo, still using cached old prettytable for some reason. I'll clear all cache. |
Is caching |
Even though we removed icetea, requirements.txt still has an old prettytable listed (using == instead of >=): https://github.com/ARMmbed/mbed-os/blob/master/requirements.txt#L3 maybe this needs an update? |
Gah, that looks like the issue. Good spot. |
Raised ARMmbed/mbed-os#14940 |
Merged that mbed-os PR, cleared Travis cache for this PR, and re-kicked it off |
Thanks @Patater, it's finally resolved! 🎉 |
Requires: ARMmbed/mbed-os#14880
The test application uses the greentea client for synchronization with the host and unity for assertions, and unity in Mbed OS contains references to utest. We have recently split them from one mbed-greentea library into three, so the test application needs to link all of them.
Notes: