ORC-1100: Add vcpkg release document #1746
Conversation
|
|
||
| > Expected hash: 141afbd6d83b8c8032df071838e7da61eedb3d22289642f76669b6efd167d7550b200bd3542f012d0b63c9ae2572d83fcb1b7f76537b6fa6f980aebf37e2cde2 | ||
|
|
||
| > Actual hash (you should update): 7b9d6f9613f48b41843618465c0c71ed70e5695b4fc4b3db15d0bbfe4c5126e0c6c8f1114d6c450a56b4ddf0357d89bd40cb03c0964275c59761cc0f8fec1291 |
There was a problem hiding this comment.
Is the actual hash a constant value or we get it from somewhere?
Don't know much about vcpkg
There was a problem hiding this comment.
Thank you for making a PR with the example, @deshanxiao .
The SHA512 value is provided by Apache ORC community like the following.
7f46f2c184ecefd6791f1a53fb062286818bd8710c3f08b94dd3cac365e240ee
Do you mean to use the error message recommended value, Expected hash, in the PR?
|
@dongjoon-hyun yes, I got the hash from the error message. From the vcpkg document, it recommand us to update it to zero and get the hash from error message too. |
|
For orc 1.9.2: 141afbd6d83b8c8032df071838e7da61eedb3d22289642f76669b6efd167d7550b200bd3542f012d0b63c9ae2572d83fcb1b7f76537b6fa6f980aebf37e2cde2 |
### What changes were proposed in this pull request? This PR is aimed to add vcpkg document to note how to update vcpkg package when we release a new version. ### Why are the changes needed? To help release manager to release our latest package in vcpkg. ### How was this patch tested? N/A ### Was this patch authored or co-authored using generative AI tooling? No Closes #1746 from deshanxiao/deshan/update-doc. Authored-by: deshanxiao <deshanxiao@microsoft.com> Signed-off-by: Dongjoon Hyun <dongjoon@apache.org> (cherry picked from commit c87707c) Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
|
Thank you, @deshanxiao and all. Merged to main/2.0. We can revise it later if needed. |
What changes were proposed in this pull request?
This PR is aimed to add vcpkg document to note how to update vcpkg package when we release a new version.
Why are the changes needed?
To help release manager to release our latest package in vcpkg.
How was this patch tested?
N/A
Was this patch authored or co-authored using generative AI tooling?
No