phhusson / Superuser

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

REQUESTOR multiple define #48

Open ehem opened 8 years ago

ehem commented 8 years ago

In branch stableL, Superuser/Superuser/jni/Android.mk for LOCAL_MODULE:=su, there TWO definitions for REQUESTOR, "-DREQUESTOR=\"$(SUPERUSER_PACKAGE)\"" and "-DREQUESTOR=\"$(shell cat packageName)\"". The preprocessor doesn't really like this. I'd simply rip it out, but it looks like it evolved that way for some reason, so ripping it out without knowing the reason seems bad.