Skip to content

HarmonyOS/OpenHarmony Port #13152

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

Open
wants to merge 42 commits into
base: main
Choose a base branch
from
Open

Conversation

Jack253-png
Copy link

Description

SDL support for harmonyos

Existing Issue(s)

#9837

@Jack253-png
Copy link
Author

@slouken Request for review

@Jack253-png
Copy link
Author

@madebr @sezero Request for review

@madebr
Copy link
Contributor

madebr commented May 31, 2025

Can you please rebase on top of current master and remove all commits which you did not author?

git fetch https://github.com/libsdl-org/SDL main
git rebase -i FETCH_HEAD
<remove all commits which you did not author. In vim, "dd" removes the current line, "<ESC>:wq" saves and exits)>
git push git@github.com:Jack253-png/SDL HEAD:OpenMinecraft-Dev -f

@madebr
Copy link
Contributor

madebr commented May 31, 2025

Seeing you implemented vulkan support, does SDL_gpu work? Do the SDL_gpu_examples work?

@Jack253-png
Copy link
Author

Seeing you implemented vulkan support, does SDL_gpu work? Do the SDL_gpu_examples work?

I will test later, because I don't have test platform, I need to find other people to do this.

@Jack253-png
Copy link
Author

Seeing you implemented vulkan support, does SDL_gpu work? Do the SDL_gpu_examples work?

@Starcloudsea here is the owner of the test platform

@sezero
Copy link
Contributor

sezero commented Jun 1, 2025

Do you need the dynapi functionality of SDL? If you don't, insert a similar disabler for ohos around here.

@Starcloudsea
Copy link

To clarify, I'm not a developer, I was just called to test because the computer has good performance, and I don't know about HarmonyOS development. 😅

@Jack253-png
Copy link
Author

Jack253-png commented Jun 1, 2025

Do you need the dynapi functionality of SDL? If you don't, insert a similar disabler for ohos around here.

Maybe I need it, is this PR ready for merging?

@Jack253-png
Copy link
Author

@madebr request for review, I have tested the egl & vulkan surface creation, and it works.
I also finished the touch & keyboard event.

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.

4 participants