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
a2jmidid: fix compilation on aarch64-linux #91920
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 LGTM
- Commits need squashed together, shouldn't be separate
- Builds fine locally
- Doesn't run cleanly locally, probably b/c I don't have the proper hardware?? Log:
[nix-shell:~/.cache/nixpkgs-review/pr-91920]$ a2jmidid
Directory "/home/.../.log" does not exist. Creating...
Directory "/home/.../.log/a2j" does not exist. Creating...
Directory "/home/.../.config/a2j" does not exist. Creating...
JACK MIDI <-> ALSA sequencer MIDI bridge, version 8 () built on Wed Dec 31 19:00:00 1969
Copyright 2006,2007 Dmitry S. Baikov
Copyright 2007,2008,2009,2011,2012 Nedko Arnaudov
Bridge starting...
Using JACK server 'default'
Hardware ports will not be exported.
ALSA lib seq_hw.c:466:(snd_seq_hw_open) open /dev/snd/seq failed: No such file or directory
ERROR: a2j_new: failed to open alsa seq
ERROR: a2j_start: a2j_new() failed.
Hi, yes, you need an ALSA sequencer (MIDI) device for this program to work, which is precisely what the error message tells you. Thanks for the review! |
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.
Approved.
- Comments addressed
- Builds cleanly on OfBorg aarch64 (don't have local aarch64 to test).
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.
- working on a RPi 4:
[root@dca63203d971:~]# /nix/store/pvc43db7p1scq2z5zjl7c8hlz2x3xwp9-a2jmidid-8/bin/a2jmidid
Directory "/root/.log" does not exist. Creating...
Directory "/root/.log/a2j" does not exist. Creating...
Directory "/root/.config/a2j" does not exist. Creating...
JACK MIDI <-> ALSA sequencer MIDI bridge, version 8 () built on Thu Jan 1 00:00:01 1970
Copyright 2006,2007 Dmitry S. Baikov
Copyright 2007,2008,2009,2011,2012 Nedko Arnaudov
Bridge starting...
Using JACK server 'default'
Hardware ports will not be exported.
Bridge started
Press ctrl-c to stop the bridge
port created: Midi Through [14] (capture): Midi Through Port-0
port created: Midi Through [14] (playback): Midi Through Port-0
```
Motivation for this change
I wanted to use the services.jack on a raspberrypi (aarch64) system and a2jmidid not building on that system kept me from doing that. So I fixed it. I cherry picked a diff from the commit that fixed this in the project's repository.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)