-
Notifications
You must be signed in to change notification settings - Fork 6
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
Docs/windows install #6
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have requested some minor changes in your PR.
docs/README-windows.md
Outdated
|
||
## Prerequisite | ||
|
||
The steps are tested under this envionment. Please prepare these installed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo: environment
docs/README-windows.md
Outdated
|
||
## Notes on install Cmake | ||
|
||
In order to run Cmake in `CMD`, please make sure `PATH envionment variable` contains the folder of Cmake. eg: `C:\Program Files\CMake\bin` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo: environment
docs/README-windows.md
Outdated
## Notes on install OpenCV | ||
|
||
The folder location of OpenCV is important. This program depends on this library. | ||
The downloaded OpenCV file is usually an archive (self-extracting archive). Please extract to a certain folder. eg: `D:\opencv` (i.e. the `opencv extrated folder`) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo: extracted
docs/README-windows.md
Outdated
opencv_world3411d.pdb | ||
``` | ||
|
||
**Alternatively**, one may add the path of OpenCV binary (eg: `D:\opencv\build\x64\vc15\bin`) to `PATH envionment variable`. You may refer to this [web page](https://www.deciphertechnic.com/install-opencv-with-visual-studio/). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo: Environment
docs/README-windows.md
Outdated
|
||
## Notes on install libwebp | ||
|
||
It locates within `Repo folder`, eg: `D:\splitwebp\3rdparty\libwebp\lib\windows-x64` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Instead of calling it Repo folder
, wouldn't it be specific call it something like "splitwebp
root directory"
docs/README-windows.md
Outdated
|
||
In Properties, go to C/C++ > Additional Include Directories > click on the value and see a pull down menu > click Edit.. > click New Line. And add these paths: | ||
|
||
(if your `opencv extrated folder` is `D:\opencv`) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo: extracted
docs/README-windows.md
Outdated
|
||
1) > By not providing "FindOpenCV.cmake" in CMAKE_MODULE_PATH | ||
|
||
Solution: set the `envionment variable` named `OPENCV_DIR`. Open `CMD`, and run this command (This may require Administrator right. ): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo: environment
docs/README-windows.md
Outdated
|
||
- `System32 folder` An important folder for Windows to run. It contains system files and libraries binary. It usually locates at `C:\Windows\System32` | ||
|
||
- `opencv extrated folder` eg: `D:/opencv` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo: extracted
docs/README-windows.md
Outdated
## Glossory | ||
- `CMD` Command Prompt in Windows, accept user's command. Run it by: Start Menu => search "cmd" => click "Command Prompt" OR Start Menu > "Run" > type "cmd" > press Enter | ||
|
||
- `Repo folder` The folder of this repository cloned into your local machine. eg: `D:\splitwebp` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you changed the "repo folder" to "splitwebp", then don't forget to delete this line :P
I pushed new commit to fix the typo mistakes, remove the pre-built webp library binary, add the instructions for using |
a39c5ad
to
e6c4eda
Compare
Hi. I added some steps to build in Windows 10 as a README file for issue #1
But it seems need further code modification for successfully making this to split animated webp files.