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

Add an Android bindings project #24

Merged
merged 2 commits into from
Apr 15, 2024
Merged

Add an Android bindings project #24

merged 2 commits into from
Apr 15, 2024

Conversation

hwsmm
Copy link
Collaborator

@hwsmm hwsmm commented Apr 15, 2024

This PR adds an Android bindings project to the repository, and a workflow to build binaries for Android.
I separated it from the original project since building it requires Xamarin, and I thought that most consumers of this project won't like it.

Workflow may need to be run once to replace my binaries. Couldn't test workflow since I couldn't run in my forked repository, probably due to secrets?

@smoogipoo
Copy link
Contributor

Please remove all binaries from this PR. We'll run the GH action once after it's merged.

@hwsmm
Copy link
Collaborator Author

hwsmm commented Apr 15, 2024

I just removed them.

@smoogipoo
Copy link
Contributor

Lgtm. I bundled one additional change here for a header that's been removed upstream.

Copy link
Contributor

@smoogipoo smoogipoo left a comment

Choose a reason for hiding this comment

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

Let's do this!

@smoogipoo smoogipoo merged commit 98f308f into ppy:master Apr 15, 2024
1 check passed
EndProject
Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "SDL3-CS.Tests", "SDL3-CS.Tests\SDL3-CS.Tests.csproj", "{CF980481-8227-4BED-970E-6433C83F64CB}"
Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "SDL3-CS-Android", "SDL3-CS-Android\SDL3-CS-Android.csproj", "{CA28F49C-D0BE-47D6-9E82-7A0B8C380B8B}"
Copy link
Member

Choose a reason for hiding this comment

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

Hmm, we'd usually name this SDL3-CS.Android, not -Android

Copy link
Contributor

Choose a reason for hiding this comment

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

Yep, I've got fixes on the way...

kvnp referenced this pull request in kvnp/SDL3-CS May 15, 2024
Add an Android bindings project
@hwsmm hwsmm deleted the android branch August 11, 2024 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants