leezu commented on a change in pull request #17583: Python 2 cleanup
URL: https://github.com/apache/incubator-mxnet/pull/17583#discussion_r378646079
 
 

 ##########
 File path: ci/docker/runtime_functions.sh
 ##########
 @@ -43,11 +43,10 @@ scala_prepare() {
 
 check_cython() {
     set -ex
-    local python_ver=$1
-    local is_cython_used=$(python${python_ver} <<EOF
+    local is_cython_used=$(python3 <<EOF
 
 Review comment:
   What do you mean? `check_cython` is only called at one place and I modified 
that place to not pass the `python_ver`.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to