[GitHub] marcoabreu commented on a change in pull request #10206: [Doc] Update windows installation doc

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

marcoabreu commented on a change in pull request #10206: [Doc] Update windows installation doc
URL: https://github.com/apache/incubator-mxnet/pull/10206#discussion_r176591643
 
 

 ##########
 File path: docs/install/windows_setup.md
 ##########
 @@ -25,7 +25,7 @@ To build and install MXNet yourself, you need the following dependencies. Instal
 2. Download and Install [CMake](https://cmake.org/) if it is not already installed.
 3. Download and install [OpenCV](http://sourceforge.net/projects/opencvlibrary/files/opencv-win/3.0.0/opencv-3.0.0.exe/download).
 4. Unzip the OpenCV package.
-5. Set the environment variable ```OpenCV_DIR``` to point to the ```OpenCV build directory``` (```c:\utils\opencv\build``` for example).
+5. Set the environment variable ```OpenCV_DIR``` to point to the ```OpenCV build directory``` (```D:\opencv\build\x64\vc14``` for example). Also, you need to add the OpenCV bin directory (```D:\opencv\build\x64\vc14\bin``` for example) to the ``PATH`` variable.
 
 Review comment:
   Nit: Use C instead of D - some people like copy&paste :D

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