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

Leak mspdbsrv.exe processes on Windows #3162

Merged
merged 1 commit into from
Oct 6, 2016

Conversation

alexcrichton
Copy link
Member

Instead of having our job object tear them down, instead leak them intentionally
if everything succeeded.

Closes #3161

Instead of having our job object tear them down, instead leak them intentionally
if everything succeeded.

Closes rust-lang#3161
@rust-highfive
Copy link

r? @brson

(rust_highfive has picked a reviewer for you, use r? to override)

@alexcrichton
Copy link
Member Author

Most of this was all taken from https://github.com/alexcrichton/rustjob which we've been using on the buildbots for quite some time now.

@alexcrichton
Copy link
Member Author

This'll also likely want to be ported wholesale to rustup

@brson
Copy link
Contributor

brson commented Oct 5, 2016

@bors r+

@bors
Copy link
Collaborator

bors commented Oct 5, 2016

📌 Commit 5ede71e has been approved by brson

@bors
Copy link
Collaborator

bors commented Oct 6, 2016

⌛ Testing commit 5ede71e with merge 505b9b0...

bors added a commit that referenced this pull request Oct 6, 2016
Leak mspdbsrv.exe processes on Windows

Instead of having our job object tear them down, instead leak them intentionally
if everything succeeded.

Closes #3161
@bors
Copy link
Collaborator

bors commented Oct 6, 2016

☀️ Test successful - cargo-cross-linux, cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64
Approved by: brson
Pushing 505b9b0 to master...

@bors bors merged commit 5ede71e into rust-lang:master Oct 6, 2016
@alexcrichton alexcrichton deleted the ugh-mspdbsrv branch October 6, 2016 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants