-
Notifications
You must be signed in to change notification settings - Fork 3k
u-blox rearrange ODIN target #4510
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
The first jenkins CI failed with
Please have a look ( I retriggered the build to confirm this). |
Most likely the binaries for the ODIN driver are the old ones. |
@andreaslarssonublox TARGET_UBLOX_EVK_ODIN_W2\sdk\TOOLCHAIN_GCC_ARM\libublox-odin-w2-driver.a contained that version string, but the other two binaries did not. |
8526666
to
6a270fb
Compare
/morph test |
Result: SUCCESSYour command has finished executing! Here's what you wrote!
OutputAll builds and test passed! |
@kegilbert @andreaslarssonublox I just checked all three binaries, and they have the string "odin_driver_os_5_v2.0_rc1 (as part of a file string E:\build_odin_driver_os_5_v2.0_rc1\ublox-odin-w2-drivers-confidential\source\cb_wlan\common\cb_version.c). That should be fine. |
Changed cherry-picking of #4089 from original PR (#4467).
No one assigned yet as reviewer, wanted to get a CI run through to see if this helped anything.
Rebase against master for: #4089
Smaller chunk of the Workshop branch additions: #4412
Feel free to add in however else should be a reviewer @c1728p9