Closed mdirik closed 8 years ago
You confirm it's ok for you?
Sorry I forgot about this. Linking issue seems to be resolved (even though I couldn't get it working due to not being able to install properly). README still points to the NDK r9b, though. It doesn't work and it should point to NDK r10 instead.
Thanks!
yeah. the readme really ought to be updated. Spent some time tackling the same issue until i bothered to check this issue.
I'm not sure if my build environment is not correctly set up or they are legitimately issues, but there is 2 related issues I encountered while trying to build Superuser.
README file lists API 19 and NDK 9b as requirements, but su binary doesn't compile correctly while using them.
It looks like sys/attr.h was added later in SDK 21.
If I switch to NDK 10, it compiles correctly but the resulting binary won't work in Android 4.4
but no libs/*/placeholder exits as the README says?
And the resulting binary doesn't work
I know selinux contexts don't match but linking problem is seperate and it is a known issue
I transcripted this on a x86 4.4 emulator but the error was same a 4.4 armv7 phone last night. I also tried changing include lines to
linux/attr.h
as they were on older SDK versions, but they seem to be different headers thansys/attr.h
so there were compile errors. I didn't dig it further.