su-binary fails to compile using AOSP 4.1.1_r1 (LOGx API updated)

ChainsDD / su-binary

su binary for android Superuser

Other

456 stars 322 forks source link

su-binary fails to compile using AOSP 4.1.1_r1 (LOGx API updated) #11

Closed r3n4ud closed 12 years ago

r3n4ud commented 12 years ago

./system/core/include/cutils/log.h has changed in 4.1.1. When using su-binary in place of the bare system/extras/su, the compilation process failed on LOGx undefined errors.

git-core commented 12 years ago

This issue has been fixed in the gc branch in slightly different way. See commit ffe8b01e0aaaa2676265fb3ab09657220a6310b8. Considering your patch, it changes some parts that irrelevant of the issue. Did you make it intentionally?

r3n4ud commented 12 years ago

OK, the gc branch fix is for the best, I'll use that. The irrelevant parts of the commit are due to my emacs-tabs-are-evil-before-save-hook settings ;)

I'll close the issue but why don't merge the gc branch jb fix in the master branch or make the gc the default branch or document what exactly are those branch (I have browsed gc-ics but it didn't seem intuitive to me to have to browse the gc branch to find jb support).

Anyway, thanks for your quick response!