Skip to content
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

Add minimal Cpal audio library example #11

Merged
merged 1 commit into from
Aug 12, 2022
Merged

Add minimal Cpal audio library example #11

merged 1 commit into from
Aug 12, 2022

Conversation

rib
Copy link
Collaborator

@rib rib commented Aug 11, 2022

Based on the android example that's in the cpal repo, this test plays
a 440hz sine wave and is based on GameActivity

Note: this requires a workaround branch for cpal that comments out
a dev-dependency on ndk-glue that cpal has for its android example.
This is needed because Cargo is spuriously propagating this dev
dependency outside of the cpal package (which leads to a crash)

@rib rib force-pushed the cpal-example branch 5 times, most recently from 7f73d83 to e284b71 Compare August 12, 2022 18:14
Based on the android example that's in the cpal repo, this test plays
a 440hz sine wave and is based on GameActivity

Note: this requires a workaround branch for cpal that comments out
a dev-dependency on ndk-glue that cpal has for its android example.
This is needed because Cargo is spuriously propagating this dev
dependency outside of the cpal package (which leads to a crash)
@rib
Copy link
Collaborator Author

rib commented Aug 12, 2022

While testing this I was hitting a crash when pausing/resuming the app which I was initially assuming was a oboe-related issue in cpal but it turned out to be an NDK 0.7 related issue that was actually affecting all the examples: 93828a1

@rib rib merged commit ecd03ed into main Aug 12, 2022
@rib rib deleted the cpal-example branch December 20, 2022 16:38
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.

1 participant