Paul Adenot (:padenot) Reporter
	Description • 9 years ago

      No description provided.

	Maire Reavy [:mreavy]
	Comment 1 • 8 years ago

I'm putting this in the parking lot. Supporting ALSA is not on the roadmap.

	Ismael Luceno
	Comment 2 • 7 years ago

I would like to tackle this one.

	Raffaello D. Di Napoli
	Comment 3 • 7 years ago

(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.

	Zibi Braniecki [:zbraniecki][:gandalf]
	Comment 4 • 7 years ago

> 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

	a1984993
	Comment 5 • 7 years ago

Should it be fixed in this repo? https://github.com/kinetiknz/cubeb

	Paul Adenot (:padenot) Reporter
	Comment 6 • 7 years ago

Yes, although we can take care of upstreaming stuff if needed.

	Jan Beich
	Comment 7 • 7 years ago

Was it fixed by bug 1331869 via https://github.com/kinetiknz/cubeb/commit/e5f6b69?

	Paul Adenot (:padenot) Reporter
	Comment 8 • 7 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.