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

[dpdk] Fix the hash of the downloaded archive #26657

Merged
merged 1 commit into from
Sep 2, 2022

Conversation

ljishen
Copy link
Contributor

@ljishen ljishen commented Sep 2, 2022

Describe the pull request

  • What does your PR fix?

    Fix the mismatch SHA512 of the downloaded archive

  • Which triplets are supported/not supported? Have you updated the CI baseline?

    No

  • Does your PR follow the maintainer guide?

    Yes

  • If you have added/updated a port: Have you run ./vcpkg x-add-version --all and committed the result?

    Yes

@@ -29,7 +29,7 @@ vcpkg_from_github(
OUT_SOURCE_PATH SOURCE_PATH
REPO DPDK/dpdk
REF v${PORT_VERSION}
Copy link
Member

Choose a reason for hiding this comment

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

Can we change this to the commit SHA instead so that it doesn't break like this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Do you mean using the commit SHA instead of the release tag for the REF value? The upstream didn't update the release and it's my fault missing the change. But I think CI should be able to find this problem.

Copy link
Member

Choose a reason for hiding this comment

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

Oh! I see, this is a missed update from #26643 . In that case nevermind!

@BillyONeal
Copy link
Member

The CLA bot was happy with #26643 so I'm assuming this is OK.

@BillyONeal BillyONeal merged commit 910f20f into microsoft:master Sep 2, 2022
@ljishen ljishen deleted the dpdk branch September 2, 2022 22:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants