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

Docs/windows install #6

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

oktak
Copy link

@oktak oktak commented Oct 10, 2020

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.

Copy link
Owner

@Razdeep Razdeep left a 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.


## Prerequisite

The steps are tested under this envionment. Please prepare these installed.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo: environment


## 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`
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo: environment

## 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`)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo: extracted

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/).
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo: Environment


## Notes on install libwebp

It locates within `Repo folder`, eg: `D:\splitwebp\3rdparty\libwebp\lib\windows-x64`
Copy link
Owner

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"


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`)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo: extracted


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. ):
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo: environment


- `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`
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo: extracted

## 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`
Copy link
Owner

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

@oktak
Copy link
Author

oktak commented Oct 11, 2020

I pushed new commit to fix the typo mistakes, remove the pre-built webp library binary, add the instructions for using vcpkg.

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

Successfully merging this pull request may close these issues.

None yet

2 participants