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

[Fluidsynth] add sndfile feature #24635

Merged
merged 2 commits into from
May 11, 2022

Conversation

hmoffatt
Copy link
Contributor

  • What does your PR fix?

Fixes #24619. Add optional feature sndfile to fluidsynth, causing it to be built with libsndfile enabling extra functionality.

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

All triplets supported, no change to baseline

Yes

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

Yes

@ghost
Copy link

ghost commented May 10, 2022

CLA assistant check
All CLA requirements met.

@FrankXie05 FrankXie05 added the category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist label May 10, 2022
@FrankXie05 FrankXie05 added the requires:all-feature-testing vcpkg install port[all features supported by that port] needs to be demonstrated to function label May 11, 2022
@FrankXie05
Copy link
Contributor

Feature test successfully in the following triplet:

  • x86-windows
  • x64-windows

@FrankXie05 FrankXie05 added info:reviewed Pull Request changes follow basic guidelines and removed requires:all-feature-testing vcpkg install port[all features supported by that port] needs to be demonstrated to function labels May 11, 2022
@FrankXie05 FrankXie05 changed the title add sndfile feature to Fluidsynth port [Fluidsynth] add sndfile feature May 11, 2022
@hmoffatt
Copy link
Contributor Author

hmoffatt commented May 11, 2022

I have also tested on arm64-osx and x64-osx.

@BillyONeal BillyONeal merged commit a7dfa7d into microsoft:master May 11, 2022
@BillyONeal
Copy link
Member

Thanks!

@hmoffatt hmoffatt deleted the fluidsynth-sndfile branch May 11, 2022 23:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[fluidsynth] link to libsndfile
3 participants