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

[beta] ci: Fix broken link in build-powerpc64le-toolchain.sh #45744

Merged
merged 1 commit into from
Nov 5, 2017

Conversation

kennytm
Copy link
Member

@kennytm kennytm commented Nov 3, 2017

This is almost a backport of #45734, but uses vault.centos.org instead for better longevity.

r? @alexcrichton cc @cuviper

The previous base URL at mirror.centos.org no longer provides packages for
7.3.1611. It has been moved to vault.centos.org.
@rust-highfive
Copy link
Collaborator

warning Warning warning

  • Pull requests are usually filed against the master branch for this repo, but this one is against beta. Please double check that you specified the right target!

@kennytm kennytm added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 3, 2017
@alexcrichton
Copy link
Member

@bors: r+

Looks and sounds great to me, thanks so much for handling this @kennytm!

@bors
Copy link
Contributor

bors commented Nov 3, 2017

📌 Commit f065bc5 has been approved by alexcrichton

@kennytm kennytm 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 Nov 4, 2017
@bors
Copy link
Contributor

bors commented Nov 5, 2017

⌛ Testing commit f065bc5 with merge 0f18b88...

bors added a commit that referenced this pull request Nov 5, 2017
[beta] ci: Fix broken link in `build-powerpc64le-toolchain.sh`

This is almost a backport of #45734, but uses vault.centos.org instead for better longevity.

r? @alexcrichton cc @cuviper
@bors
Copy link
Contributor

bors commented Nov 5, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 0f18b88 to beta...

@bors bors merged commit f065bc5 into rust-lang:beta Nov 5, 2017
@kennytm kennytm deleted the centos-vault-beta branch November 5, 2017 06:45
kennytm added a commit to kennytm/rust that referenced this pull request Nov 5, 2017
This commit reverts rust-lang#45734 and applies rust-lang#45744. We expect the vault links
to be more stable than mirror.centos.org.
kennytm added a commit to kennytm/rust that referenced this pull request Nov 10, 2017
…crichton

Miscellaneous changes for CI, Docker and compiletest.

This PR contains 7 independent commits that improves interaction with CI, Docker and compiletest.

1. a4e5c91 — Forces a newline every 100 dots when testing in quiet mode. Prevents spurious timeouts when abusing the CI to test Android jobs.

2. 1b5aaf2 — Use vault.centos.org for dist-powerpc64le-linux, see rust-lang#45744.

3. 33400fb — Modify `src/ci/docker/run.sh` so that the docker images can be run from Docker Toolbox for Windows on Windows 7. I haven't checked the behavior of the newer Docker for Windows on Windows 10. Also, "can run" does not mean all the test can pass successfully (the UDP tests failed last time I checked)

4. d517668 — Don't emit a real warning the linker segfault, which affects UI tests like rust-lang#45489 (comment). Log it instead.

5. 51e2247 — During run-pass, trim the output if stdout/stderr exceeds 416 KB (top 160 KB + bottom 256 KB). This is an attempt to avoid spurious failures like rust-lang#45384 (comment)

6. 9cfdaba — Force `gem update --system` before deploy. This is an attempt to prevent spurious error rust-lang#44159.

7. eee10cc — Tries to print the crash log on macOS on failure. This is an attempt to debug rust-lang#45230.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants