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

Re-add PTRACE_DETACH for mips64 GNU targets #1393

Merged
merged 1 commit into from
Jun 7, 2019

Conversation

Susurrus
Copy link
Contributor

@Susurrus Susurrus commented Jun 7, 2019

During the refactor in 4bd419e, PTRACE_DETACH was accidentally removed on
these targets. This re-adds it.

Fixes brokenness encountered during nix-rust/nix#1035

CC @asomers

During the refactor in 4bd419e, PTRACE_DETACH was accidentally removed on
these targets. This re-adds it.
@rust-highfive
Copy link

r? @gnzlbg

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

@gnzlbg
Copy link
Contributor

gnzlbg commented Jun 7, 2019

@bors: r+

Thanks!

@bors
Copy link
Contributor

bors commented Jun 7, 2019

📌 Commit 1f27f5d has been approved by gnzlbg

bors added a commit that referenced this pull request Jun 7, 2019
Re-add PTRACE_DETACH for mips64 GNU targets

During the refactor in 4bd419e, `PTRACE_DETACH` was accidentally removed on
these targets. This re-adds it.

Fixes brokenness encountered during nix-rust/nix#1035

CC @asomers
@bors
Copy link
Contributor

bors commented Jun 7, 2019

⌛ Testing commit 1f27f5d with merge d5a599e...

@bors
Copy link
Contributor

bors commented Jun 7, 2019

💔 Test failed - checks-cirrus-freebsd-12

@bors
Copy link
Contributor

bors commented Jun 7, 2019

☀️ Test successful - checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-travis, status-appveyor
Approved by: gnzlbg
Pushing d5a599e to master...

@bors bors merged commit 1f27f5d into rust-lang:master Jun 7, 2019
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.

None yet

4 participants