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

Remove several deprecated constants and functions #1255

Merged
merged 2 commits into from
Jun 12, 2020

Conversation

asomers
Copy link
Member

@asomers asomers commented Jun 5, 2020

  • unistd::daemon on Apple
  • unistd::pipe2 on Apple
  • sys::event::FilterFlag::NOTE_EXIT_REPARENTED on Apple
  • sys::event::FilterFlag::NOTE_REAP on Apple
  • sys::ptrace::ptrace on Android and Linux

All have been deprecated for more than two releases and one year.

@asomers asomers mentioned this pull request Jun 5, 2020
@Susurrus
Copy link
Contributor

Susurrus commented Jun 5, 2020

Some tests still use pipe2 and fail to compile now. But fix that and I think it looks good to merge.

* `unistd::daemon` on Apple
* `unistd::pipe2` on Apple
* `sys::event::FilterFlag::NOTE_EXIT_REPARENTED` on Apple
* `sys::event::FilterFlag::NOTE_REAP` on Apple
* `sys::ptrace::ptrace` on Android and Linux

All have been deprecated for more than two releases and one year.
@asomers asomers force-pushed the remove_deprecated2 branch from f71f3b1 to f698fd3 Compare June 5, 2020 03:56
@asomers
Copy link
Member Author

asomers commented Jun 5, 2020

I fixed the cfg gate on the test, and tweaked the formatting of the man page.

Copy link
Contributor

@Susurrus Susurrus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Otherwise LGTM.

target_os = "emscripten",
target_os = "freebsd",
target_os = "linux",
target_os = "redox",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

redox should be last.

CHANGELOG.md Outdated
@@ -69,6 +69,12 @@ This project adheres to [Semantic Versioning](http://semver.org/).
optional arguments.
(#[1242](https://github.com/nix-rust/nix/pull/1242))

- Removed `unistd::daemon`, `unistd::pipe2`,
`sys::event::FilterFlag::NOTE_EXIT_REPARENTED`, and
`sys::event::FilterFlag::NOTE_REAP` on OSX and ios. Removed
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd split these as separate line items since they aren't really related even though they're part of the same PR.

@asomers
Copy link
Member Author

asomers commented Jun 12, 2020

bors r+

bors bot added a commit that referenced this pull request Jun 12, 2020
1255: Remove several deprecated constants and functions r=asomers a=asomers

* `unistd::daemon` on Apple
* `unistd::pipe2` on Apple
* `sys::event::FilterFlag::NOTE_EXIT_REPARENTED` on Apple
* `sys::event::FilterFlag::NOTE_REAP` on Apple
* `sys::ptrace::ptrace` on Android and Linux

All have been deprecated for more than two releases and one year.

Co-authored-by: Alan Somers <[email protected]>
@bors
Copy link
Contributor

bors bot commented Jun 12, 2020

Build failed:

@asomers
Copy link
Member Author

asomers commented Jun 12, 2020

bors retry

@bors
Copy link
Contributor

bors bot commented Jun 12, 2020

Build succeeded:

@bors bors bot merged commit ea099dd into nix-rust:master Jun 12, 2020
@asomers asomers deleted the remove_deprecated2 branch June 12, 2020 20:36
@khuey
Copy link
Contributor

khuey commented Aug 11, 2020

It's unfortunate that ptrace was removed without creating nix wrappers for the remaining Request types (e.g. PTRACE_GETREGSET).

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.

3 participants