asitstands commented on a change in pull request #10951: [MXNET-545] Fix broken cython build URL: https://github.com/apache/incubator-mxnet/pull/10951#discussion_r254390020
########## File path: ci/Jenkinsfile_utils.groovy ########## @@ -67,7 +67,7 @@ def pack_lib(name, libs, include_gcov_data = false) { sh returnStatus: true, script: """ set +e echo "Packing ${libs} into ${name}" -echo ${libs} | sed -e 's/,/ /g' | xargs md5sum +for i in \$(echo ${libs} | sed -e 's/,/ /g'); do md5sum \$i; done Review comment: This is to handle glob patterns in `$libs`. The names of the shared library files produced by cython depend on the python version and platform. Using glob pattterns is convenient. ---------------------------------------------------------------- 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