Omission in SITL for windows documentation

Hello, I just came across an omission at the SITL for Windows documentation.

The final command of section Install Required Python Packages:

python -m pip install --user lxml

fails with error:

In file included from /usr/include/libxml2/libxml/parser.h:810:0,
from /usr/include/libxml2/libxml/xmlerror.h:10,
from /usr/include/libxml2/libxml/xpath.h:26,
from /tmp/xmlXPathInitKp6gpK.c:1:
/usr/include/libxml2/libxml/encoding.h:28:19: fatal error: iconv.h: No such file or directory
compilation terminated.
*********************************************************************************
Could not find function xmlCheckVersion in library libxml2. Is libxml2 installed?
*********************************************************************************
error: command ‘gcc’ failed with exit status 1

----------------------------------------

Command “/usr/bin/python -u -c “import setuptools, tokenize;file=‘/tmp/pip-build-Zp4od7/lxml/setup.py’;f=getattr(tokenize, ‘open’, open)(file);code=f.read().replace(‘\r\n’, ‘\n’);f.close();exec(compile(code, file, ‘exec’))” install --record /tmp/pip-aR1FvW-record/install-record.txt --single-version-externally-managed --compile --user --prefix=” failed with error code 1 in /tmp/pip-build-Zp4od7/lxml/

I have followed the instructions found here, installed “libxslt-devel” from the cygwin setup and the error went away.

Please update the documentation to include “libxslt-devel” in the list of packages for the cygwin installer.

I have followed the instructions found here, installed “libxslt-devel” from
the cygwin setup and the error went away.

Please update the documentation to include “libxslt-devel” in the list of
packages for the cygwin installer.

Thanks for the report, and clearly stating the solution. However, there’s
actually no need to ask the devs to patch the docs - you can do it
yourself, and it might have taken less time that to make your report :slight_smile:

In the top-right-hand-corner of the URL you linked to
(SITL setup on Windows using Cygwin (not recommended) — Dev documentation),
there’s an “Edit on GitHub” link. If you follow that link and correct the
documentaiton, a PR will be generated which can be easily merged.

Please let me know if you’re happy to do that.