search

olvaffe / percetto

Apache License 2.0
19 stars 10 forks source link

Fix to build Android port #22

Closed fnwinter closed 2 years ago

fnwinter commented 2 years ago

ndk has no pthread_getname_np symbol yet, so it needs to be protected by ANDROID. Also std::atomic_fetch_or / and are not provived by ndk. so change them to member functions.

fnwinter commented 2 years ago

I didn't test this patch for linux yet.

rpavlik commented 2 years ago

atomic_fetch_or are actually available, they just needed manual qualification, see #17

The pthread symbol is in platform 26 and above - the cmake build system in #26 handles that.

fnwinter commented 2 years ago

@rpavlik thank you for comments. this patch is not necessary anymore so I'll close this.