1171072 - Make libcubeb build for iOS

Status: RESOLVED FIXED

(Core :: Audio/Video, defect)

Description

[(not currently active) Ted Mielczarek]

I got it to build with a lot of hacking, but I would bet it doesn't actually work, I left a lot of TODOs in there. I think I exhausted my abilities in this space, I could use some help.

Comment 1

[(not currently active) Ted Mielczarek]

Also, I'm not totally clear, but maybe these changes need to go upstream first?
https://github.com/kinetiknz/cubeb

Comment 2

[(not currently active) Ted Mielczarek]

Attachment: MozReview Request: bug 1171072 - make libcubeb build on iOS. r?kinetik

bug 1171072 - make libcubeb build on iOS. r?kinetik

Comment 3

[(not currently active) Ted Mielczarek]

This patch is kind of a mess, but it builds, so that's something.

Comment 4

[Matthew Gregan [:kinetik]]

(In reply to Ted Mielczarek [:ted.mielczarek] from comment #1)
> Also, I'm not totally clear, but maybe these changes need to go upstream
> first?
> https://github.com/kinetiknz/cubeb

Ideally, yes... especially if you're volunteering for the pain of updating the configure-y stuff for the iOS build. :-)

Comment 5

[Matthew Gregan [:kinetik]]

Comment on attachment 8614805 [details]
MozReview Request: bug 1171072 - make libcubeb build on iOS. r?kinetik

https://reviewboard.mozilla.org/r/10063/#review8887

Thanks for doing this! It looks like most of the missing/TODO bits are for optional functionality, so that's cool.  We do want the sample rate and latency stuff for WebAudio/WebRTC, but it will work (suboptimally) without it, so the patch is basically fine to land as a first pass.  I don't know RemoteIO at all, so I can't offer much immediate help, but happy to help out where I can.

::: media/libcubeb/src/cubeb_audiounit.c:447
(Diff revision 1)
> -
> +#if !TARGET_OS_IPHONE

Return CUBEB_ERROR_NOT_SUPPORTED for iOS for now.

::: media/libcubeb/src/cubeb_audiounit.c:462
(Diff revision 1)
> +#if !TARGET_OS_IPHONE

Ditto here.

::: media/libcubeb/src/cubeb_audiounit.c:648
(Diff revision 1)
> +#endif

Move the endif below the buffer_size setting stuff, since latency_range will be uninitialized here.

::: media/libcubeb/src/cubeb_audiounit.c:848
(Diff revision 1)
> +  //TODO

Return CUBEB_ERROR_NOT_SUPPORTED.

::: media/libcubeb/src/cubeb_audiounit.c:887
(Diff revision 1)
> +  return CUBEB_ERROR;

CUBEB_ERROR_NOT_SUPPORTED

Comment 6

[(not currently active) Ted Mielczarek]

https://hg.mozilla.org/integration/mozilla-inbound/rev/d832a498605f

Comment 7

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/d832a498605f