nickguletskii commented on pull request #18452: URL: https://github.com/apache/incubator-mxnet/pull/18452#issuecomment-636870483
> @nickguletskii Thanks for point it out for the fix:), would you mind to backport that PR into v1.7.x branch as well? > Regarding the rest of symlink change in this PR, I suggest to use `ln -sf` instead of `ln -s`, what do you think? I've created a backport here: #18456 . Concerning the other symlink changes: let's see if the CI passes without the ninja link first. Personally, I don't think overwriting symlinks managed by the system's package manager is a very good idea, even in containers, so unless the CI jobs fail without `ln -sf`, I don't think these changes are necessary. ---------------------------------------------------------------- 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: us...@infra.apache.org