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.

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

Download crosstool-ng from GitHub #46715

Merged
merged 1 commit into from
Dec 13, 2017

Conversation

kennytm
Copy link
Member

@kennytm kennytm commented Dec 13, 2017

Workaround the current problem where http://crosstool-ng.org was done, causing all non-x86 jobs to fail spuriously (cc #40474).

If http://crosstool-ng.org becomes online before this PR is merged, this PR should be closed and the tree should be reopened.

@kennytm kennytm added the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Dec 13, 2017
@kennytm kennytm force-pushed the download-crosstool-ng-from-github branch 2 times, most recently from fa7793a to 703c1b5 Compare December 13, 2017 20:07
@kennytm kennytm changed the title [WIP] Download crosstool-ng from GitHub Download crosstool-ng from GitHub Dec 13, 2017
@kennytm
Copy link
Member Author

kennytm commented Dec 13, 2017

Green! Meanwhile, crosstool-ng.org is still offline.

r? @rust-lang/infra

@kennytm kennytm added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Dec 13, 2017
@TimNN
Copy link
Contributor

TimNN commented Dec 13, 2017

@bors r+ p=10

@bors
Copy link
Contributor

bors commented Dec 13, 2017

📌 Commit 703c1b5 has been approved by TimNN

@TimNN
Copy link
Contributor

TimNN commented Dec 13, 2017

@bors p=30

@kennytm
Copy link
Member Author

kennytm commented Dec 13, 2017

@bors treeclosed-

@bors
Copy link
Contributor

bors commented Dec 13, 2017

⌛ Testing commit 703c1b5cf620e31b12d12d17cc6bfe098df5cd4d with merge c5b06063e7ddf60c9b3b07959d1b589f6649fe18...

@bors
Copy link
Contributor

bors commented Dec 13, 2017

💔 Test failed - status-travis

@kennytm
Copy link
Member Author

kennytm commented Dec 13, 2017

@bors treeclosed=2

Temporary workaround since crosstool-ng.org was down. Consider mirroring
the release tarball as a more permanent solution.
@kennytm kennytm force-pushed the download-crosstool-ng-from-github branch from 703c1b5 to bf0653e Compare December 13, 2017 20:50
@kennytm
Copy link
Member Author

kennytm commented Dec 13, 2017

@bors r=TimNN treeclosed-

Change the crosstool version back to 1.22. Maybe 1.23 is not compatible with powerpc yet.

@bors
Copy link
Contributor

bors commented Dec 13, 2017

📌 Commit bf0653e has been approved by TimNN

@bors
Copy link
Contributor

bors commented Dec 13, 2017

⌛ Testing commit bf0653e with merge f8af59d...

bors added a commit that referenced this pull request Dec 13, 2017
…imNN

Download crosstool-ng from GitHub

Workaround the current problem where http://crosstool-ng.org was done, causing all non-x86 jobs to fail spuriously (cc #40474).

If http://crosstool-ng.org becomes online before this PR is merged, this PR should be closed and the tree should be reopened.
@bors
Copy link
Contributor

bors commented Dec 13, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: TimNN
Pushing f8af59d to master...

@bors bors merged commit bf0653e into rust-lang:master Dec 13, 2017
@alexcrichton
Copy link
Member

Thanks for fixing this @kennytm!

@kennytm kennytm deleted the download-crosstool-ng-from-github branch December 14, 2017 05:14
@kennytm kennytm mentioned this pull request Dec 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants