Skip to content
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

[CI] Mark MinGW build failures allowed #21243

Closed
asfimport opened this issue Feb 28, 2019 · 3 comments
Closed

[CI] Mark MinGW build failures allowed #21243

asfimport opened this issue Feb 28, 2019 · 3 comments

Comments

@asfimport
Copy link

Currently we are requiring MinGW tests to pass on AppVeyor. Almost nobody will use MinGW builds if regular MSVC builds work fine. So it should be on the onus of the few people caring about MinGW to ensure that the build chain works on those platforms.

Example here, apparently the uriparser library doesn't build on MinGW:
https://ci.appveyor.com/project/pitrou/arrow/build/job/t64xwyj2axhl1jgr

There is a tendency to inflate the number of different configurations in our CI matrices. Not only it makes builds longer and adds delays (see how long you have to wait before you get a CI result on AppVeyor), but it's of dubious utility. I'm not sure it serves the project's general interest.

Rant off ;)

Reporter: Antoine Pitrou / @pitrou

Note: This issue was originally created as ARROW-4720. Please see the migration documentation for further details.

@asfimport
Copy link
Author

Kouhei Sutou / @kou:
Ruby users use MinGW build.

R users will also use MinGW build.

@asfimport
Copy link
Author

Antoine Pitrou / @pitrou:
minGW builds seem to have been stable for a while, closing.

@asfimport
Copy link
Author

Todd Farmer / @toddfarmer:
Transitioning issue from Resolved to Closed to based on resolution field value.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant