meson: adrian-aec requires libm

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Make webrtc optional

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Make hal-compat optional

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Make udev optional

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Make speex optional

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Make openssl optional, fix build when openssl is disabled

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Make lirc optional

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Make fftw optional

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Make jack optional

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Use assert() to validate the prefix

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Fix bluez5 version requirement

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Make bluez optional

I can't promise that the logic is *exactly* the same as the logic
currently in use with the autotools, but it seems correct to me.

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Make avahi optional

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Make systemd optional

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Also set HAVE_ALSA when alsa is enabled

Just following the configure file.

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Make alsa optional

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

module-stream-restore: Silence compiler warnings when dbus is disabled

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Make dbus optional, fix build with dbus disabled

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Adjust fixme comment for daemon

Man pages are handled in man/meson.build, that's where FIXMEs and TODOs
are now.

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Install start-pulseaudio-x11 if x11 is found

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Build pax11publish only if X11 is found

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Make x11 optional, bump meson requirement

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Tighter coupling of XOrg dependencies

If `x11-xcb` is found, then let's force other X11 dependencies to be
there as well. That makes things a bit easier, and that's also what is
done in the autotools build system.

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Add a comment to mark the beginning of the configuration data section

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Create bindir and libdir variables

This is to avoid using the construct 'join_paths(prefix, get_option(...))'
everywhere in the meson files. It's better to settle the paths question
once and for all at the beginning.

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Add a sanity check to the prefix option

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Add pulsedspdir option

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

alsa-ucm: Fix UCM devices which names share a prefix being seen as the same

Before this commit ucm_port_contains() was using a strncmp to compare
UCM-device-names without first checking that the part of the port_name
being compared and the device-name have the same length, this was causing
it to return true for both "InternalMic-IN1" and "InternalMic-IN12" when
port_name contained "InternalMic-IN1".

We hit this with the bytcr_rt5651 UCM profile which has "InternalMic-IN1",
"InternalMic-IN2" and "InternalMic-IN12" devices, for devices with their
internal mic connected to IN1, or IN2, or using stereo internal mics
connected to both. This problem resulted in various problems including
the RECMIXL? BST2 switch getting turned on when selecting only
"InternalMic-IN1", as well as confusing the gnome-control-center sound
panel, which could not figure out which device is selected in this case.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

dockerfile: Install Meson from upstream tarball

This is because the meson build requires meson 0.47, which is not
available in the current Ubuntu LTS (18.04).

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

module-alsa-card: Update the active profile's availability last

The previous commit introduces logic in module-switch-on-port-available
that may change a card's active profile when its availability changes to
PA_AVAILABLE_NO. To choose the new active profile, it needs a consistent
view of the new availability of all profiles, so this commit changes the
order which the ALSA driver updates all profiles' availability to ensure
the active profile is last.

This is not generic enough to cover cases were we may want to take an
action on availability changes of profiles other than the active one
that also need a consistent view of all profiles' availability. But we
don't have any callbacks implementing such action at the moment.

switch-on-port-available: Check if we need to change the active profile

When a port becomes unavailble its profile may also become unavailable.
If that profile is the card's active profile, we need to switch the
card's active profile to a different one.

If we don't do that a card may get stuck on a profile without available
ports, but its sink and source will still exist, preventing
module-rescue-streams to move the streams to a different card with
available ports.

The relation between port availability and profile availability is
defined by the driver, and for the ALSA driver a profile is considered
available if there is at least one (available || unknown) port for each
direction implemented by the profile. Because of that we can only check
the profile's availability and priority when looking for the best
profile and don't need to look at port's priorities.

https://phabricator.endlessm.com/T24904

build-sys: introduce a special build flag to explicitly disables running from build tree

It is helpful to improve reproducibility build [1] since
PA_SRCDIR/PA_BUILDDIR contains build path,
--disable-running-from-build-tree could drop these macros at
precompilation.

[1] https://reproducible-builds.org/

Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>

switch-on-connect: Do not ignore HDMI sinks

HDMI ports are normally present on cards connected to an internal bus,
and module-switch-on-connect should switch to them when a HDMI monitor
is plugged.

This is specially relevant on setups where the HDMI port of a machine is
connected to a different audio card then the analog outputs, which is
the case for machines with AMD graphics cards.

alsa-mixer: Add support for 2018 Arctis 7

raop: fix typo: modules -> modulus

raop: add error handling to rsa_encrypt()

When reviewing another change in rsa_encrypt(), Felipe Sateler pointed
out some deficiencies in error handling. This patch adds error handling
for all openssl calls in rsa_encrypt().

This patch doesn't propagate the error all the way up to the
pa_rtsp_client owner, because there's no mechanism for doing that. I
could implement such mechanism myself, but I think it's better I don't
make such complex changes to the RAOP code, because I don't have any
RAOP hardware to test the changes. The result is that module-raop-sink
will just sit around without doing anything. I think this is still
better than having no error handling at all.

alsa-mixer: Update to support Arctis Pro Wireless headset

The Arctis 7 configuration can be used as is - the ALSA PCM and mixer
interfaces are the same.

alsa-sink/source, sink, source: Consider sample format for avoid-resampling/passthrough

Sample format(e.g. 16 bit, 24 bit) was not considered even if the
avoid-resampling option is set or the passthrough mode is used.
This patch checks both sample format and rate of a stream to
determine whether to avoid resampling in case of the option is set.
In other word, it is possble to use the stream's original sample
format and rate without resampling as long as these are supported
by the device.

pa_sink_input_update_rate() and pa_source_output_update_rate() are
renamed to pa_sink_input_update_resampler() and pa_source_output
_update_resampler() respectively.

functions are added as below.
 pa_sink_set_sample_format(), pa_sink_set_sample_rate(),
 pa_source_set_sample_format(), pa_source_set_sample_rate()

Signed-off-by: Sangchul Lee <sc11.lee@samsung.com>

resampler: Fix confusion between rear and side channels for 5.1 layouts

mpv and vlc play "normal" 5.1 AC3 and DTS files as if they had a
"5.1 (Side)" layout. Which is fine and consistent with ITU
recommendations if the user has a proper 7.1 system. But if the user
actually has a 5.1 system, PulseAudio will try to remap, poorly, between
the "5.1 (Side)" and "5.1" layouts, sending either an average between
front and rear channels, or an attenuated version of that average,
depending on the remixing-use-all-sink-channels setting.

This is not desired, the "Side" channels should be sent to "Rear", it is
only an unfortunate nomenclature confusion.

This patch does not fix 5.1 <-> 7.1 remixing.

Signed-off-by: Alexander E. Patrakov <patrakov@gmail.com>

alsa-mixer: swap lineout and headphone priorities

Headphones should have higher priority than lineout. Many people have
speakers always connected to lineout, and when plugging in headphones,
the audio should move to the headphones, which requires headphones
to have higher priority than lineout.

Previously this was handled by marking lineout unavailable when plugging
in headphones, but we don't do that any more.

alsa-mixer: don't make lineout unavailable when headphones are plugged in

This reverts commit 66f97c35bda9524985b8d34d9a7dbfa9b1359df5. The commit
message was:

    alsa-mixer: Disable line-out if headphone jack is plugged

    Line-out gets muted when headphones are plugged in on HDA cards, encode
    this in the line-out path so pulse can match that state.

I don't think the mentioned auto-muting happens any more by default,
and some users want to use lineout while having headphones plugged in.

Fixes: https://gitlab.freedesktop.org/pulseaudio/pulseaudio/issues/583

.gitignore: add passthrough-test

daemon-conf: fix printing avoid-resampling in pa_daemon_conf_dump()

The bool was inverted for some reason - maybe because the next line
prints enable-remixing that needs to be inverted from disable_remixing,
and somehow this logic was accidentally copied to the avoid-resampling
handling.

Fixes: https://gitlab.freedesktop.org/pulseaudio/pulseaudio/issues/568

gitignore: Ignore cscope files

Remove active_profile

vala: add pa_card_port_info bindings

vala: Fix card profile array bindings

volume-test: Allow rounding without having to allow a random number of errors

passthrough-test: Add a test for volume set/restore

tests: Add a unit test for passthrough support

This adds some basic infrastructure to test passthrough support. Right
now, it just creates a passthrough stream and makes sure negotiation
works. We'll add in more tests as we go along.

null-sink: Add support for reconfiguration and non-PCM formats

This makes it easier to manually test as well as add unit tests for
passthrough support.

build-sys: Fix meson version detection logic

This sync the meson version detection to match what we do in the
autotools build, which is to use git-version-gen, which in turn looks
for <topdir>/.tarball-version, or generates the version based on a git
tag or environment variable.

i18n: Provide *ngettext() stubs when NLS is diabled

card: Log initial profile selection

Add logs to pa_card_choose_initial_profile and pa_card_set_profile to
have detailed logging of the profile initialization logic.

device-port: Use pa_available_to_string

Replace using the ternary operator with pa_available_to_string to print
value of a pa_available_t.

cli-text: Use pa_available_to_string

Replace the local available_to_string with calling
pa_available_to_string from card.h.

card: Create pa_available_to_string

