search

yvesf / andiodine

Android VPN over DNS (no root required). Repo moved to gitlab
https://gitlab.com/andiodine/andiodine
149 stars 27 forks source link

Fix a double header definition on newer NDK versions. #37

Closed LenaWil closed 6 years ago

LenaWil commented 6 years ago

Newer Android NDK versions (>21) have APRA Nameser_compat.h, which causes a double struct definition. This patch checks if APRA nameser.h includes APRA nameser_compat.h and if that is the case, it disables android_dns.h.

I don't really know how to install a old version of a NDK, so I can't 100% verify that I didn't break anything, but it should be fine.

LenaWil commented 6 years ago

The continuous integration is failing because someone forgot to accept a license. :(

yvesf commented 6 years ago

I believe we should change to a never NDK version in the travis configuration-file too.

yvesf commented 6 years ago

Thanks for the quick response and for the improvement in general!

LenaWil commented 6 years ago

:-)