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

fix typo: existance -> existence

build-sys: Add CoC to the dist'ed files

Adopt a Code of Conduct

This is simply the Contributor Covenant Code of Conduct document,
version 1.4.1.

fix typo: satisifed -> satisfied

Fix typo: documenation -> documentation

Fix typo: managable -> manageable

Fix typo: substract -> subtract

Fix typo: transmited -> transmitted

Fix typo: definately -> definitely

Fix typo: prefered -> preferred

Fix typo: comming -> coming

Fix typo: recieved -> received

Fix typo: distuingish -> distinguish

Fix typo: properies -> properties

Fix typo: explicitely -> explicitly

Fix typo: differnt -> different

Fix typo: Extention -> Extension

Fix typo: Doulbe -> Double

Fix typo: dependancy -> dependency

Fix typo: occured -> occurred

Fix typo: occurrencies -> occurrences

Fix typo: Retreive -> Retrieve

Fix typo: controling -> controlling

gitlab: Add meson build to CI

build-sys: meson: Generate version.h

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

Update Brazilian Portuguese translation

gitlab: Add support for GitLab CI

This adds a Dockerfile to generate a Docker image with the required
dependencies on top of the standard Ubuntu 18.04 image. The Gitlab CI
then runs the PulseAudio build within this image.

bluetooth: Re-add rtp.h to Makefile.am

This was never explicitly added as a dependency of module-bluez5-device,
but that was hidden by the fact that it was specified for bluez4.

alsa: Support the older model of NI's Traktor Audio 2 DJ

object: Fix pa_object_new macro (missing close paren)

pacat: Fix code style

alsa: Suppress udev detection of sound card for some units on IEEE 1394 bus

A bug was filed to bugzilla.kernel.org for a quirk of some models which
ALSA BeBoB driver supports.

Bug 199365 - repeating bus resets on Firewire bus with Focusrite Saffaire 26/io
https://bugzilla.kernel.org/show_bug.cgi?id=199365

Some models (two models as long as I know) have a quirk to disappear from
IEEE 1394 bus at disconnections of packet streaming. Corresponding
character devices are removed according to 'remove' callbacks of relevant
drivers from Linux dd core. Then the models re-appear on the bus by
generating bus resets and corresponding character devices are added
according to 'probe' callbacks from Linux dd core.

In a view of ALSA applications, this looks that plug-out/plug-in occur in
a sequential order for the models when they stop playback/capture substream.
For most applications, this doesn't cause large issue. However, this quirk
is not good for combination of below modules in PulseAudio. PulseAudio
enters endless loop to detect the models and start/stop PCM substream.
 - module-udev-detect
 - module-alsa-card
 - module-suspend-on-idle

In detail, please read my comment no.6:
https://bugzilla.kernel.org/show_bug.cgi?id=199365#c6

This commit suppressed udev detection of sound card for the issued models.
For the models, 'PULSE_IGNORE' flag is added to udev rules, then
module-udev-detect don't handle the models and PulseAudio never uses the
models automatically. In a scenario for users to load
module-alsa-card/module-alsa-sink/module-alsa-source by hand, although
these modules can still stop PCM substreams with module-suspend-on-idle,
PulseAudio never enters the endless loop because udev detection doesn't
work for the models. In this case, as long as special files for ALSA
character devices for these models are the same, corresponding sinks and
sources are available even if the voluntary plug-out/plug-in occur.

(Focusrite Saffire Pro 10 i/o with systemd 237)
$ udevadm info -q all -p /devices/pci0000:00/0000:00:07.0/fw1/fw1.0/sound/card1
P: /devices/pci0000:00/0000:00:07.0/fw1/fw1.0/sound/card1
E: DEVPATH=/devices/pci0000:00/0000:00:07.0/fw1/fw1.0/sound/card1
E: ID_BUS=firewire
E: ID_FOR_SEAT=sound-pci-0000_00_07_0
E: ID_ID=firewire-0x00130e01000606e0
E: ID_MODEL=Pro10IO
E: ID_MODEL_FROM_DATABASE=XIO2213A/B/XIO2221 IEEE-1394b OHCI Controller [Cheetah Express]
E: ID_MODEL_ID=0x000006
E: ID_PATH=pci-0000:00:07.0
E: ID_PATH_TAG=pci-0000_00_07_0
E: ID_PCI_CLASS_FROM_DATABASE=Serial bus controller
E: ID_PCI_INTERFACE_FROM_DATABASE=OHCI
E: ID_PCI_SUBCLASS_FROM_DATABASE=FireWire (IEEE 1394)
E: ID_SERIAL=0x00130e01000606e0
E: ID_SERIAL_SHORT=0x00130e01000606e0
E: ID_VENDOR=Focusrite
E: ID_VENDOR_FROM_DATABASE=Texas Instruments
E: ID_VENDOR_ID=0x00130e
E: SOUND_INITIALIZED=1
E: SUBSYSTEM=sound
E: SYSTEMD_WANTS=sound.target
E: TAGS=:systemd:seat:
E: USEC_INITIALIZED=957089064

