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

build: avoid timing out in Travis #4773

Merged
merged 3 commits into from
Nov 5, 2019
Merged

build: avoid timing out in Travis #4773

merged 3 commits into from
Nov 5, 2019

Conversation

tesuji
Copy link
Contributor

@tesuji tesuji commented Nov 5, 2019 •
edited
Loading

cc #4770
changelog: none

@tesuji
Copy link
Contributor Author

tesuji commented Nov 5, 2019 •
edited
Loading

Succeed: https://travis-ci.com/rust-lang/rust-clippy/jobs/253075203

This should close fix (wrong word) #4770 . cc @Manishearth

@Manishearth
Copy link
Member

Can you include the other PR that disables the stdsimd test? We can't land this without both

@tesuji
Copy link
Contributor Author

tesuji commented Nov 5, 2019

Done.

@Manishearth
Copy link
Member

@bors r=yaah, manishearth

@bors
Copy link
Collaborator

bors commented Nov 5, 2019

📌 Commit b8c2cb1 has been approved by yaah,

@Manishearth Manishearth mentioned this pull request Nov 5, 2019
Closed
bors added a commit that referenced this pull request Nov 5, 2019
@bors
Auto merge of #4773 - lzutao:rls/print-something, r=yaah,
de980f8 
build: avoid timing out in Travis

cc #4770
changelog: none
@bors
Copy link
Collaborator

bors commented Nov 5, 2019

⌛ Testing commit b8c2cb1 with merge de980f8...

@tesuji
Copy link
Contributor Author

tesuji commented Nov 5, 2019

The build didn't start
@bors r-

@tesuji
Copy link
Contributor Author

tesuji commented Nov 5, 2019

Hmm, it's building now: https://travis-ci.com/rust-lang/rust-clippy/builds/135085299

@tesuji
Copy link
Contributor Author

tesuji commented Nov 5, 2019

network error
@bors retry

@bors
Copy link
Collaborator

bors commented Nov 5, 2019

@lzutao: 🔑 Insufficient privileges: not in try users

@Manishearth Manishearth closed this Nov 5, 2019
@Manishearth Manishearth reopened this Nov 5, 2019
@Manishearth
Copy link
Member

@bors r+ retry

@bors
Copy link
Collaborator

bors commented Nov 5, 2019

📌 Commit b8c2cb1 has been approved by Manishearth

@bors
Copy link
Collaborator

bors commented Nov 5, 2019

⌛ Testing commit b8c2cb1 with merge d1e0490...

bors added a commit that referenced this pull request Nov 5, 2019
@bors
Auto merge of #4773 - lzutao:rls/print-something, r=Manishearth
d1e0490 
build: avoid timing out in Travis

cc #4770
changelog: none
@bors
Copy link
Collaborator

bors commented Nov 5, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: Manishearth
Pushing d1e0490 to master...

@bors bors merged commit b8c2cb1 into rust-lang:master Nov 5, 2019
@tesuji tesuji deleted the rls/print-something branch November 6, 2019 02:50
@ghost ghost mentioned this pull request Nov 6, 2019
Closed
@ecstatic-morse
Copy link
Contributor

ecstatic-morse commented Nov 7, 2019 •
edited
Loading

@lzutao @Manishearth rust-lang/rust#66016 should now be fixed on the latest nightly.

@mati865
Copy link
Contributor

mati865 commented Nov 7, 2019

@ecstatic-morse indeed, stdsimd test was re-enabled by #4777.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@tesuji @Manishearth @bors @ecstatic-morse @mati865