marcoabreu commented on a change in pull request #13532: [MXNET-43] Fix Jetson compilation URL: https://github.com/apache/incubator-mxnet/pull/13532#discussion_r238889199
########## File path: ci/docker/Dockerfile.build.jetson ########## @@ -77,6 +77,7 @@ RUN JETPACK_DOWNLOAD_PREFIX=https://developer.download.nvidia.com/devzone/devcen dpkg -i --force-architecture $ARM_NVINFER_INSTALLER_PACKAGE && \ dpkg -i --force-architecture $ARM_NVINFER_DEV_INSTALLER_PACKAGE && \ apt update -y || true && apt install -y cuda-libraries-dev-9-0 libcudnn7-dev libnvinfer-dev +RUN cd /usr/include/aarch64-linux-gnu/ && ln -s cudnn_v7.h cudnn.h Review comment: Because that adds additional side effects to this command. If there's no reason to introduce side effects, you generally would want to avoid it. For that symlink, I don't see much reason. Otherwise, follow up commands might now write to that new path since they assume an unchanged path. ---------------------------------------------------------------- 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
