Re: [PATCH] Add continuous integration build configuration for Cygwin

[Jon Turney <jon dot turney at dronecode dot org dot uk>]

On 28/06/2016 10:40, Qian Hong wrote:

> I'd like to volunteer to maintain a(nother) continuous integration
> build for Cygwin.
[...]

Nice.

> +    commands:
> +      - uname -a
> +      - id
> +      - C:/cygwin-installer.exe --site http://mirrors.tea-ci.org/cygwin --local-package-dir Z:/tmp/cygwin -W -P gettext-devel,zlib-devel,libiconv,libiconv-devel,mingw64-i686-gcc-g++,mingw64-i686-zlib,mingw64-x86_64-gcc-core,mingw64-x86_64-gcc-g++,mingw64-x86_64-zlib,dejagnu,dblatex,docbook-xml45,docbook-xsl,xmlto -q &> /dev/null

A couple of points:

Firstly, the cygwin FAQ is out of date and docbook2X is also required to 
build the documentation.

I guess this isn't causing visible failures since, unfortunately, the 
makefile just ignores any errors while building documentation.

I guess it would be better if this was handled by configuration so 
documentation build errors were reported if we had determined we could 
built the documentation.

Secondly, it's only obvious from context that 'cygwin-installer' is copy 
of cygwin setup pre-installed in your build VM.  You might want to 
consider using 'wget -N' and/or it's normal name here.