Add MinGW and MinGW-w64 builds to AppVeyor (take 2) #48
Conversation
traversaro
changed the title
.appveyor.yml
Outdated
| @@ -138,6 +138,7 @@ before_build: | |||
| - set PATH=%PATH:C:\Program Files\Git\usr\bin;=% | |||
| - set PATH=C:\MinGW\bin;%PATH% | |||
| - if /I "%GENERATOR%" == "MinGW Makefiles" (set "GENERATOR_ARGS=") else (set "GENERATOR_ARGS=-A %PLATFORM%") | |||
| - if /I "%GENERATOR%" == "MinGW Makefiles" (set "INSTALL_TARGET=install") else (set "INSTALL_TARGET=INSTALL") | |||
There was a problem hiding this comment.
Are you sure that lowercase install does not work with MSVC?
There was a problem hiding this comment.
lowercase install is working with MSVC, see: https://ci.appveyor.com/project/dlfcn-win32/dlfcn-win32/builds/22271048/job/p12xqd0frrr7oq0q#L159
There was a problem hiding this comment.
Fixed in 56c6cbc .
.appveyor.yml
Outdated
| @@ -134,10 +134,13 @@ matrix: | |||
| CMAKE_PREFIX_PATH: "C:\\mingw-w64\\x86_64-7.2.0-posix-seh-rt_v5-rev1" | |||
|
|
|||
| before_build: | |||
| - set Path=C:\\MinGW\\bin;%Path% | |||
| # See https://help.appveyor.com/discussions/problems/3193-cmake-building-for-mingw-issue-with-git-shexe | |||
| - set PATH=%PATH:C:\Program Files (x86)\Git\bin;=% | |||
There was a problem hiding this comment.
Looks like this is more elegant solution as my original which renamed sh.exe.
Similar problem I had with building C module for Perl as on AppVeyor are 3 different perl versions and 2 different MinGW versions which Perl can detect. And I just renamed binaries...:
perl5-dbi/DBD-MariaDB@97d28bb#diff-180360612c6b8c4ed830919bbb4dd459R36
.appveyor.yml
Outdated
| BUILD_SHARED_LIBS: OFF | ||
|
|
||
| - APPVEYOR_BUILD_WORKER_IMAGE: Visual Studio 2015 | ||
| GENERATOR: "MinGW Makefiles" | ||
| CMAKE_PREFIX_PATH: "C:\\mingw-w64\\i686-5.3.0-posix-dwarf-rt_v4-rev0" | ||
| cc_path: "C:\mingw-w64\i686-5.3.0-posix-dwarf-rt_v4-rev0\mingw32\bin" |
There was a problem hiding this comment.
This is yaml string, so backslash in double quotes needs to be escaped... "C:\\mingw-"
There was a problem hiding this comment.
This was fixed in the later commit: 6fda813 .
|
@pali If AppVeyor is happy and you approve the PR, I think I can squash it an merge. |
traversaro
force-pushed
the
appveyor-mingw-take-two
branch
from
56c6cbc
to
23d7753
Compare
Based on #46 .