Skip to content

[vcpkg baseline][vtk-m] fix the sha512#14884

Merged
strega-nil merged 6 commits intomicrosoft:masterfrom
strega-nil:vtk-m-hash
Dec 3, 2020
Merged

[vcpkg baseline][vtk-m] fix the sha512#14884
strega-nil merged 6 commits intomicrosoft:masterfrom
strega-nil:vtk-m-hash

Conversation

@strega-nil
Copy link
Contributor

Describe the pull request

  • What does your PR fix? Fixes #

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

  • Does your PR follow the maintainer guide?

@Neumann-A
Copy link
Contributor

osx already disagrees with that change.

@strega-nil
Copy link
Contributor Author

@Neumann-A this is because osx has a cached binary that is wrong

@Neumann-A
Copy link
Contributor

any knowledge why the SHA changed?

@NancyLi1013 NancyLi1013 added info:internal category:port-bug The issue is with a library, which is something the port should already support labels Dec 2, 2020
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.

Do we need to update version in CONTROL file?

@JackBoosY
Copy link
Contributor

@Neumann-A Do you know what has changed in the current version?

@NancyLi1013 NancyLi1013 changed the title [vtk-m] fix the sha512 [vcpkg baseline][vtk-m] fix the sha512 Dec 2, 2020
@strega-nil
Copy link
Contributor Author

I believe this is the same issue as #14343.

@NancyLi1013
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@NancyLi1013 NancyLi1013 added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Dec 3, 2020
@NancyLi1013
Copy link
Contributor

@strega-nil

Please help merge this PR first if all CI checks pass.

@strega-nil
Copy link
Contributor Author

Merging as "the only issue was strawberry perl not downloading"

@strega-nil strega-nil merged commit ef0c303 into microsoft:master Dec 3, 2020
@strega-nil strega-nil deleted the vtk-m-hash branch December 10, 2020 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:port-bug The issue is with a library, which is something the port should already support info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants