[GitHub] szha commented on a change in pull request #11317: Fix the inaccruate instruction on installing nose

[GitBox <gi...@apache.org>]

szha commented on a change in pull request #11317: Fix the inaccruate instruction on installing nose
URL: https://github.com/apache/incubator-mxnet/pull/11317#discussion_r195914841
 
 

 ##########
 File path: python/README.md
 ##########
 @@ -10,12 +10,12 @@ To install MXNet Python package, visit MXNet [Install Instruction](http://mxnet.
 
 For running unit tests, you will need the [nose PyPi package](https://pypi.python.org/pypi/nose). To install:
 ```bash
-pip install --upgrade nose
+sudo pip install --upgrade nose
 
 Review comment:
   I believe this depends on the environment, and it's up to the user to decide whether a sudo is suitable or not.
   For example, I may have pip that comes from virtualenv, conda, pyenv, or something else that requires only user access.

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