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
osmid: init at 0.6.8 #69084
osmid: init at 0.6.8 #69084
Conversation
I wonder if the bot is still on the fritz... |
ping @grahamc for another example of bot not building. |
1686cff
to
0781c3a
Compare
Thanks for your review! I've made the changes. |
0781c3a
to
dc4df25
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.
Builds with nix-review on nixos and runs (at least the help messages do)
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.
@c0deaddict I apologize I didn't get back to this yesterday. I meant to during my lunch hour but ended up busier than expected. 2 minor changes and then based on testing by @alexarice we're good to go 👍
dc4df25
to
fe74062
Compare
@GrahamcOfBorg build osmid |
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.
Thanks! 🎉
Motivation for this change
Osmid is an optional dependency of Sonic-Pi to communicatie to and from MIDI devices.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @