-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Add support for supplying OTA download file path #15660
Add support for supplying OTA download file path #15660
Conversation
PR #15660: Size comparison from 18ab5f4 to 19e7d60 Increases above 0.2%:
Increases (2 builds for linux, nrfconnect)
Full report (30 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
ota_test.sh changes look good to me! |
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.
Please make sure a followup is filed to fix the consumers assuming a CharSpan is null-terminated....
22dee37
to
441d7a3
Compare
|
PR #15660: Size comparison from 169b48a to 441d7a3 Increases above 0.2%:
Increases (1 build for nrfconnect)
Full report (21 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Updated ota_test.sh script to compare the ota binaries to declare success
441d7a3
to
06c7554
Compare
Squashed and rebased on master. |
PR #15660: Size comparison from 169b48a to 06c7554 Increases above 0.2%:
Increases (1 build for nrfconnect)
Full report (21 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Problem
Add support to the Linux OTA-requestor for supplying a fully-qualified destination path for the OTA file to be downloaded to
Change overview
Added a simple command line option for the same
Testing
Tested by running an OTA on a Linux machine and verifying that the file path was chosen successfully.