phhusson / Superuser

Keeping koush's Superuser fresh
GNU General Public License v3.0
424 stars 50 forks source link

Update Android.mk for building into ROM #19

Closed tombriden closed 8 years ago

tombriden commented 8 years ago

The extra include is to fix

external/Superuser/Superuser/jni/su/su.c: In function 'socket_send_request':
external/Superuser/Superuser/jni/su/su.c:416:5: error: implicit declaration of function 'htonl' [-Werror=implicit-function-declaration]
     write_token(fd, "version", PROTO_VERSION);
phhusson commented 8 years ago

Erm... There is no more include $(BUILD_EXECUTABLE) that sounds wrong

tombriden commented 8 years ago

Yep, I screwed that right up. I've repushed with that restored and also the correct LOCAL_SRC_FILES values

phhusson commented 8 years ago

Sounds good enough to me. I'm a bit sad to have both this Android.mk and Superuser/jni/Android.mk (the later is for ndk-build) I'd appreciate if you could merge them.