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

Add checking for tool distribution in Tier 1 #54638

Merged
merged 3 commits into from
Oct 4, 2018
Merged

Add checking for tool distribution in Tier 1 #54638

merged 3 commits into from
Oct 4, 2018

Conversation

pvdrz
Copy link
Contributor

@pvdrz pvdrz commented Sep 28, 2018

This fixes #54483
r? @kennytm

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 28, 2018
@kennytm
Copy link
Member

kennytm commented Sep 28, 2018

@bors try

@bors
Copy link
Contributor

bors commented Sep 28, 2018

⌛ Trying commit 0724ed6 with merge f033c42307636d4b7844ae977a3442501c4d9d32...

@bors
Copy link
Contributor

bors commented Sep 28, 2018

💔 Test failed - status-travis

@rust-highfive

This comment has been minimized.

Sign in to view
@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Sep 28, 2018
@alexcrichton
Copy link
Member

@bors: retry

@bors
Copy link
Contributor

bors commented Sep 28, 2018

⌛ Trying commit 0724ed6 with merge 1a89f39...

bors added a commit that referenced this pull request Sep 28, 2018
@bors
Auto merge of #54638 - christianpoveda:master, r=<try>
1a89f39 
Add checking for tool distribution in Tier 1

This fixes #54483
r? @kennytm
@bors
Copy link
Contributor

bors commented Sep 28, 2018

💔 Test failed - status-travis

@rust-highfive

This comment has been minimized.

Sign in to view
@kennytm
Copy link
Member

kennytm commented Sep 29, 2018

Seems legit

[00:03:42] /checkout/src/ci/run.sh: line 79: [: too many arguments

@kennytm
Copy link
Member

kennytm commented Sep 30, 2018 •
edited
Loading

@bors try

@bors
Copy link
Contributor

bors commented Sep 30, 2018

⌛ Trying commit d8b428cd77213fdce6d0d51ea149114674c85a22 with merge e2f749de3e4f8ff5b3fdc9c5c738f65b5df1d80a...

@bors
Copy link
Contributor

bors commented Sep 30, 2018

💔 Test failed - status-travis

@rust-highfive

This comment has been minimized.

Sign in to view
@rust-highfive

This comment has been minimized.

Sign in to view
@bors
Copy link
Contributor

bors commented Sep 30, 2018

⌛ Trying commit 502a3659750cd5e2bf8e5729071cd6d0d6725d45 with merge d493e9a851d56506bfaf31ac4575eea9dcdd8bee...

@bors
Copy link
Contributor

bors commented Sep 30, 2018

💔 Test failed - status-travis

@rust-highfive

This comment has been minimized.

Sign in to view
@rust-highfive

This comment has been minimized.

Sign in to view
@rust-highfive

This comment has been minimized.

Sign in to view
@rust-highfive

This comment has been minimized.

Sign in to view
@kennytm
Copy link
Member

kennytm commented Oct 2, 2018

@bors retry

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 2, 2018
@alexcrichton alexcrichton mentioned this pull request Oct 2, 2018
Closed
@bors
Copy link
Contributor

bors commented Oct 3, 2018

⌛ Testing commit 2765575 with merge add7b316e99e5f2d6e7c1843e27e078dcbb7079d...

@bors
Copy link
Contributor

bors commented Oct 3, 2018

💔 Test failed - status-appveyor

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Oct 3, 2018
@kennytm
Copy link
Member

kennytm commented Oct 3, 2018

@bors retry

3 hour timeout

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 3, 2018
kennytm added a commit to kennytm/rust that referenced this pull request Oct 3, 2018
@kennytm
Rollup merge of rust-lang#54638 - christianpoveda:master, r=kennytm
72aad24 
Add checking for tool distribution in Tier 1

This fixes rust-lang#54483
r? @kennytm
@kennytm kennytm mentioned this pull request Oct 3, 2018
Closed
@bors
Copy link
Contributor

bors commented Oct 4, 2018

⌛ Testing commit 2765575 with merge 11f9e67dbdc194e09c02df7fbfc3ef3b33f94b32...

@bors
Copy link
Contributor

bors commented Oct 4, 2018

💔 Test failed - status-travis

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Oct 4, 2018
@rust-highfive
Copy link
Collaborator

The job dist-mipsel-linux of your PR failed on Travis (raw log). Through arcane magic we have determined that the following fragments from the build log may contain information about the problem.

Click to expand the log. 
travis_time:end:060dd7f2:start=1538635662725765967,finish=1538635662733977988,duration=8212021
travis_fold:end:after_failure.3
travis_fold:start:after_failure.4
travis_time:start:00070db8
$ ln -s . checkout && for CORE in obj/cores/core.*; do EXE=$(echo $CORE | sed 's|obj/cores/core\.[0-9]*\.!checkout!\(.*\)|\1|;y|!|/|'); if [ -f "$EXE" ]; then printf travis_fold":start:crashlog\n\033[31;1m%s\033[0m\n" "$CORE"; gdb --batch -q -c "$CORE" "$EXE" -iex 'set auto-load off' -iex 'dir src/' -iex 'set sysroot .' -ex bt -ex q; echo travis_fold":"end:crashlog; fi; done || true
travis_fold:end:after_failure.4
travis_fold:start:after_failure.5
travis_time:start:269045d2
travis_time:start:269045d2
$ cat ./obj/build/x86_64-unknown-linux-gnu/native/asan/build/lib/asan/clang_rt.asan-dynamic-i386.vers || true
cat: ./obj/build/x86_64-unknown-linux-gnu/native/asan/build/lib/asan/clang_rt.asan-dynamic-i386.vers: No such file or directory
travis_fold:end:after_failure.5
travis_fold:start:after_failure.6
travis_time:start:0a9ca44e
$ dmesg | grep -i kill

I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact @TimNN. (Feature Requests)

@kennytm
Copy link
Member

kennytm commented Oct 4, 2018

@bors retry travis-ci/travis-ci#9696

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 4, 2018
@bors
Copy link
Contributor

bors commented Oct 4, 2018

⌛ Testing commit 2765575 with merge 8a0e5cb...

bors added a commit that referenced this pull request Oct 4, 2018
@bors
Auto merge of #54638 - christianpoveda:master, r=kennytm
8a0e5cb 
Add checking for tool distribution in Tier 1

This fixes #54483
r? @kennytm
@bors
Copy link
Contributor

bors commented Oct 4, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: kennytm
Pushing 8a0e5cb to master...

@bors bors merged commit 2765575 into rust-lang:master Oct 4, 2018
@pnkfelix pnkfelix mentioned this pull request Nov 9, 2018
Closed
@spastorino spastorino mentioned this pull request Jun 10, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kennytm kennytm kennytm left review comments

Assignees

@kennytm kennytm

Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Verify that the tools are actually distributed on beta/stable channels
5 participants
@pvdrz @kennytm @bors @rust-highfive @alexcrichton