marcoabreu commented on a change in pull request #19034:
URL: https://github.com/apache/incubator-mxnet/pull/19034#discussion_r479622471



##########
File path: ci/docker/Dockerfile.build.centos7
##########
@@ -69,8 +70,8 @@ RUN yum -y check-update || true && \
         libzstd-devel && \
     yum clean all
 
-# Make GCC7, Python 3.5 and Maven 3.3 Software Collections available by default
-# during build and runtime of this container
+# Make Python 3.6 and Maven 3.3 Software Collections available by default 
during
+# the following build steps in this Dockerfile
 SHELL [ "/usr/bin/scl", "enable", "devtoolset-7", "rh-python36", "rh-maven35" ]

Review comment:
       I assume the goal here is to test gcc-7 in CI and gcc-9 in CD? Shall we 
not just entirely go to gcc-9?




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


Reply via email to