Skip to content

add sndio support #313

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

Closed
wants to merge 2 commits into from
Closed

add sndio support #313

wants to merge 2 commits into from

Conversation

jg1uaa
Copy link
Contributor

@jg1uaa jg1uaa commented Feb 13, 2021

No description provided.

@jg1uaa
Copy link
Contributor Author

jg1uaa commented Feb 13, 2021

Currently supporting PortAudio enables using direwolf on OpenBSD, but PortAudio is dedicated for single audio device not for multiple.
Even if PA_SNDIO_AUDIODEVICES variable can handle multiple devices (http://openbsd-archive.7691.n7.nabble.com/audio-portaudio-svn-Add-support-for-multiple-sndio-devices-td358622.html), it is difficult to select device for input/output. Both specifying PA_SNDIO_AUDIODEVICE and ADEVICE in direwolf.conf is needed.

Therefore, native sndio support is strongly required. And this will help other *BSD users such as NetBSD and FreeBSD. Some Linux distribution has sndio support, so this code can be tested with modified CMakeLists.txt.

@jg1uaa
Copy link
Contributor Author

jg1uaa commented Feb 25, 2021

I will resend pull request to dev branch, so I close this request (to master).

@jg1uaa jg1uaa closed this Feb 25, 2021
@wb2osz
Copy link
Owner

wb2osz commented Feb 25, 2021

Please be patient. I have a job, family, and other things I need to take care of. If something is in the queue I will get to it eventually. Thanks.

@jg1uaa
Copy link
Contributor Author

jg1uaa commented Feb 26, 2021

Sorry, I found a serious trouble that sndio-supported direwolf crashes without any messages when unplugging USB audio or device error. So I had to once close this request.

Now I reconstructed sndio support on dev branch, it looks no problem. I will resend PR within a few days, please reveiw new code if you have enough time. Regards,

postscript:
be careful to use direwolf with cheap USB audio device, it is sometimes unreliable...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants