Skip to content

Conversation

@rnestler
Copy link
Member

@rnestler rnestler commented Jul 8, 2015

It's probably a good thing if the spaceapi builds with the latest stable
toolchain, not only with beta / nightly.
Also I removed nightly because recently we hit a compiler bug: https://travis-ci.org/coredump-ch/spaceapi/builds/69862138, see rust-lang/rust#26807 which fixed the error.
But maybe that's a good thing, so we uncover bugs in rustc.

rnestler and others added 2 commits July 8, 2015 21:45
It's probably a good thing if the spaceapi builds with the latest stable
toolchain, not only with beta and nightly.
@schmijos
Copy link
Member

schmijos commented Jul 9, 2015

Do you know, why the build is failing?

@rnestler
Copy link
Member Author

rnestler commented Jul 9, 2015

Have you looked at it? It fails to download the toolchain and timeouts after 10 minutes.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this only allow dead code for tests, or for every build?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think only for tests

@dbrgn
Copy link
Member

dbrgn commented Jul 21, 2015

I totally agree about building on stable and beta :) Back when we introduced testing, stable was not yet available.

dbrgn added a commit that referenced this pull request Jul 22, 2015
travis: test with stable and beta, but not nightly
@dbrgn dbrgn merged commit cdd6f04 into rust Jul 22, 2015
@dbrgn dbrgn deleted the test-on-stable branch July 22, 2015 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants