larroy commented on a change in pull request #11055: [MXNET-57] Added support 
android64 
URL: https://github.com/apache/incubator-mxnet/pull/11055#discussion_r194963500
 
 

 ##########
 File path: ci/docker/runtime_functions.sh
 ##########
 @@ -163,6 +163,30 @@ build_android_arm64() {
     cp dist/*.whl /work/build
 }
 
+build_android_arm64() {
+    set -ex
+    cd /work/build
+#        -DCMAKE_ANDROID_NDK=${CROSS_ROOT} \
+#        -DCMAKE_SYSTEM_VERSION=${ANDROID_NDK_REVISION} \
+#        -DCMAKE_SYSTEM_NAME=Android \
+    cmake\
+        -DANDROID=ON \
+        -DUSE_CUDA=OFF\
+        -DUSE_SSE=OFF\
+        -DUSE_LAPACK=OFF\
+        -DUSE_OPENCV=OFF\
+        -DUSE_OPENMP=OFF\
+        -DUSE_SIGNAL_HANDLER=ON\
+        -DCMAKE_BUILD_TYPE=RelWithDebInfo\
 
 Review comment:
   I don't see how the points you are making about an experimental build which 
is not official nor tested are of any relevance at this stage of Android 
support. This is a development build and I want debug symbols to diagnose 
crashes, this should be reason enough to have it as it is, because size of 
build doesn't really matter right now, and debugability is more important.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to