Skip to content

use RWlock when accessing os::env (take 2) - #83121

Merged
bors merged 1 commit into
rust-lang:masterfrom
the8472:env-rwlock-2
Mar 15, 2021
Merged

use RWlock when accessing os::env (take 2)#83121
bors merged 1 commit into
rust-lang:masterfrom
the8472:env-rwlock-2

Conversation

@the8472

Copy link
Copy Markdown
Member

This reverts commit acdca31 (#82877) i.e. redoes #81850 since the invalid unlock attempts in the child process have been fixed in #82949

r? @joshtriplett

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 14, 2021
@joshtriplett

Copy link
Copy Markdown
Member

@bors r+ rollup=never

@bors

bors commented Mar 15, 2021

Copy link
Copy Markdown
Collaborator

📌 Commit e22143c has been approved by joshtriplett

@borsbors 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 15, 2021
@bors

bors commented Mar 15, 2021

Copy link
Copy Markdown
Collaborator

⌛ Testing commit e22143c with merge 107896c...

@bors

bors commented Mar 15, 2021

Copy link
Copy Markdown
Collaborator

☀️ Test successful - checks-actions
Approved by: joshtriplett
Pushing 107896c to master...

@borsbors added the merged-by-bors This PR was explicitly merged by bors. label Mar 15, 2021
@bors
bors merged commit 107896c into rust-lang:masterMar 15, 2021
@rustbotrustbot added this to the 1.52.0 milestone Mar 15, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merged-by-borsThis PR was explicitly merged by bors.S-waiting-on-borsStatus: 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.

5 participants

@the8472@joshtriplett@bors@rust-highfive@rustbot