Skip to content

SDL3 - #47

Merged
IvanSavenko merged 1 commit into
vcmi:mainfrom
Laserlicht:sdl3
Sep 1, 2026
Merged

SDL3#47
IvanSavenko merged 1 commit into
vcmi:mainfrom
Laserlicht:sdl3

Conversation

@Laserlicht

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread conan_profiles/base/android Outdated
Comment thread conan_profiles/base/ios

# desktop OpenGL is unavailable here anyway and the recipe would produce a broken
# package_info(): it references opengl::opengl without requiring it
sdl/*:opengl=False

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

shouldn't it be fixed in upstream?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Maybe. But not on prio list. Would rather spend time on render PR itself.

Comment thread conan_profiles/base/msvc-intel Outdated
Comment thread conan_profiles/base/msvc-intel
Comment thread conanfile.py
Comment thread conanfile.py Outdated
Comment thread conanfile.py Outdated
Comment thread conanfile.py Outdated
# static on "single app" platforms
isSdlShared = not isMobile
self.options["sdl"].shared = isSdlShared
# SDL's HIDAPI backend calls into libusb, which is not available on iOS and would cause a link error

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

shouldn't it be fixed in upstream (either SDL or recipe)?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Same here

@IvanSavenko
IvanSavenko merged commit f61de7b into vcmi:main Sep 1, 2026
21 of 22 checks passed
@Laserlicht
Laserlicht deleted the sdl3 branch September 1, 2026 19:53
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