-
Notifications
You must be signed in to change notification settings - Fork 53
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
initial appveyor.yml for windows CI builds #533
Conversation
- PlatformToolset: mingw-w64 | ||
|
||
platform: | ||
#- x64 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
x64 build could be enabled by uncommenting this
|
||
configuration: | ||
- Release | ||
#- Debug |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
debug builds could be enabled by uncommenting this
@@ -62,12 +62,12 @@ ifdef WIN32_RELOCATABLE | |||
endif | |||
|
|||
# cygwin: built native w32 code | |||
NOCYGWIN=-mno-cygwin | |||
#NOCYGWIN=-mno-cygwin |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this commented out?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See https://ci.appveyor.com/project/chcg/aspell/builds/27190547
mingw32-make : g++: error: unrecognized command line option '-mno-cygwin'; did you mean '-mno-clwb'?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See https://www.boost.org/doc/libs/1_31_0/tools/build/v1/gcc-nocygwin-tools.html
This page describes the gcc-nocygwin toolset, which builds Boost using the -mno-cygwin option of the Cygwin gcc compiler. This avoids introducing dependencies on the Cygwin Unix-emulation layer, allowing you to build Windows executables which are not dependant on cygwin1.dll. If you're not worried about having the dependencies, you should probably be using the plain gcc toolset.
The other option for producing windows-native executables with gcc is the mingw version of gcc with the mingw toolset. However, if you're already using Cygwin, this toolset could save you having to install an additional compiler.
, so this seems to be a cygwin specific compiler option, not available with mingw gcc
@@ -0,0 +1,216 @@ | |||
/*File generated during static filter build | |||
Automatically generated file | |||
*/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this file checked in? It should be generated automatically.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See above:
- perl script mk-static-filter.pl to create static_filters.src.cpp still has a yet unresolved problem therefore win32\static_filters.src.cpp was added
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be better to resolve the issues, this file is already outdated with the new markdown filter. Please file a separate issue for this with a patch to fix the generated file and I can likely fix the script if required.
@kevina See https://ci.appveyor.com/project/chcg/aspell/builds/28107265 build from https://github.com/chcg/aspell/commits/appveyor2 with empty dummy file win32/gen/static_filters.src.cpp, because git doesn't add empty directories. Maybe you could take a look how to generate static_filters.src.cpp at the same gen dir like used for the normal build. |
, see https://ci.appveyor.com/project/chcg/aspell/build/0.60.6.1.37
Mingw dlls are dynamically linked, so there exists some additional dependencies. This could be reduced to just libgcc_s_xxx.dll by replacing
LIBS=$(GCC2LIBS)
with
LIBS= -static-libstdc++ -Wl,-Bstatic -lstdc++ -lpthread -Wl,-Bdynamic $(GCC2LIBS)