Skip to content

Travis is failing during setup #776

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

Closed
elliotekj opened this issue Jun 14, 2017 · 4 comments · Fixed by #777
Closed

Travis is failing during setup #776

elliotekj opened this issue Jun 14, 2017 · 4 comments · Fixed by #777

Comments

@elliotekj
Copy link
Contributor

elliotekj commented Jun 14, 2017

Travis seems to failing whilst setting up diesel_cli:

screen shot 2017-06-14 at 12 53 30

I'm seeing this in #775 but @Turbo87 is seeing it too in #772.

At the time of writing this, #772 is 13 hours old. The last passing test looks to be this commit from #771 20 hours ago.

cc'ing @sgrif because Diesel :)

@carols10cents
Copy link
Member

Ah, we added cache: cargo: true. We should probably add --force to always get the latest version of diesel, as the error message suggests. Can you send me a PR? Otherwise I can get to this in a few hours :)

@elliotekj
Copy link
Contributor Author

Good call @carols10cents — adding --force fixed the diesel_cli issue. Unfortunately Travis failed with a similar error when installing rustfmt.

screen shot 2017-06-14 at 16 09 43

If --force is going to be needed on both of the cargo installs then reverting 209dab0#diff-354f30a63fb0907d4ad57269548329e3 seems like the best course of action because we won't be getting any benefit from it anyway (unless I'm missing something?).

@carols10cents
Copy link
Member

I think it'll cache all of our other dependencies, which should help overall time of the travis builds. I think adding --force to rustfmt too is fine!

@elliotekj
Copy link
Contributor Author

Right'o, I'll make the change :)

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 a pull request may close this issue.

2 participants