leezu commented on a change in pull request #20474:
URL: https://github.com/apache/incubator-mxnet/pull/20474#discussion_r725211061



##########
File path: tools/staticbuild/build_lib.sh
##########
@@ -36,13 +40,18 @@ cmake -GNinja -C $cmake_config \
       -DCMAKE_OSX_DEPLOYMENT_TARGET=10.13 \
       ..
 ninja
+if [[ ! $PLATFORM == 'darwin' ]] && [[ $BLAS == 'mkl' ]]; then
+    patchelf --set-rpath 
"/opt/intel/oneapi/mkl/${INTEL_MKL}/lib/intel64/:\$ORIGIN" --force-rpath 
libmxnet.so

Review comment:
       Will oneapi/mkl always be in `/opt/intel`? Is there a reason for not 
asking users to fix their run-time search path environment variables instead 
(which ideally would automatically be set correctly upon installation of 
oneapi/mkl)?




-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to