Skip to content

Conversation

@philpax
Copy link
Contributor

@philpax philpax commented Oct 1, 2025

This PR is a prospective update of the cpal dependency to the latest Git version. It is not intended to be merged (at least until the next cpal release is cut); instead, it's intended as a version of rodio that can have its cpal patched with a fork of the ~current cpal HEAD (e.g. my RustAudio/cpal#1027).

The only changes so far:

  • running cargo update -p objc2 to break a dependency resolution conflict
  • replacing From<Input> for cpal::Device with Input::into_inner, as cpal now has a conflicting trait implementation. I considered implementing Into, but the asymmetry of not having the From is generally frowned upon. (the From blanket impl blocks this)
  • enables U24 support
  • removes the cpal-shared-stdcxx feature, which activates the oboe backend for cpal, which has since been removed

@philpax philpax force-pushed the latest-cpal-fixes branch from 528137e to 6851f06 Compare October 1, 2025 18:50
@roderickvd
Copy link
Member

Thanks for this preparatory work. Now that RustAudio/cpal#1011 is merged, we can also enable the U24 sample format in stream.rs and microphone.rs.

@philpax
Copy link
Contributor Author

philpax commented Oct 14, 2025

Done and updated to the latest cpal 🙂

@roderickvd
Copy link
Member

I'd like to release v0.17 soon. call for final testing!
RustAudio/cpal#1065

@philpax philpax marked this pull request as ready for review December 10, 2025 01:21
@philpax
Copy link
Contributor Author

philpax commented Dec 10, 2025

I'm currently on holiday and can't update my PRs, but that seems fine at a glance! Feel free to use this PR / close it / do whatever you need to 🙂

@roderickvd
Copy link
Member

Updated it to latest cpal master.

@roderickvd roderickvd added the dependencies Pull requests that update a dependency file label Dec 10, 2025
@roderickvd
Copy link
Member

Just released cpal v0.1.70. Updating this PR to match and merging when the CI passes.

@roderickvd roderickvd merged commit 744f27b into RustAudio:master Dec 20, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants