Closed Bug 1577220 Opened 2 years ago Closed 2 years ago

update to the r20 NDK

Categories

(Firefox Build System :: General, task)

All
Android
task
Not set
normal

Tracking

(firefox71 fixed)

RESOLVED FIXED
mozilla71
Tracking Status
firefox71 --- fixed

People

(Reporter: froydnj, Assigned: froydnj)

References

(Depends on 1 open bug)

Details

Attachments

(1 file)

This version of the NDK contains updated libc++ headers, which have more C++17 stuff that we need for bug 1566288 and friends.

r20 libc++ apparently has some issues with visibility macros getting changed:

https://treeherder.mozilla.org/#/jobs?repo=try&revision=d3630beca44a1061589011a9d749b8f2dc47cf40&selectedJob=263882694

/builds/worker/fetches/android-ndk/sources/cxx-stl/llvm-libc++/include/support/xlocale/__posix_l_fallback.h:23:8: error: visibility does not match previous declaration
/builds/worker/fetches/android-ndk/sources/cxx-stl/llvm-libc++/include/support/xlocale/__posix_l_fallback.h:27:8: error: visibility does not match previous declaration
/builds/worker/fetches/android-ndk/sources/cxx-stl/llvm-libc++/include/support/xlocale/__posix_l_fallback.h:31:8: error: visibility does not match previous declaration
/builds/worker/fetches/android-ndk/sources/cxx-stl/llvm-libc++/include/support/xlocale/__posix_l_fallback.h:35:8: error: visibility does not match previous declaration
/builds/worker/fetches/android-ndk/sources/cxx-stl/llvm-libc++/include/support/xlocale/__posix_l_fallback.h:39:8: error: visibility does not match previous declaration
/builds/worker/fetches/android-ndk/sources/cxx-stl/llvm-libc++/include/support/xlocale/__posix_l_fallback.h:43:8: error: visibility does not match previous declaration
/builds/worker/fetches/android-ndk/sources/cxx-stl/llvm-libc++/include/support/xlocale/__posix_l_fallback.h:47:8: error: visibility does not match previous declaration
/builds/worker/fetches/android-ndk/sources/cxx-stl/llvm-libc++/include/support/xlocale/__posix_l_fallback.h:51:8: error: visibility does not match previous declaration
/builds/worker/fetches/android-ndk/sources/cxx-stl/llvm-libc++/include/support/xlocale/__posix_l_fallback.h:55:8: error: visibility does not match previous declaration
/builds/worker/fetches/android-ndk/sources/cxx-stl/llvm-libc++/include/support/xlocale/__posix_l_fallback.h:59:8: error: visibility does not match previous declaration
/builds/worker/fetches/android-ndk/sources/cxx-stl/llvm-libc++/include/support/xlocale/__posix_l_fallback.h:63:8: error: visibility does not match previous declaration
/builds/worker/fetches/android-ndk/sources/cxx-stl/llvm-libc++/include/support/xlocale/__posix_l_fallback.h:67:8: error: visibility does not match previous declaration
/builds/worker/fetches/android-ndk/sources/cxx-stl/llvm-libc++/include/support/xlocale/__posix_l_fallback.h:71:8: error: visibility does not match previous declaration
/builds/worker/fetches/android-ndk/sources/cxx-stl/llvm-libc++/include/support/xlocale/__posix_l_fallback.h:75:8: error: visibility does not match previous declaration
/builds/worker/fetches/android-ndk/sources/cxx-stl/llvm-libc++/include/support/xlocale/__posix_l_fallback.h:79:8: error: visibility does not match previous declaration
/builds/worker/fetches/android-ndk/sources/cxx-stl/llvm-libc++/include/support/xlocale/__posix_l_fallback.h:83:8: error: visibility does not match previous declaration
/builds/worker/fetches/android-ndk/sources/cxx-stl/llvm-libc++/include/support/xlocale/__posix_l_fallback.h:87:8: error: visibility does not match previous declaration
/builds/worker/fetches/android-ndk/sources/cxx-stl/llvm-libc++/include/support/xlocale/__posix_l_fallback.h:91:8: error: visibility does not match previous declaration
/builds/worker/fetches/android-ndk/sources/cxx-stl/llvm-libc++/include/support/xlocale/__posix_l_fallback.h:95:8: error: visibility does not match previous declaration

which we worked around once upon a time:

https://searchfox.org/mozilla-central/source/build/moz.configure/toolchain.configure#1304-1355

but now we need to update our hacks.

It's also possible that upgrading not quite so far will still bring the libc++ features we want without the internal refactorings that break our visibility shenanigans.

r18b's headers are broken with respect to 16-bit floating-point support. r19c's headers have the same issue as r20. Time to fix things. :(

Depends on: 1578478
Depends on: 1578825
Depends on: 1579523
Depends on: 1579527

This version has a newer libc++, brings along some linker fixes, etc.

Depends on: 1581628
Pushed by nfroyd@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/62bb12065004
update to the r20 Android NDK; r=snorp,nalexander
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla71
You need to log in before you can comment on or make changes to this bug.