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

build: fix dependency on SDL2 #335

Merged
merged 1 commit into from
Mar 10, 2024

Conversation

rkitover
Copy link
Contributor

This is a followup on 81d5a24 (build: fix building with MSVC, 2024-03-07) and 68a555e (build: fix attribute((aligned(x))) on gcc/clang, 2024-03-09) for the purposes of creating a vcpkg FAudio port.

Add a Requires.private: sdl2 to the pkgconf file for the SDL2 dependency when not using PLATFORM_WIN32.

Add the SDL2 dependency to the generated cmake config file when not using PLATFORM_WIN32 as well.

This is a followup on 81d5a24 (build: fix building with MSVC,
2024-03-07) and 68a555e (build: fix __attribute__((aligned(x))) on
gcc/clang, 2024-03-09) for the purposes of creating a vcpkg FAudio port.

Add a `Requires.private: sdl2` to the pkgconf file for the SDL2
dependency when not using PLATFORM_WIN32.

Add the SDL2 dependency to the generated cmake config file when not
using PLATFORM_WIN32 as well.

Signed-off-by: Rafael Kitover <[email protected]>
@flibitijibibo flibitijibibo merged commit cfdc4db into FNA-XNA:master Mar 10, 2024
5 checks passed
rkitover added a commit to rkitover/vcpkg that referenced this pull request Mar 10, 2024
Add new port FAudio, an "accuracy-focused XAudio reimplementation for
open platforms".

Use the PLATFORM_WIN32 cmake flag on Windows, otherwise use the sdl2
dependency.

MSVC build PR is here:

FNA-XNA/FAudio#333

, followup PR to fix alignment is here:

FNA-XNA/FAudio#334

, and another followup for the SDL2 dependency:

FNA-XNA/FAudio#335

, the MSYS2 package is here:

https://github.com/msys2/MINGW-packages/blob/master/mingw-w64-faudio/PKGBUILD
.

Signed-off-by: Rafael Kitover <[email protected]>
@rkitover rkitover deleted the build-sdl2-dependency branch March 12, 2024 16:53
@rkitover
Copy link
Contributor Author

@flibitijibibo wanted to give you an update, my faudio port for vcpkg has been merged, and I also fixed the MSYS2 faudio package to use PLATFORM_WIN32.

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.

3 participants