-
Notifications
You must be signed in to change notification settings - Fork 58
Use portable atomic, remove explicit/redundant feature declaration & move to edition 2021 #103
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
base: main
Are you sure you want to change the base?
Conversation
75046d1 to
068dae0
Compare
068dae0 to
6b5ac37
Compare
6b5ac37 to
54dfe17
Compare
|
Please merge this pull request to enable use for AVR and other archs. It externalizes the management of missing atomics to specialized crate (portable-atomic). |
|
I would keep the @jamesmunns: Is |
|
@Sympatron I'm not actively maintaining it at the moment, I've added @ithinuel as a maintainer, but I think they are busy as well. If you'd like to be added as a maintainer, I'd be happy to add you. |
|
That would be nice. I would like to help maintain |
|
@Sympatron sent you an invite! Feel free to merge PRs as you see fit. Feel free to ping me if you want to coordinate a release. |
|
@ithinuel Are you ok with me making changes to this PR to resolve conflicts? And as I said, I would like to keep the |
|
Sorry for the slow reply. |
|
I came looking to do the same thing. Sounds like there aren't any objections to merging this. @Sympatron is there anything I could do to help get this merged? I would like to use bbqueue in my project and this is blocking me. |
|
@ggriffiniii I would strongly recommend checking out If there are no objections and the other maintainers are busy, I'm going to just merge bbq2 into bbqueue and maintain that instead. |
|
Thanks for the pointer. Yes, indeed bbq2 seems like it does exactly what I'm looking for. |
No description provided.