Skip to content
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

[azure-iot-sdk-c] Update public-preview feature to branch with fixed telemetry #9059

Merged
merged 6 commits into from
Nov 22, 2019
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion ports/azure-iot-sdk-c/CONTROL
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
Source: azure-iot-sdk-c
Version: 2019-10-11.2
Version: 2019-11-21.1
Build-Depends: azure-uamqp-c, azure-umqtt-c, azure-c-shared-utility, parson, azure-uhttp-c, azure-macro-utils-c, umock-c
Description: A C99 SDK for connecting devices to Microsoft Azure IoT services

Expand Down
20 changes: 0 additions & 20 deletions ports/azure-iot-sdk-c/cmake-hsm-option.patch

This file was deleted.

6 changes: 3 additions & 3 deletions ports/azure-iot-sdk-c/portfile.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -6,10 +6,10 @@ if("public-preview" IN_LIST FEATURES)
vcpkg_from_github(
OUT_SOURCE_PATH SOURCE_PATH
REPO Azure/azure-iot-sdk-c
REF d1cdf78b5160af8e08354e102a6b96395eee79e1
SHA512 0efbfc19e5eef4831b55ded0e8d88e83194bc0f26886841ddc83405c15b7f1bae983e22dc569e22846acd78b843b9e7492883b7c502f4eed92ff80ef45a9942d
REF cb2e8d390df56ffa31d08ca0a79ab58ff96160cc
SHA512 6798b17d6768b3ccbd0eb66719b50f364cd951736eb71110e2dc9deca054a1566ff88b9e8c5e9b52536e4308cad6cd3cbebff3282c123083e3afaee5535e724b
HEAD_REF public-preview
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This commit commits on the branch public-preview-merge, so need to modify HEAD_REF to public-preview-merge.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@LilyWangL - thanks for feedback. You're correct on public-preview-merge branch. I wanted to get a CI pass of the fix as it was even though our public-preview branch was being updated in parallel. public-preview now has the changed merged into it, and I've updated our port file to point at public-preview's SHA.

I'm moving PR as ready-to-review now.

PATCHES improve-external-deps.patch cmake-hsm-option.patch
PATCHES improve-external-deps.patch
)
else()
vcpkg_from_github(
Expand Down