You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@mxnet.apache.org by GitBox <gi...@apache.org> on 2018/11/30 15:41:22 UTC

[GitHub] larroy commented on a change in pull request #13431: Use ~/.ccache as default ccache directory so is not cache is not eras…

larroy commented on a change in pull request #13431: Use ~/.ccache as default ccache directory so is not cache is not eras…
URL: https://github.com/apache/incubator-mxnet/pull/13431#discussion_r237902937
 
 

 ##########
 File path: ci/build.py
 ##########
 @@ -200,7 +200,7 @@ def default_ccache_dir() -> str:
         ccache_dir = "/tmp/_mxnet_ccache"
 
 Review comment:
   I was going to look at this, but the pr was merged. We can change it in another PR but have to test locally that docker can access the home folder.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services