-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
[sdl3-image] Add new port sdl3-image #42849
Conversation
Note: I will be converting your PR to draft status. When you're ready, please revert to "ready for review". That way, I can be aware that you've responded since you can't modify the tags. |
6b08c0d
to
9554171
Compare
@shybovycha
|
@Mengna-Li i can confirm this happens. apparently, SDL3-image requires SDL3: if(NOT TARGET SDL3::Headers OR NOT TARGET ${sdl3_target_name})
find_package(SDL3 ${SDL_REQUIRED_VERSION} REQUIRED COMPONENTS ${sdl_required_components})
endif() which would require a separate dependency entry in both |
I think this way is better. |
Note: I will be converting your PR to draft status. When you're ready, please revert to "ready for review". That way, I can be aware that you've responded since you can't modify the tags. |
@Mengna-Li i have added the |
Usage tested pass on |
d786f36
to
465ee6c
Compare
Add
SDL3-image
port. There already is anSDL3
port, but none for the relevant components (mixer, ttf, image). Used the latest 3.x release tag (preview-3.1.0
)find_package
calls are REQUIRED, are satisfied byvcpkg.json
's declared dependencies, or disabled with CMAKE_DISABLE_FIND_PACKAGE_Xxx.vcpkg.json
matches what upstream says.vcpkg.json
matches what upstream says../vcpkg x-add-version --all
and committing the result.