Skip to content

Conversation

@daft-panda
Copy link

The current Audio Unit initialization logic produces errors when trying to use output devices as input Audio Units. This results in only some output devices being present when enumerating the host devices.

Copy link

@ahcm ahcm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This fixes the error:
Error getting supported output configs: BackendSpecific { err: BackendSpecificError { description: "Invalid property value" } }

Find attached output from the example enumerate, with and without the patch.
Please merge.

Best
Andreas

cpal_enumerate_default_N9_daft-panda.txt
cpal_enumerate_default_N9.txt
cpal_enumerate_default_speaker_daft-panda.txt
cpal_enumerate_default_speakers.txt

@mdwn
Copy link

mdwn commented Dec 18, 2024

This looks to me to be a pretty benign one-liner change -- I'd love to get this merged. @abique, @est31, would it be possible to get this merged?

@etsuko-fm
Copy link

I applied this change in my local fork and it fixes an issue that I've been experiencing with output devices not being listed. When merged I expect it would close #814 and by extension #905.

Copy link
Contributor

@will3942 will3942 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This fixes a long standing issue on MacOS - It would be great if we can get this merged @est31

@spennythug
Copy link

I can confirm that this fixes the issues already mentioned for me

I have also been experiencing an issue where I can only use certain output devices if they are also selected as the system's output device. By applying this same change in the default_config function, (around line 400) this issue is fixed for me as well. I assume this only happens when using default_output_config

I'd be happy to put in a separate PR, if that's necessary, but it seems like this change could simply be tacked on to this one

@mdwn
Copy link

mdwn commented Mar 19, 2025

All, I think this was fixed by #946 in a similar fashion. @daft-panda would you be able to confirm?

@AxelMolina1
Copy link

If this fix is merged, do we have a visibility on when it would be deployed? I'm seeing last version was a year ago and I'm starting to feel concerned for my project that relies a lot on cpal

@ahcm
Copy link

ahcm commented Apr 17, 2025

@mdwn yes #946 fixed it.

@etsuko-fm
Copy link

etsuko-fm commented Apr 17, 2025

@AxelMolina1
To get the latest into my projects already, I refer to a git commit in Cargo.toml:

[dependencies]
cpal = { git = "https://github.com/RustAudio/cpal.git", rev="c5a163e" }

@AxelMolina1
Copy link

@etsuko-io Oh okay thanks I'll try this !

@est31
Copy link
Member

est31 commented Apr 24, 2025

closing as we've merged #946 instead.

@est31 est31 closed this Apr 24, 2025
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.

8 participants