Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

README.md: bad PATH and PYTHONPATH setting for windows #719

Open
fasterpython opened this issue Aug 31, 2018 · 0 comments
Open

README.md: bad PATH and PYTHONPATH setting for windows #719

fasterpython opened this issue Aug 31, 2018 · 0 comments

Comments

@fasterpython
Copy link

In README.md, it says:
SETX PATH C:/Python27/;C:/Python27/Scripts/;C:/OpenCV2.3/opencv/build/x86/vc10/bin/;%PATH%
SETX PYTHONPATH C:/OpenCV2.3/opencv/build/python/2.7/;%PYTHONPATH%

however, the correct one should be:
SETX PYTHONPATH C:\OpenCV2.3\opencvbuild\python\2.7;%PYTHONPATH%
SETX PATH C:\Python27;C:\Python27\Scripts;C:\OpenCV2.3\opencv\build\x86\vc10\bin;%PATH%

and also, it's only correctly in 32-bit python (x86 or x64 cpu, but 32-bit python)
if the platform is x64 cpu and 64-bit python, it should be:
SETX PATH C:\Python27;C:\Python27\Scripts;C:\OpenCV2.3\opencv\build\x64\vc10\bin;%PATH%

==========================================================

and also, since the latest opencv2.4.13.6 (released on 2018-2), the correct PYTHONPAHT should be:
64bit python:
SETX PYTHONPATH C:\OpenCV2.3\opencvbuild\python\2.7\x64;%PYTHONPATH%
or for 32bit python:
SETX PYTHONPATH C:\OpenCV2.3\opencvbuild\python\2.7\x86;%PYTHONPATH%

Because OpenCV2.3.1 was released on 2011, 7 years ago, that's a too too old version for users now.
when I follow README.md

Download OpenCV 2.3 Superpack: http://sourceforge.net/projects/opencvlibrary/files/opencv-win/2.3.1/OpenCV-2.3.1-win-superpack.exe/download

there is a error that:
Cannot load OpenCV library which is required by SimpleCV

so I download the latest opencv2.4.13.6 (released on 2018-2) and setup correct paths, and it all works fine now.

Please update the README.md as suggest before, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant