-
Notifications
You must be signed in to change notification settings - Fork 62
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
compilation fix for no-std #69
base: master
Are you sure you want to change the base?
compilation fix for no-std #69
Conversation
cc: @tcharding Would be nice to add this check to CI somehow. Do you have any idea how to do it? |
Co-authored-by: Tobin C. Harding <[email protected]>
9c3cee9
to
8653fb2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK 8653fb2
Can you merge this? |
Saw this comment because PR turned up in my notifications. Dunno if this is just as stale comment but there is a check for this in CI ATM on |
cc @stevenroose |
This PR fixes following compilation problem for no-std environments:
The problem can be reproduced with the following steps: