Update sdk vcpkg portfile with VERSION and HEAD_REF#5516
Conversation
|
Thank you for your contribution @WangWeiLin-MV! We will review the pull request and get back to you soon. |
|
Thank you so much! - microsoft/vcpkg#38085 (comment) |
|
|
antkmsft
left a comment
There was a problem hiding this comment.
Release works as intended - Azure/azure-sdk-vcpkg-betas@e41c4e7
LarryOsterman
left a comment
There was a problem hiding this comment.
This looks quite reasonable to me.
One small question: Why the addition of HEAD_REF?
|
CI checks failed with It could be fixed by upgrading the |
There is a recommended optional parameter HEAD_REF for This value allows the option --head in command |
Update 18 sdk vcpkg
portfile.cmake, and updateInitialize-VcpkgRelease.ps1regex.Reference updates PR microsoft/vcpkg#38085 and microsoft/vcpkg#29514 and daily validation PR microsoft/vcpkg#34835
Changes
sdk\**\vcpkg\portfile.cmakeHEAD_REFto allow HEAD builds by matching(?<=\s{4})SHA512 0\n\)withSHA512 0\n HEAD_REF main\n)REFto use variable${VERSION}and be quoted by matching(?<=\s{4})REF (azure.+)@AZ_LIBRARY_VERSION@$withREF "$1${VERSION}"REF azure.*toREF \"azure.*\"to keep "Overriding REF with test release ref" forPublishDailyVcpkgin archetype-cpp-release.yml#L266-L302