Skip to content

Conversation

Mark-Simulacrum
Copy link
Member

@Mark-Simulacrum Mark-Simulacrum commented Jun 17, 2021

This is the follow up to master/beta promotion, as well as the first round of backports:

r? @Mark-Simulacrum

@rust-highfive
Copy link
Contributor

⚠️ 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!

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

@bors r+ rollup=never p=10

Would like to get this merged so we can do master bootstrap bump today/tomorrow as well.

@bors
Copy link
Collaborator

bors commented Jun 17, 2021

📌 Commit 0f197bc 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 Jun 17, 2021
@rust-log-analyzer

This comment has been minimized.

@Mark-Simulacrum
Copy link
Member Author

Hm, something about the build environment is causing trouble... maybe refs/remotes/origin/master doesn't exist? will need to investigate

@Mark-Simulacrum
Copy link
Member Author

@bors r-

@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-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Jun 17, 2021
@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@cuviper
Copy link
Member

cuviper commented Jun 18, 2021

FWIW, I can run that docker image here and it gets the beta rev just fine. More ideas to look at in CI:

  • pwd / std::env::current_dir()
  • git rev-parse --git-dir
  • git rev-parse --show-toplevel
  • git branch --remote

@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

Mark-Simulacrum and others added 4 commits June 19, 2021 10:27
Forwarding `clone_from` to the inner value changes the observable
behavior, as previously the inner value would *not* be dropped by the
default implementation.
@rust-log-analyzer

This comment has been minimized.

@bors
Copy link
Collaborator

bors commented Jun 23, 2021

💔 Test failed - checks-actions

@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 Jun 23, 2021
@Mark-Simulacrum
Copy link
Member Author

@bors retry hopefully spurious network

@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 Jun 23, 2021
@bors
Copy link
Collaborator

bors commented Jun 23, 2021

⌛ Testing commit b36bc9e with merge d026c35c1180e68dc6eab40596cf217713537e65...

@Mark-Simulacrum
Copy link
Member Author

@bors r+

@bors
Copy link
Collaborator

bors commented Jun 23, 2021

📌 Commit 680c6ce has been approved by Mark-Simulacrum

@bors
Copy link
Collaborator

bors commented Jun 23, 2021

⌛ Testing commit 680c6ce with merge 00b8c0493985802d78227250e90ca013e143dfef...

@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@Mark-Simulacrum
Copy link
Member Author

@bors r+

@bors
Copy link
Collaborator

bors commented Jun 23, 2021

📌 Commit c4aeccd has been approved by Mark-Simulacrum

@bors
Copy link
Collaborator

bors commented Jun 23, 2021

⌛ Testing commit c4aeccd with merge bf62f4d...

@bors
Copy link
Collaborator

bors commented Jun 23, 2021

☀️ Test successful - checks-actions
Approved by: Mark-Simulacrum
Pushing bf62f4d to beta...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

9 participants