search

cnlohr / rawdrawandroid

Build android apps without any java, entirely in C and Make
MIT License
3.55k stars 226 forks source link

Add cnfa example #59

Closed DavidVentura closed 1 year ago

DavidVentura commented 1 year ago

Per https://github.com/cntools/cnfa/issues/17, add an example of using cnfa on Android.

dreua commented 1 year ago

You can now rebase your prs on master to fix the ci.

DavidVentura commented 1 year ago

Cool, have rebased

cnlohr commented 1 year ago

This is still failing CI. Any chance you can troubleshoot that before I accept?

DavidVentura commented 1 year ago

Before, the CI was broken on some apt-get issues - now it's unable to find a header.

I had the same error, and found the necessary header at -I$(NDK)/sysroot/usr/include/android in my local NDK. In the PR, I've added an include path which does not seem to work.

I'll try to replicate this NDK setup

cnlohr commented 1 year ago

Thank you.

DavidVentura commented 1 year ago

Had the wrong include path :upside_down_face:

cnlohr commented 1 year ago

This is insufficient for older SDKs, please include both include paths, not just altering it.

DavidVentura commented 1 year ago

I think the Github UI made the diff confusing, I've not removed any path, just added a new one. I've flipped the order so the diff is clearer

DavidVentura commented 1 year ago

Ah, I think I get it; I've added the other path as well

cnlohr commented 1 year ago

Ah!! Ok I see.

cnlohr commented 1 year ago

I know a little annoying to ask now - can you make it so like pressing something triggers the sound? So it doesn't always run? Ram Jamming a loud eeeeeee is bit annoying for a simple test.