Skip to content

Conversation

ehuss
Copy link
Contributor

@ehuss ehuss commented Mar 17, 2022

This is an attempt to fix the cmake build. For some reason, it has recently started failing with a permission denied trying to overwrite /tmp/build.log. This file exists from the build-toolchains.sh step, which is owned by the rustbuild user. I think there is some behavior where a sticky /tmp directory doesn't allow overwriting files owned by other users even when running as root. I do not know why this has suddenly started, and I can't reproduce locally with my own docker setup. However, this fix seems to work on CI.

@rust-highfive
Copy link
Contributor

r? @Mark-Simulacrum

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 17, 2022
@Mark-Simulacrum
Copy link
Member

@bors r+ p=101

Seems at least harmless, and hopefully fixes things.

@bors
Copy link
Collaborator

bors commented Mar 17, 2022

📌 Commit 8322cdb has been approved by Mark-Simulacrum

@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 Mar 17, 2022
@bors
Copy link
Collaborator

bors commented Mar 17, 2022

⌛ Testing commit 8322cdb with merge 58f1179...

@bors
Copy link
Collaborator

bors commented Mar 17, 2022

☀️ Test successful - checks-actions
Approved by: Mark-Simulacrum
Pushing 58f1179 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Mar 17, 2022
@bors bors merged commit 58f1179 into rust-lang:master Mar 17, 2022
@rustbot rustbot added this to the 1.61.0 milestone Mar 17, 2022
@Dylan-DPC
Copy link
Member

@bors treeclosed-

@rust-timer
Copy link
Collaborator

Finished benchmarking commit (58f1179): comparison url.

Summary: This benchmark run did not return any relevant results.

If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf.

@rustbot label: -perf-regression

@Mark-Simulacrum Mark-Simulacrum added beta-nominated Nominated for backporting to the compiler in the beta channel. beta-accepted Accepted for backporting to the compiler in the beta channel. labels Mar 18, 2022
@cuviper cuviper modified the milestones: 1.61.0, 1.60.0 Mar 18, 2022
@cuviper cuviper removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label Mar 18, 2022
bors added a commit to rust-lang-ci/rust that referenced this pull request Mar 18, 2022
[beta] backports + bootstrap bump

Add pending backports:

* Revert accidental stabilization rust-lang#94805
* Do not recover from Ty? in macro parsing rust-lang#94593
* Fix cmake build. rust-lang#95050

And updates the bootstrap compiler to 1.59.0.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beta-accepted Accepted for backporting to the compiler in the beta channel. merged-by-bors This PR was explicitly merged by bors. 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.

8 participants