Skip to content

Conversation

@yuriy-odonnell-epic
Copy link
Contributor

Describe the pull request

github-actions[bot]
github-actions bot previously approved these changes Jul 14, 2022
@yuriy-odonnell-epic
Copy link
Contributor Author

yuriy-odonnell-epic commented Jul 14, 2022

I've tested this port locally in my application, however I'm only able to test Windows at this point.
The CI seems to have succeeded for ARM and Linux, so I'm cautiously happy to call it "done", at least as an initial minimal version of the port.

I'll squash/rebase it when it passes the reviews.

github-actions[bot]
github-actions bot previously approved these changes Jul 14, 2022
@yuriy-odonnell-epic
Copy link
Contributor Author

Potentially in the future this port should be updated to use BLAKE3-s own CMake script, when/if BLAKE3-team/BLAKE3#247 is merged.

@JonLiu1993 JonLiu1993 added category:new-port The issue is requesting a new library to be added; consider making a PR! requires:author-response labels Jul 15, 2022
@yuriy-odonnell-epic
Copy link
Contributor Author

Rebased and squashed.

github-actions[bot]
github-actions bot previously approved these changes Jul 15, 2022
@yuriy-odonnell-epic
Copy link
Contributor Author

Is there anything preventing this from merging? I'm confused by author-response tag.

@JonLiu1993 JonLiu1993 added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Jul 18, 2022
@vicroms vicroms merged commit c39678d into microsoft:master Jul 18, 2022
@yuriy-odonnell-epic yuriy-odonnell-epic deleted the new-port-blake3 branch July 21, 2022 23:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:new-port The issue is requesting a new library to be added; consider making a PR! 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.

[New Port Request] BLAKE3

3 participants