Skip to content

Conversation

AlexAltea
Copy link
Contributor

Fixes issue #10789.

@LilyWangL LilyWangL assigned LilyWangL and Cheney-W and unassigned LilyWangL Apr 20, 2020
Copy link
Contributor

@Cheney-W Cheney-W left a comment

Choose a reason for hiding this comment

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

Thanks for your PR!

  1. Could you please modify the Version: 0.2.2-2 in vcpkg/ports/libyaml/CONTROL to Version: 0.2.2-3?
  2. Could you please modify configure_file(${SOURCE_PATH}/LICENSE ${CURRENT_PACKAGES_DIR}/share/libyaml/copyright COPYONLY) in vcpkg/ports/libyaml/portfile.cmake to configure_file(${SOURCE_PATH}/LICENSE ${CURRENT_PACKAGES_DIR}/share/${PORT}/copyright COPYONLY)?

@Cheney-W Cheney-W added category:port-bug The issue is with a library, which is something the port should already support waiting for response labels Apr 20, 2020
@AlexAltea
Copy link
Contributor Author

@Cheney-W Fixed!

@Cheney-W Cheney-W added info:reviewed Pull Request changes follow basic guidelines and removed waiting for response category:port-bug The issue is with a library, which is something the port should already support labels Apr 21, 2020
@ras0219-msft ras0219-msft merged commit 84041ca into microsoft:master Apr 21, 2020
@ras0219-msft
Copy link
Collaborator

Thanks for the PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants