Skip to content

Conversation

@tgross35
Copy link
Contributor

These were all removed in main in e425743 ("Remove mach-related items") but don't currently have notices. Fix that here.

I am not sure whether the _NS constant should be deprecated since it doesn't appear to be in mach.

@rustbot
Copy link
Collaborator

rustbot commented Nov 18, 2024

r? @JohnTitor

rustbot has assigned @JohnTitor.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

These were all removed in `main` in e425743 ("Remove mach-related
items") but don't currently have notices. Fix that here.

I am not sure whether the _NS constant should be deprecated since it
doesn't appear to be in `mach`.
@tgross35 tgross35 changed the title Update mach-related deprecation notices [0.2] Update mach-related deprecation notices Nov 18, 2024
@tgross35 tgross35 added this pull request to the merge queue Nov 18, 2024
Merged via the queue into rust-lang:libc-0.2 with commit 6a9512f Nov 18, 2024
44 checks passed
@tgross35 tgross35 deleted the mach-deprecated branch November 18, 2024 18:39
@devsnek
Copy link

devsnek commented Dec 18, 2024

@tgross35 @JohnTitor some of these methods are not implemented in the mach2 crate yet, they should not have been deprecated here.

@JohnTitor
Copy link
Member

Then, we have to declare them on the mach2 crate instead, feel free to send a PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants