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.

Sign up for GitHub

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

Jump to bottom

Migrate from AppVeyor to Travis CI Windows? #6346

Closed
dwijnand opened this issue Nov 25, 2018 · 7 comments
Closed

Migrate from AppVeyor to Travis CI Windows? #6346

dwijnand opened this issue Nov 25, 2018 · 7 comments
Labels
A-building-cargo-itself Area: issues with building cargo A-testing-cargo-itself Area: cargo's tests

Comments

@dwijnand
Copy link
Member

With Travis CI adding support for Windows (https://blog.travis-ci.com/2018-10-11-windows-early-release) should we transition from AppVeyor to Travis CI for Windows testing too?

Any feels for or against?
@dwijnand dwijnand added A-building-cargo-itself Area: issues with building cargo A-testing-cargo-itself Area: cargo's tests labels Nov 25, 2018
@dwijnand
Copy link
Member Author

One benefit is I seem to be able to restart Travis CI jobs, but not AppVeyor jobs.

@Arzte
Copy link

Arzte commented Nov 25, 2018

An argument for it is that it would mean everything would be in one place, all the builds together.

@alexcrichton
Copy link
Member

This seems like it's worthwhile to test for! The main reason for us I believe is that we have more capacity on Travis than AppVeyor so builds may finish more quickly. Should be fine to test out a PR regardless!

@Eh2406
Copy link
Contributor

Eh2406 commented Nov 26, 2018

People are reporting that it is nearly unusable slow do to rust-lang/rustup#998 (comment)
It is worth a shot, but I am not hopefully until that is fixed.

@dwijnand
Copy link
Member Author

Hmm. OK, then let's park this for now.

@thedavidmeister thedavidmeister mentioned this issue Dec 17, 2018
Open
@dwijnand dwijnand mentioned this issue Jan 30, 2019
Open
@dwijnand
Copy link
Member Author

Once rust-lang/rustup#998 is fixed, resolving this might allow resolving downstream issues, like #4495.

@slonopotamus
Copy link

rust-lang/rustup#998 is now fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-building-cargo-itself Area: issues with building cargo A-testing-cargo-itself Area: cargo's tests
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@alexcrichton @slonopotamus @dwijnand @Eh2406 @Arzte