Inkscape requirement is undocumented?

[Erisa]

If Inkscape is not installed on the machine/VM running the build, the following error is displayed:

CMake Error at src/gui/CMakeLists.txt:246 (message):
  Could not find inkscape.  Set INKSCAPE_DIR to the path of executable.


-- Configuring incomplete, errors occurred!
See also "C:/Nextcloud/client-building/desktop/build/CMakeFiles/CMakeOutput.log".
See also "C:/Nextcloud/client-building/desktop/build/CMakeFiles/CMakeError.log".
"*** Build FAILED: desktop Release Win64 (single-build-desktop.bat)"
"***** Build FAILED (build.bat)"

Should this be documented? Is there a reason it isn't already?
I've built on two separate VMs and in both cases installing Inkscape to the system resolves the error and results in a successful build.

[FlexW]

Feel free to open a pull request that adds this documentation.

[alexanderharm]

Opened PR #94 to address this.

[Erisa]

Thank you @alexanderharm! I had originally planned to PR this but life got in the way and then I ended up forgetting 😅