Re: [PATCH v2 13/13] Add appveyor.yml

David Gibson <[email protected]> Tue, 27 Sep 2016 15:20:23 +1000
Newsgroups	org.ozlabs.lists.ccan
Message-ID	<[email protected]>
On Thu, Sep 22, 2016 at 09:33:16PM -0600, Kevin Locke wrote:
> This file defines the AppVeyor CI (appveyor.com) settings.
> 
> It builds using make+bash under MSYS2 so that the current build system
> can be used with minimal changes.  It currently only builds configurator
> and generates config.h.
> 
> The build and test commands for more thorough testing are left as
> comments in appveyor.yml so interested parties can use them as a
> starting point for future work.
> 
> Note that several compiler errors not related to configurator are
> printed due to make attempting to generate and include test-depends.
> Although Windows-specific code could be added to Makefile to avoid
> these, it seemed unwarranted if the compile errors may be fixed
> soon.

Nice, though I can't really test it until we've got the rest of the
Windows / MSVC fixes applied.

If possible it would be nice to have in the commit message, and maybe
as a comment in the yml file too, some instructions on how to view the
appveyor results - what page to go to, do you need a login etc.

> 
> Signed-off-by: Kevin Locke <[email protected]>
> ---
>  appveyor.yml | 25 +++++++++++++++++++++++++
>  1 file changed, 25 insertions(+)
>  create mode 100644 appveyor.yml
> 
> diff --git a/appveyor.yml b/appveyor.yml
> new file mode 100644
> index 0000000..822c4bd
> --- /dev/null
> +++ b/appveyor.yml
> @@ -0,0 +1,25 @@
> +version: 0.0.{build}
> +
> +platform:
> +- x86
> +- amd64
> +
> +build_script:
> +# Reset %PATH% to avoid msys/cygwin DLL conflicts which cause
> +# *** fatal error - cygheap base mismatch detected
> +- set PATH=C:\Windows\System32;C:\Windows;C:\Windows\System32\Wbem;C:\msys64\usr\bin
> +# Set environment variables for chosen compiler
> +- "\"C:\\Program Files (x86)\\Microsoft Visual Studio 14.0\\VC\\vcvarsall.bat\" %Platform%"
> +# Ensure config.h can be generated.
> +# Note: Dash options avoid POSIX path conversion by msys
> +- "make config.h CC=cl CCAN_CFLAGS=\"-nologo -Zi -FS -W4 -wd4200 -D_CRT_NONSTDC_NO_WARNINGS -D_CRT_SECURE_NO_WARNINGS\" CFLAGS_FORCE_C_SOURCE=-TC OUTPUT_OPTION=-Fo:$@ DEPGEN= LD=link SHELL=bash"
> +# FIXME:  Work in progress.  Disabled due to unfixed compile errors.
> +# FIXME:  -j%NUMBER_OF_PROCESSORS% won't work without DEPGEN for config.h
> +#         It may be possible to generate .d from cl using /showIncludes
> +#         See https://stackoverflow.com/q/37685069
> +#- "make tools CC=cl CCAN_CFLAGS=\"-nologo -Zi -FS -W4 -wd4200 -D_CRT_NONSTDC_NO_WARNINGS -D_CRT_SECURE_NO_WARNINGS\" CFLAGS_FORCE_C_SOURCE=-TC OUTPUT_OPTION=-Fo:$@ DEPGEN= LD=link SHELL=bash"
> +#- "make -k CC=cl CCAN_CFLAGS=\"-nologo -Zi -FS -W4 -wd4200 -D_CRT_NONSTDC_NO_WARNINGS -D_CRT_SECURE_NO_WARNINGS\" CFLAGS_FORCE_C_SOURCE=-TC OUTPUT_OPTION=-Fo:$@ DEPGEN= LD=link SHELL=bash"
> +
> +test_script:
> +# FIXME:  Work in progress.  Disabled due to unfixed compile errors.
> +#- "make -k check CC=cl CCAN_CFLAGS=\"-nologo -Zi -FS -W4 -wd4200 -D_CRT_NONSTDC_NO_WARNINGS -D_CRT_SECURE_NO_WARNINGS\" CFLAGS_FORCE_C_SOURCE=-TC OUTPUT_OPTION=-Fo:$@ DEPGEN= LD=link SHELL=bash"

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

_______________________________________________
ccan mailing list
[email protected]
https://lists.ozlabs.org/listinfo/ccan
signature.asc (application/pgp-signature, 819 B)