Add a new function to the card API that returns the string
representation of a pa_available_t variable.

build-sys: Set minimum required meson version to 0.44.0

We now use get_unquoted() during manual page generation which requires
0.44.0 or greater.

i18n: Fixed plural forms handling

meson: Build and install man pages

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

webrtc: Silence -Wsign-compare warning

  webrtc.cc:202:19: warning: comparison of integer expressions of different signedness:
  'int' and 'std::vector<webrtc::CartesianPoint<float> >::size_type' {aka 'long unsigned int'}

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Enable echo-cancel module, add libwebrtc_util lib, add speex and webrtc dependencies

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Enable ladspa module

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Enable oss module and padsp util

Regarding the module:

This is unlike the autotools where liboss-util is built as a library,
here we build everything in the oss module, as apparently there's no
other consumer for liboss-util.

Regarding padsp:

Setting the install mode for padsp requires meson 0.47, so instead we
set padsp.in as executable in the git repository (which is what glib
does for gdbus-codegen btw).

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Enable bluetooth modules, add bluez_util lib, add bluez and sbc dependencies

Please notice that the bluez5 version seems wrong in the dependency
declaration: `>= 4.x`, while we're talking about version 5.

The ofono part will need to be made optional when we start to work on
the meson_options file.

I follow the current configure.ac to define 'HAVE_BLUEZ', but it looks
like this part would benefit from a bit of rework. Setting HAVE_BLUEZ
when we have dbus+sbc sounds weird, there's probably a better name for
this variable.

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Do not build dbus files as a library

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Enable console-kit module

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Enable lirc module, add lirc dependency

This is unlike the autotools where we check that a header exist, here we
use pkgconfig because upstream ships a pkgconfig. I don't know from
which version though...

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Enable mmkbd-evdev module

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Enable zeroconf modules

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Enable x11 modules, add x11 related dependencies

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Enable raop-discover module, add avahi dependency

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: modules/raop: Fix rtsp_client.h include path

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Enable raop-sink module, add raop lib, add openssl dependency

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Enable jack modules, add jack dependency

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Enable dbus-protocol module

This is similar to alsa and rtp modules.

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Fix all the missing dependencies uncovered by -Wl,--no-undefined

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Build modules with '-Wl,no-undefined'

This flag will make the loader fail if symbols are not resolved. It
seems to be our best bet to uncover every missing module dependencies.

For more details, I recommend to read:
<http://www.kaizou.org/2015/01/linux-libraries/>

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Comment bluez5 modules for now

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Add the rtp module library

This is based on the existing libalsa-util library.

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Add pulsecore internal libraries

This is based on the initial protocol_native library that is already
defined, and then by looking at the Makefile.am to work out the
dependencies.

It's not clear whether we really need database_c_args, maybe there's
things that can be simplified.

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Make equalizer-sink modulde depend on fftw, add fftw dependency

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Make systemd-login module depend on systemd

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Propagate systemd dep to libpulsecommon and daemon

This is needed now that we define HAVE_SYSTEMD_xxx

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Add systemd-login to core dependencies

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: modules/alsa: Fix udev-util include path

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: modules/alsa: Make alsa-util a shared library

This is to be consistent. In pa currently, as built by the autotools,
libalsa-util is a shared library. Moreover, all the libraries for the
modules, as defined in `src/meson.build`, are also shared libraries.

So let's stick to shared libraries everywhere for now, for simplicity.

We can rework that later on.

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: modules/alsa: Add udev support

This is needed now that we define HAVE_UDEV

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

meson: Define HAVE_UDEV when udev is found

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

core: Fix a log function when HAVE_SYSTEMD_JOURNAL && !HAVE_SYSLOG

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

gitlab-ci: Create two different jobs for autotools and meson builds

Signed-off-by: Arnaud Rebillout <arnaud.rebillout@collabora.com>

card: Fix typo in comments

module-rescue-streams: Fix tab used for identation

switch-on-connect: Add debug logs

Log when module-switch-on-connect tries to change the default sink or
source.

switch-on-connect: Clean-up tabs used for identation

switch-on-connect: Improve readability

Make code easier to read by moving pa_proplist_gets out of the if
condition and using pa_safe_streq.

switch-on-port-available: Improve readability

Split a big conditional into separate checks and use pa_safe_streq
instead of checking if a pointer is valid and calling pa_streq inside a
conditional.

i18n: Updated Czech translation

man: Fix a syntax issue in pacat xml

'cmd' is not a valid child of the 'p' tag, so we switch it to 'file',
which is how this is done later in the file already.

gitlab: Update Dockerfile to pull xmllint