Bug 226101 - Patch for audio/rtaudio
Summary: Patch for audio/rtaudio
Status: Closed Not Accepted
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Yuri Victorovich
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-02-21 23:55 UTC by Shane
Modified: 2019-08-11 02:20 UTC (History)
0 users

See Also:
bugzilla: maintainer-feedback? (yuri)


Attachments
additional patch for audio/rtaudio (3.55 KB, patch)
2018-02-21 23:55 UTC, Shane
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Shane 2018-02-21 23:55:33 UTC
Created attachment 190877 [details]
additional patch for audio/rtaudio

While starting a new port (audio/vcvrack), I found some changes to rtaudio were needed.

As the maintainer of devel/godot (which has the rtaudio files bundled) I needed to adjust the OSS support to get it to work. To get godot OSS audio working I changed rtaudio to open /dev/dsp instead of /dev/mixer. This change has been part of the godot port for almost 1.5 years without any bug reports.

With vcvrack I needed to make the same changes to audio/rtaudio and also found another issue. The available formats is only listing input formats, this means that the output channels did not have any supported formats and vcvrack would only allow connections to channels with supported formats. This leads to only being able to connect to an input channel when it is trying to output sound.

To get this working I changed the format tests to consider both input and output formats, so the format list becomes a combination of both input and output formats. This leads to every audio port being listed and got sound output working.
Comment 1 Yuri Victorovich freebsd_committer 2018-02-22 03:29:18 UTC
Hi Shane,

Thank you for your inquiry.

I think you should create a pull request for them here: https://github.com/thestk/rtaudio

Port can't be patched so extensively. Port can only include an upstream patch if there is an emergency. "thestk" account there is very responsive in my experience. Once they accept your pull request, I can move the port revision to that point if necessary.

Yuri
Comment 2 Yuri Victorovich freebsd_committer 2018-02-22 09:35:46 UTC
Needs to be submitted upstream.