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*] Update to 2020-07-19 #12493

Merged
merged 9 commits into from
Jul 28, 2020
Merged

[Azure-IoT-Sdk*] Update to 2020-07-19 #12493

merged 9 commits into from
Jul 28, 2020

Conversation

ewertons
Copy link
Contributor

Azure-IoT-Sdk for C release 2020-07-19

@JackBoosY JackBoosY added the category:port-update The issue is with a library, which is requesting update new revision label Jul 20, 2020
ports/azure-c-shared-utility/CONTROL Outdated Show resolved Hide resolved
ports/azure-iot-sdk-c/CONTROL Outdated Show resolved Hide resolved
ports/azure-macro-utils-c/CONTROL Outdated Show resolved Hide resolved
ports/azure-uamqp-c/CONTROL Outdated Show resolved Hide resolved
ports/azure-uhttp-c/CONTROL Outdated Show resolved Hide resolved
ports/azure-umqtt-c/CONTROL Outdated Show resolved Hide resolved
ports/umock-c/CONTROL Outdated Show resolved Hide resolved
@NancyLi1013
Copy link
Contributor

Hi @ewertons
Thanks for this PR.
Have you tested these features?

@ewertons
Copy link
Contributor Author

Hi @NancyLi1013 , thanks for the review and suggested changes.
Yes, our packages have been tested. Please merge the PR as soon as possible, so we can proceed with our release process.
Thanks.

Copy link
Contributor

@NancyLi1013 NancyLi1013 left a comment

Choose a reason for hiding this comment

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

There are no portfile.cmake files that also need to be updated for azure-macro-utils-c and umock-c.
Could you please update them?

@ewertons
Copy link
Contributor Author

Thanks @NancyLi1013 , actually upon verification these two ports do not need to change. I updated the PR not to update azure-macro-utils-c and umock-c.

@NancyLi1013
Copy link
Contributor

@ewertons
Thanks for your clarification.
LGTM now.

@ewertons
Copy link
Contributor Author

@NancyLi1013 , if the changes are good could we please get them merged? Our latest release depends on this update. Thanks.

@NancyLi1013
Copy link
Contributor

@ewertons
Since there is something wrong with CI checks. I will try to rerun this PR.
We will merge this if all CI checks have passed.

@NancyLi1013
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@NancyLi1013 NancyLi1013 changed the title Azure-IoT-Sdk for C release 2020-07-19 [Azure-IoT-Sdk*] Update to 2020-07-19 Jul 24, 2020
@NancyLi1013
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ras0219-msft
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ras0219-msft ras0219-msft merged commit ffa4158 into microsoft:master Jul 28, 2020
hellozee pushed a commit to hellozee/vcpkg that referenced this pull request Sep 11, 2020
* Azure-IoT-Sdk for C release 2020-07-19

* Update ports/azure-c-shared-utility/CONTROL

Co-authored-by: NancyLi1013 <[email protected]>

* Update ports/azure-iot-sdk-c/CONTROL

Co-authored-by: NancyLi1013 <[email protected]>

* Update ports/azure-uamqp-c/CONTROL

Co-authored-by: NancyLi1013 <[email protected]>

* Update ports/azure-macro-utils-c/CONTROL

Co-authored-by: NancyLi1013 <[email protected]>

* Update ports/azure-umqtt-c/CONTROL

Co-authored-by: NancyLi1013 <[email protected]>

* Update ports/azure-uhttp-c/CONTROL

Co-authored-by: NancyLi1013 <[email protected]>

* Update ports/umock-c/CONTROL

Co-authored-by: NancyLi1013 <[email protected]>

* Do not update azure-macro-utils-c and umock-c for release 2020-07-19

Co-authored-by: NancyLi1013 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-update The issue is with a library, which is requesting update new revision
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants