Write a full-duplex ALSA cubeb backend

NEW
Unassigned

Status

()

defect
P5
normal
Rank:
60
4 years ago
11 months ago

People

(Reporter: padenot, Unassigned)

Tracking

(Blocks 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

Comment hidden (empty)
Component: Audio/Video: MediaStreamGraph → Audio/Video: cubeb
Rank: 15
Priority: -- → P1
Rank: 15 → 19
I'm putting this in the parking lot. Supporting ALSA is not on the roadmap.
Rank: 19 → 60
Priority: P1 → P5

Comment 2

2 years ago
I would like to tackle this one.
(In reply to Maire Reavy [:mreavy] Please needinfo me from comment #1)
> I'm putting this in the parking lot. Supporting ALSA is not on the roadmap.

As a multi-year donor to the Mozilla Foundation, I would like to remind Mozilla’s product management that there are paying Firefox users who do not have PulseAudio installed and who would like ALSA support *fixed* instead of “put in the parking lot”.

The rubber wall erected in #1345661 is also not a way to encourage donors to keep on supporting Mozilla.

For the record, since I’m using Gentoo all I had to do was switch to the “compile from source” package instead of Mozilla’s official binary package, but from the winds around here I can sense that in a few Firefox versions ALSA might be dropped altogether, so even with Gentoo I won’t have PulseAudio-free options.

I’m stating this here because #1345661 has been locked down and I don’t want to hear “why didn’t you voice your need for ALSA?” from Mozilla PMs when ALSA is finally dropped, just like they’re doing now that ALSA has been replaced by PA as the default in Mozilla builds.

(In reply to Ismael Luceno from comment #2)
> I would like to tackle this one.

Ismael – while I have no experience in Firefox’s code base, I’m a software engineer by trade specialized in C++, so please let me know if I can be of any help.
> I’m stating this here because #1345661 has been locked down and I don’t want to hear

Please, don't. This is not the right place - go to a newsgroup if you want to discuss [0].

Bug 1345661 has been restricted for a reason - the reason was the misuse of bugzilla. If you bring that reason to another bug, we'll have to lock it down as well to prevent the misuse.

Focus on writing code here. Create a patch, make it a good one. Then request a review.

[0] https://groups.google.com/forum/#!searchin/mozilla.dev.platform/alsa%7Csort:relevance/mozilla.dev.platform/jRAqSTri66I/GnzpvBxqBgAJ

Comment 5

2 years ago
Should it be fixed in this repo? https://github.com/kinetiknz/cubeb
(Reporter)

Comment 6

2 years ago
Yes, although we can take care of upstreaming stuff if needed.
(Reporter)

Comment 8

2 years ago
The right code is in mozilla-central. However, it looks like it does not work too well. I tried forcing using alsa, and it crashes.
You need to log in before you can comment on or make changes to this bug.