[GitHub] [incubator-mxnet] Zha0q1 commented on a change in pull request #20294: [v1.x] Enable MKLDNN + APL Integration in CD

[GitBox <gi...@apache.org>]

Zha0q1 commented on a change in pull request #20294:
URL: https://github.com/apache/incubator-mxnet/pull/20294#discussion_r639251054



##########
File path: cd/python/docker/Dockerfile
##########
@@ -36,6 +36,12 @@ RUN apt-get install -y libgomp1
 
 ARG MXNET_VARIANT
 RUN if [ "$MXNET_VARIANT" = "aarch64_cpu" ] ; then echo "not installing libquadmath0 on aarch64" ; else apt-get install -y libquadmath0 ; fi
+RUN if [ "$MXNET_VARIANT" = "aarch64_cpu" ] ; \
+    then wget https://armkeil.blob.core.windows.net/developer/Files/downloads/hpc/arm-performance-libraries/21-0-0/Ubuntu18.04/arm-performance-libraries_21.0_Ubuntu-18.04_gcc-10.2.tar && \
+    tar -xvf arm-performance-libraries_21.0_Ubuntu-18.04_gcc-10.2.tar && \
+    arm-performance-libraries_21.0_Ubuntu-18.04_gcc-10.2/arm-performance-libraries_21.0_Ubuntu-18.04.sh -a; \
+    fi

Review comment:
       It's in the line above. I thought I would use two `RUN`'s because libqudamath and apl are two different packages, what would you think?




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org