Skip to content
This repository has been archived by the owner on Dec 8, 2022. It is now read-only.

Let vendor choose which OTA backend to enable #1617

Merged
merged 2 commits into from
Dec 14, 2019

Conversation

tgsong
Copy link

@tgsong tgsong commented Dec 13, 2019

Description

We cannot automatically figure out which backend to enable in OTA. This PR refactors the OTA CMake file to let vendor choose which one to enable.

Checklist:

  • I have tested my changes. No regression in existing tests.
  • My code is Linted.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@tgsong tgsong requested review from qiutongs and pvyawaha December 13, 2019 20:49
qiutongs
qiutongs previously approved these changes Dec 13, 2019
@tgsong tgsong force-pushed the fix/ota_cmake branch 2 times, most recently from 4fbe974 to 6f517f7 Compare December 13, 2019 22:29
@tgsong
Copy link
Author

tgsong commented Dec 13, 2019

/bot run checks

pvyawaha
pvyawaha previously approved these changes Dec 13, 2019
@tgsong tgsong changed the title Make OTA depend on http only when wifi is enabled Let vendor choose which OTA backend to enable Dec 13, 2019
@cobusve
Copy link
Member

cobusve commented Dec 13, 2019

This looks like the kind of change which would require a documentation update?

@tgsong
Copy link
Author

tgsong commented Dec 13, 2019

@cobusve Yes, I'll work with Robert to update this page, https://docs.aws.amazon.com/freertos/latest/portingguide/afr-porting-ota.html

@tgsong tgsong merged commit a192b58 into aws:release-candidate Dec 14, 2019
@tgsong tgsong deleted the fix/ota_cmake branch December 14, 2019 00:32
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.

5 participants