(Focusrite Saffire Pro 26 i/o with systemd 237)
$ udevadm info -q all -p /devices/pci0000:00/0000:00:07.0/fw1/fw1.0/sound/card1
P: /devices/pci0000:00/0000:00:07.0/fw1/fw1.0/sound/card1
E: DEVPATH=/devices/pci0000:00/0000:00:07.0/fw1/fw1.0/sound/card1
E: ID_BUS=firewire
E: ID_FOR_SEAT=sound-pci-0000_00_07_0
E: ID_ID=firewire-0x00130e0100030cdd
E: ID_MODEL=Pro26IO
E: ID_MODEL_FROM_DATABASE=XIO2213A/B/XIO2221 IEEE-1394b OHCI Controller [Cheetah Express]
E: ID_MODEL_ID=0x000003
E: ID_PATH=pci-0000:00:07.0
E: ID_PATH_TAG=pci-0000_00_07_0
E: ID_PCI_CLASS_FROM_DATABASE=Serial bus controller
E: ID_PCI_INTERFACE_FROM_DATABASE=OHCI
E: ID_PCI_SUBCLASS_FROM_DATABASE=FireWire (IEEE 1394)
E: ID_SERIAL=0x00130e0100030cdd
E: ID_SERIAL_SHORT=0x00130e0100030cdd
E: ID_VENDOR=Focusrite
E: ID_VENDOR_FROM_DATABASE=Texas Instruments
E: ID_VENDOR_ID=0x00130e
E: SOUND_INITIALIZED=1
E: SUBSYSTEM=sound
E: SYSTEMD_WANTS=sound.target
E: TAGS=:systemd:seat:
E: USEC_INITIALIZED=1071026684

Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>

i18n: update translation bug reporting links

I looked for outdated links related to the GitLab migration. These are
the only ones I found. There were also some links to various bug
reports in the old Bugzilla, but those don't really need updating, since
Bugzilla should stay readable for a long time.

doc: Update README to point to GitLab

bluetooth: Remove leftover references to the BlueZ 4/5 split

There is no need to mention a module argument is valid only when using
BlueZ 5 now that we don't support BlueZ 4 anymore.

bluetooth: Add missing argument to module's usage

module-bluetooth-discover usage was missing the autodetect_mtu argument.

gitignore: Ignore build* directories

We already had build-aux, might as well generalise for build directories
we create for autotools out-of-tree builds and anything meson-based.

build-sys: Update meson version dependency

We're using SIMD detection logic from 0.42.

build-sys: Add some missing header/function checks to meson build

build-sys: Drop ancient check for dbus_watch_get_unix_fd()

This function was added shortly after dbus 1.1.0, and our minimum
required dbus version is greater than 1.4.

format: Expose pa_format_info convenience getters in API

We move over helper functions to get rate, channels, channel map and
sample format (if PCM) in the public API, so users of the extended API
are more easily able to pull out these values from pa_format_info.

card-restore: Don't restore profile on Bluetooth cards by default

We can provide a better overall user experience with Bluetooth cards by
always choosing the higher audio quality profile (A2DP) by default and
updating the profile selection dynamically according to which streams
are active at a certain moment. The default initial selection has been
addressed by "85daab272 bluetooth: set better priorities for profiles"
and the dynamic profile selection is covered by module-bluetooth-policy.

In addition, module-card-restore's database entries for Bluetooth devices
are retained after a device is removed from the system, leading to the
previously selected profile being restored after a new pairing with the
same device, with no way for the user to erase this memory and reset the
default profile except manually fiddling with module-card-restore's
database.

This commit adds a module argument to have module-card-restore ignore
Bluetooth profiles and this behavior is set as default.

update NEWS

operation: avoid state change from final state

The internal operation_set_state function already returns early if the
new state is the same as the existing state. The attached patch extends
this to return early if already in a finalised (done/cancelled) state,
i.e. blocks attempts to re-finalise into a different state.

This helps avoid unlinking more than once (or crashing on ref count
assertion).

I was not certain whether an assertion would be a better alternative -
with such a crash helping highlight usage problems...

The situation that lead to this was the thought of someone stupidly
trying to pa_operation_cancel() a callback within the callback
execution itself, while designing a solution for a memory leak related
to cancellation within my Rust binding. While no-one should do such a
thing, if they did, they'd either trip up a ref count assertion, or the
operation would be unlinked twice, which would be bad. It's a simple
thing to catch and mitigate, and could prove to be a useful
bulletproofing measure for this function in general.

update NEWS

i18n: update the Lithuanian translation

pipe-sink, pipe-source: fix file permissions

We recently changed the umask of the daemon from 022 to 077, which broke
module-pipe-sink in the system mode, because nobody was allowed to read
from the pipe.

module-pipe-source in the system mode was probably always broken,
because the old umask of 022 should prevent anyone from writing to the
pipe.

This patch uses chmod() after the file creation to set the permissions
to 0666, which is what the fkfifo() call tried to set.

link: https://bugs.freedesktop.org/show_bug.cgi?id=107070

ladspa-sink: fix search path

