-
Notifications
You must be signed in to change notification settings - Fork 43
Conversation
…flicting with defines in NDK, a missing LOG2 function and some missing defines for ZLIB I was able to cross compile it and run it on my SHIELD TV using: make -f makefile.libretro platform=armv-neon -j
Did you run into problems compiling it? |
The hacks I did were without any consideration for other builds, I just wanted it to compile. It was pulled a little too quickly into the main repo, see #74 :) I think this commit should be reverted and the fixes should be implemented in a way that only effects Android. The commit on my own repo was just for info and so that we could find a good way to adjust for it. |
Yeah don't worry if it's problematic I'll revert it it's no big deal if a core doesn't build one day. I was trying to build with the ndk standalone toolchain and I get this:
The buildbot generates it's own build commands deoending on some criteria, this is what it generated |
I think it doesn't like something in the log function and it's parsing some symbol as a escape sequence. |
I was thinking of looking into creating a JNI makefile instead. Do I understand it correctly, that it would make things easier for the buildbot? |
Yes jni makefiles work better and one makefile works for all the ABIS On Tue, May 17, 2016, 7:52 AM diablodiab notifications@github.com wrote:
|
Sending in behalf of @diablodiab, it should be fine
I guess I need the standalone toolchain to get this to build, I'll test once it's merged