Bug 250370 - www/chromium: Enable sndio by default
Summary: www/chromium: Enable sndio by default
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: freebsd-chromium (Nobody)
URL:
Keywords: needs-patch, needs-qa
Depends on:
Blocks:
 
Reported: 2020-10-15 15:01 UTC by Baptiste Daroussin
Modified: 2020-10-23 20:51 UTC (History)
3 users (show)

See Also:
koobs: maintainer-feedback? (chromium)


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Baptiste Daroussin freebsd_committer 2020-10-15 15:01:58 UTC
Would it be possible to turn sndio to on by default in chromium, this is the only to have proper audio in visio conferences (default alsa has crackling audio when there are multiple persons in visio) tested with google meet, microsoft teams and Jitsi meet.
Comment 1 Jan Beich freebsd_committer 2020-10-16 14:34:26 UTC
How well chromium works when sndiod is NOT running? See bug 218188. For comparison, pulseaudio doesn't support serverless mode, so the daemon is started on demand by libpulse.

From Makefile:
# With SNDIO=on we exclude audio_manager_linux from the build (see
# media/audio/BUILD.gn) and use audio_manager_openbsd which does not
# support falling back to ALSA or PulseAudio.
Comment 2 Joseph Mingrone freebsd_committer 2020-10-19 15:39:17 UTC
I have used Chromium with only SNDIO on for a few months.  As jbeich points out we currently have SNDIO_PREVENTS=ALSA PULSEAUDIO, so there will probably be some disruption, but based on my experience as a user, +1 for this change.
Comment 3 commit-hook freebsd_committer 2020-10-23 20:49:21 UTC
A commit references this bug:

Author: jrm
Date: Fri Oct 23 20:48:38 UTC 2020
New revision: 553141
URL: https://svnweb.freebsd.org/changeset/ports/553141

Log:
  www/chromium: Use SNDIO instead of ALSA for audio by default

  There are stuttering sound issues when using ALSA, so switch to something
  that works.

  PR:		250370

Changes:
  head/www/chromium/Makefile