Having a single level macro for stringizing LADSPA_PATH doesn't work,
because the '#' preprocessor operator doesn't expand any macros in its
parameter. As a result, we used the string "LADSPA_PATH" as the search
path, and obviously no plugins were ever found.

This adds a two-level macro in macro.h and uses that to expand and
stringize LADSPA_PATH.

link: https://bugs.freedesktop.org/show_bug.cgi?id=107078

alsa-sink/source: Rename a variable for supported sample rates in userdata

It is changed from 'rates' to 'supported_rates'.

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

rtp-send: add configurable RTP stream name

Add configuration option 'stream_name' for stream/session name so user
will see it on receiver side as RTP Strean ($stream_name)

ex: load-module module-rtp-send source=rtp.monitor stream_name=MyServerMedia

alsa-util/sink/source: Add infrastructure for supported sample formats

There has been a function to get supported sample rates from alsa and
an array for it in userdata of each module-alsa-sink/source. Similarly,
this patch adds a function to get supported sample formats(bit depth)
from alsa and an array for it to each userdata of the modules.

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

i18n: Update the Korean translation

Translate argument of module-filter-apply and fix words which are
not in common usage.

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

sink, source: remove the state getters

pa_sink_get_state() and pa_source_get_state() just return the state
variable. We can as well access the state variable directly.

There are no behaviour changes, except that module-virtual-source
accessed the main thread's sink state variable from its push() callback.
I fixed the module so that it uses the thread_info.state variable
instead. Also, the compiler started to complain about comparing a sink
state variable to a source state enum value in protocol-esound.c. The
underlying bug was that a source pointer was assigned to a variable
whose type was a sink pointer (somehow using the pa_source_get_state()
macro confused the compiler enough so that it didn't complain before).
I fixed the variable type.

sink-input, source-output: remove the state getters

pa_sink_input_get_state() and pa_source_output_get_state() just return
the state variable. We can as well access the state variable directly.

There are no behaviour changes, except that some filter sources accessed
the main thread's state variable from their push() callbacks. I fixed
them so that they use the thread_info.state variable instead.

sink-input: remove the DRAINED state

The only thing that the drained state was being used for was "pacmd
list-sink-inputs". In all other cases the drained and running states
were treated as equivalent. IMHO, this usage doesn't justify the
complexity that the additional state brings.

This patch was inspired by a bug report[1] that pointed out an error in
an if condition in pa_sink_input_set_state_within_thread(). The buggy
code is now removed altogether.

[1] https://bugs.freedesktop.org/show_bug.cgi?id=106982

switch-on-port-available: ignore bluetooth cards

When the user manually switches the profile of a bluetooth headset from
"off" to "a2dp_sink", the port availability changes from "unknown" to
"yes", which triggered a recursive profile change in
module-switch-on-port-available. Such recursivity isn't (and possibly
can't) be handled well (that is, PulseAudio crashed), so let's avoid
doing bluetooth profile changes from module-switch-on-port-available
(they're useless anyway).

link: https://bugs.freedesktop.org/show_bug.cgi?id=107044

travis: Update gettext along with other packages

Hopefully fixes the build.

mainloop: constify is_our_api api ref

bluetooth: backend-ofono: Demote registration failure to info

Now that both backend-native and backend-ofono can coexist and
backend-ofono is always loaded, even on systems without oFono, failing
to register with org.ofono is not necessarily an error.

This lowers the failure message log level from error to info.

i18n: update Italian translation

Signed-off-by: Milo Casagrande <milo@milo.name>

context: pa_context_get_tile_size: constify

context: pa_context_get_index: constify

context: pa_context_get_server_protocol_version: constify

context: pa_context_get_server: constify

context: pa_context_is_local: constify

context: pa_context_is_pending: constify

context: constify pa_context_set_error()

This allows constifying public API functions that report their errors
via the context error but don't modify the context in any other way.
Philosophical arguments could be made why this is wrong, but I believe
in practice this is a net positive change.

context: pa_context_rttime_restart: constify context pointer

context: pa_context_rttime_new: constify context pointer

context: pa_context_errno: constify

context: hide error attr behind pointer

Paves the way towards more of the API using const pointers.

Some pa_context_* functions return their errors by setting the context
error, even when there's no other change in the context state. This
prevented constifying the pa_context arguments of such functions. This
patch puts the error in its own struct behind a pointer, so that setting
the error doesn't any more count as modifying the pa_context object.

stream: constify internal functions

alsa-mixer: Don't move LFE in 2.1 and 4.1 modes on SB Omni Surround 5.1

A bit hacky approach, but it allows to preserve LFE output position
even in reduced output modes 2.1 and 4.1.

Signed-off-by: Nazar Mokrynskyi <nazar@mokrynskyi.com>

cli-command: Report error in pa_play_file.

Current code does not check whether pa_play_file call failed. Hence no error is
reported in the cli interface if playback failed because e.g. file isn't
readable by the daemon.

stream: pa_stream_get_monitor_stream: constify

stream: pa_stream_get_format_info: constify

stream: pa_stream_get_underflow_index: constify