summaryrefslogtreecommitdiffstats
path: root/scripts/docker
diff options
context:
space:
mode:
authorGravatar Dirk Hohndel <dirk@hohndel.org>2020-04-24 15:27:40 -0700
committerGravatar Dirk Hohndel <dirk@hohndel.org>2020-04-25 13:18:41 -0700
commitd3e495efd006abe5af03e396499bfcf60135a3e2 (patch)
treea324df8f3b65e560af19848d06003d619dae7f52 /scripts/docker
parentf112be7a619e93e20341d1ed92681cd8077e2798 (diff)
downloadsubsurface-d3e495efd006abe5af03e396499bfcf60135a3e2.tar.gz
build-system/Android: use PREFIX outside of NDK
If we install our support libraries into the NDK we later run into include path order issues that result in strange errors around the inclusion of math.h (because we find the C version of that include file that ships with the NDK before we find the libstdc++ version of math.h (because the include path for our support libraries is listed before the libstdc++ include search path). By having a distinct install-root for our libraries we can avoid this problem. Remove the previous hack that tried to work around the symptoms of this issue. Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'scripts/docker')
-rw-r--r--scripts/docker/android-build-container/Dockerfile4
1 files changed, 0 insertions, 4 deletions
diff --git a/scripts/docker/android-build-container/Dockerfile b/scripts/docker/android-build-container/Dockerfile
index 509443f85..1fcc1aa76 100644
--- a/scripts/docker/android-build-container/Dockerfile
+++ b/scripts/docker/android-build-container/Dockerfile
@@ -37,10 +37,6 @@ RUN cd /android/cmake-3.13.2 && \
# run the build wrapper in prep mode
RUN cd /android && bash -x /android/android-build-wrapper.sh -prep-only
-# uggly hack to work around some breakage in the NDK which makes our
-# compiles fail
-#RUN sed -i '313,+13s/^using/\/\/using/' /android/android-ndk-r21/sources/cxx-stl/llvm-libc++/include/cmath
-
# clean up the files that we don't need to keep the container smaller
RUN cd /android && \
apt-get remove -y gcc g++-7 libstdc++-7-dev && \