[azure] Update vcpkgs (add new port dependencies)#6511
[azure] Update vcpkgs (add new port dependencies)#6511ras0219-msft merged 3 commits intomicrosoft:masterfrom ewertons:master
Conversation
|
In this PR we are adding some new submodules our solution depends on, plus providing fixes for building our solution using vcpkg TOOLCHAIN. |
|
Thanks for the PR! The name Side notes: I bumped the control versions and normalized the spacing. I'd also recommend using a branch for future PRs (instead of master), just to help keep things straight. |
| ) | ||
| endif() | ||
|
|
||
| file(COPY ${CURRENT_INSTALLED_DIR}/share/azure-c-shared-utility/azure_iot_build_rules.cmake DESTINATION ${SOURCE_PATH}/deps/azure-c-shared-utility/configs/) |
There was a problem hiding this comment.
Ports must not write to the /installed folder directly.
Copy the file to ${CURRENT_PACKAGES_DIR} and it will be copied to the /installed directory when the installation is complete.
There was a problem hiding this comment.
Got it, thanks for the review. Yes, I fixed it and tested, it works good.
Please feel free to merge if the changes are good.
Thanks.
|
/AzurePipelines run |
|
Thanks! |
3 participants
No description provided.