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

Remove all ndk-build samples. #1023

Merged
merged 1 commit into from
May 7, 2024
Merged

Conversation

DanAlbert
Copy link
Member

@DanAlbert DanAlbert commented May 7, 2024

None of these show anything interesting. At first glance I thought a few might be worth preserving, but the few interesting things they do are actually superfluous. I'm removing all of them for now and will add a new sample soon specifically to show how to use ndk-build in a substantial enough way to be useful.

@DanAlbert DanAlbert requested a review from jfgoog May 7, 2024 19:43
@DanAlbert DanAlbert force-pushed the cleanup-ndkbuild branch from b3f8e49 to 39a16cc Compare May 7, 2024 20:01
@DanAlbert DanAlbert changed the title Remove most ndk-build samples. Remove all ndk-build samples. May 7, 2024
@DanAlbert
Copy link
Member Author

Okay, none of these are actually interesting. The most they manage to do is show how to do the gradle side of this, but the Android.mk themselves are badly written because they're trying to share with their "parent" samples. I'll add a focused sample for ndk-build soonish.

None of these show anything interesting. At first glance I thought a few
might be worth preserving, but the few interesting things they do are
actually superfluous. I'm removing all of them for now and will add a
new sample soon specifically to show how to use ndk-build in a
substantial enough way to be useful.
@DanAlbert DanAlbert force-pushed the cleanup-ndkbuild branch from 39a16cc to 51571d5 Compare May 7, 2024 20:09
@DanAlbert DanAlbert merged commit 47e515d into android:main May 7, 2024
2 checks passed
@DanAlbert DanAlbert deleted the cleanup-ndkbuild branch May 7, 2024 21:10
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