Thanks for this!

One thing that is not clear to me:

The Unbound folks will need to add the copyright and license information if the PR is accepted.

Are you referring to the text NLnet Labs adds to the start of the files or something android related?

Hi @gthess,

Are you referring to the text NLnet Labs adds to the start of the files ...

Yes.

Already merged this through the command line but before the last merge commit above and github does not see this PR as merged.
Also moved the android build scripts to contrib/ and marked the android builds as "allowed to fail" in case of future changes on the android world.
Thanks!

@gthess,

Sounds good.

Someone was asking about Android on the mailing list recently at libunbound on android. It looks like it was good timing.

It also looks like the change made its way to downstream:

$ git merge upstream/master
Merge made by the 'recursive' strategy.
 .travis.yml                                     | 22 ++++++++++++++++++----
 README-Travis.md                                | 18 +++++++++---------
 {android => contrib/android}/15-android.conf    |  0
 {android => contrib/android}/install_expat.sh   |  0
 {android => contrib/android}/install_ndk.sh     |  0
 {android => contrib/android}/install_openssl.sh |  2 +-
 {android => contrib/android}/setenv_android.sh  |  0
 doc/Changelog                                   |  2 ++
 8 files changed, 30 insertions(+), 14 deletions(-)
 rename {android => contrib/android}/15-android.conf (100%)
 rename {android => contrib/android}/install_expat.sh (100%)
 rename {android => contrib/android}/install_ndk.sh (100%)
 rename {android => contrib/android}/install_openssl.sh (94%)
 rename {android => contrib/android}/setenv_android.sh (100%)

@gthess,

It may be a good idea to update config.guess and config.sub. The existing ones are from 2016. I believe the newer ones have better support for NetBSD and Android (among other fixes).

You can fetch the latest config.guess and config.sub with:

wget -O config.guess 'https://git.savannah.gnu.org/gitweb/?p=config.git;a=blob_plain;f=config.guess;hb=HEAD'
wget -O config.sub 'https://git.savannah.gnu.org/gitweb/?p=config.git;a=blob_plain;f=config.sub;hb=HEAD'
chmod a+x config.guess config.sub

Also see config.guess, config.sub at top level in the GNU manual.

Good point, I can look into it.
Do you bring it up because you encountered an issue or just noticed?

Updated on 114d650.

@gthess ,

Do you bring it up because you encountered an issue or just noticed?

No, no problems detected. It was just something I noticed while poking around.