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

Gate PartialEq and Eq on freebsd objects behind extra_traits #2659

Merged
merged 2 commits into from
Jan 28, 2022

Conversation

tmandry
Copy link
Member

@tmandry tmandry commented Jan 28, 2022

This fixes the failure in rust-lang/rust#93351 (comment).

These derives were recently added in #2565. Other PartialEq/Eq derives in the project (and this file) are all behind the extra_traits gate.

@rust-highfive
Copy link

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @Amanieu (or someone else) soon.

Please see the contribution instructions for more information.

@Amanieu
Copy link
Member

Amanieu commented Jan 28, 2022

@bors r+

@bors
Copy link
Contributor

bors commented Jan 28, 2022

📌 Commit e5f0e5f has been approved by Amanieu

@bors
Copy link
Contributor

bors commented Jan 28, 2022

⌛ Testing commit e5f0e5f with merge 2dc7cfe...

@bors
Copy link
Contributor

bors commented Jan 28, 2022

☀️ Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: Amanieu
Pushing 2dc7cfe to master...

@bors bors merged commit 2dc7cfe into rust-lang:master Jan 28, 2022
@tmandry tmandry deleted the fix-freebsd-std branch February 3, 2022 22:29
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