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

Update to foobar2000 SDK 2022-01-04 and use vcpkg manifest mode #419

Merged
merged 1 commit into from
Jan 10, 2022

Conversation

reupen
Copy link
Owner

@reupen reupen commented Jan 9, 2022

This updates submodules to bring in changes from foobar2000 SDK 2022-01-04.

Some bits of code were tidied up as part of this and fmt was added as a dependency (std::format() isn't used as it's not currently available in VS 2022 with the /std:c++20 compiler option).

Finally, vcpkg is now used in manifest mode to simplify handling of dependencies. Currently, only a single manifest in the root directory is used rather than having manifests for each project (I'm not sure how exactly the latter would behave, particularly if the manifests conflicted with each other).

@reupen reupen force-pushed the SDK-2022-01-04 branch 3 times, most recently from c750743 to 67f083d Compare January 10, 2022 20:05
@reupen reupen marked this pull request as ready for review January 10, 2022 20:05
This updates submodules to bring in changes from foobar2000 SDK 2022-01-04.

Some bits of code were tidied up as part of this and `fmt` was added as a dependency (`std::format()` isn't used as it's not currently available in VS 2022 with the /std:c++20 compiler option).

Finally, vcpkg is now used in manifest mode to simplify handling of dependencies. Currently, only a single manifest in the root directory is used rather than having manifests for each project (I'm not sure how exactly the latter would behave, particularly if the manifests conflicted with each other).
@reupen reupen merged commit 04aafb5 into master Jan 10, 2022
@reupen reupen deleted the SDK-2022-01-04 branch January 10, 2022 20:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant