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
mididings: init at 20151117 #63207
mididings: init at 20151117 #63207
Conversation
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.
Diff looks good to me.
|
||
preBuild = with stdenv.lib.versions; '' | ||
substituteInPlace setup.py \ | ||
--replace boost_python "boost_python${major pythonPackages.python.version}${minor pythonPackages.python.version}" |
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.
What is this major and minor?
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.
Aren't these functions from stdenv.lib.versions
?
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.
@FRidh @matthiasbeyer They are. imported with with stdenv.lib.versions;
Needed because pythonPackages.boost
library is at path lib/libboost_python27.so
. Please let me know if there's a better way to find that library
@GrahamcOfBorg build mididings |
@FRidh Thank you! |
Motivation for this change
mididings is a MIDI router and processor based on Python, supporting ALSA and JACK MIDI. It is available under the GNU GPL and currently runs on Linux.
Closes #50898
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)