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

[portsmf] update to 238 #19423

Merged
merged 1 commit into from
Aug 10, 2021
Merged

[portsmf] update to 238 #19423

merged 1 commit into from
Aug 10, 2021

Conversation

Be-ing
Copy link
Contributor

@Be-ing Be-ing commented Aug 7, 2021

Describe the pull request

  • What does your PR fix?

This fixes a missing #include <cstring> in one of the library's headers (allegro.h).

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

    all

  • 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

This fixes a missing #include <cstring> in one of the library's
headers (allegro.h). Also the header files are installed to the
lowercase "portsmf" directory to match the old build system.
@Be-ing Be-ing changed the title [portsmf] update to 237 [portsmf] update to 238 Aug 7, 2021
@JonLiu1993 JonLiu1993 self-assigned this Aug 9, 2021
@JonLiu1993 JonLiu1993 added category:port-bug The issue is with a library, which is something the port should already support category:port-update The issue is with a library, which is requesting update new revision labels Aug 9, 2021
@JonLiu1993 JonLiu1993 added the info:reviewed Pull Request changes follow basic guidelines label Aug 9, 2021
@dan-shaw dan-shaw merged commit 3a37cc4 into microsoft:master Aug 10, 2021
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 category:port-update The issue is with a library, which is requesting update new revision info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants