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: remove patch for Android build #51660

Closed
wants to merge 1 commit into from

Conversation

MeowShe
Copy link
Contributor

@MeowShe MeowShe commented Feb 4, 2024

Patch to src/trap-handler/trap-handler.h is no longer needed now in Android building.

Tested on Docker with image ubuntu:latest with eca4b1a (HEAD), android-ndk-r26b-linux.

Related to PR #51632, as part of a series of build fixes for Android.

@MeowShe
Copy link
Contributor Author

MeowShe commented Feb 5, 2024

CC @richardlau

@lpinca
Copy link
Member

lpinca commented Feb 6, 2024

Wouldn't it be better to keep the patch support in android_configure.py and only remove the actual patch? We might need to apply another patch in the future, no?

@MeowShe
Copy link
Contributor Author

MeowShe commented Feb 7, 2024

Wouldn't it be better to keep the patch support in android_configure.py and only remove the actual patch? We might need to apply another patch in the future, no?

I understand your point, however, the patch was initially intended as a compromise, and all issues that arise should be addressed at their source, rather than being temporarily mitigated through patches.

But again, I think it's going to take some time to transition, so let's keep the patch for now, and thank you for your input.

@MeowShe MeowShe closed this Feb 10, 2024
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.

2 participants