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

Convert race to a default-enabled feature for targets without atomics. #137

Merged
merged 1 commit into from
Feb 24, 2021

Conversation

cr1901
Copy link
Contributor

@cr1901 cr1901 commented Feb 24, 2021

Closes #136.

@cr1901 cr1901 changed the title Convert race to a default-enabled feature for targets without atomics. Closes #136. Convert race to a default-enabled feature for targets without atomics. Feb 24, 2021
alloc = []
alloc = ["race"]
# Enables `once_cell::race` module.
race = []
Copy link
Owner

Choose a reason for hiding this comment

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

This should be added to default above I think?

Copy link
Contributor Author

@cr1901 cr1901 Feb 24, 2021

Choose a reason for hiding this comment

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

Oh, it's not transitive (std depends on alloc which depends on race)? Hmmm... anyway CI fails, so I better fix that :P.

Copy link
Owner

Choose a reason for hiding this comment

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

Ah, you are of course right!

@cr1901
Copy link
Contributor Author

cr1901 commented Feb 24, 2021

CI fails in PUBLISH... I don't think there's much I can do about that?

@matklad matklad closed this Feb 24, 2021
@matklad matklad reopened this Feb 24, 2021
@matklad
Copy link
Owner

matklad commented Feb 24, 2021

(re-triggering the CI, the error should be fixed by matklad/xaction@c297844)

@matklad
Copy link
Owner

matklad commented Feb 24, 2021

bors r+

@bors
Copy link
Contributor

bors bot commented Feb 24, 2021

Build succeeded:

@bors bors bot merged commit da01f25 into matklad:master Feb 24, 2021
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.

race module cannot be used on targets without atomics
2 participants